Page MenuHomePhabricator

Mark mixin imports consistently as `reference`
Closed, ResolvedPublicFeature

Description

Background

Mixins are implicit references.
We're currently mixin import statements for mixins:

Combobox.vue

@import ( reference ) '@wikimedia/codex-design-tokens/theme-wikimedia-ui.less';
@import './../../themes/mixins/icon-alignment.less';
@import './../../themes/mixins/element-with-menu-expanded.less';

Tabs.vue

@import ( reference ) '@wikimedia/codex-design-tokens/theme-wikimedia-ui.less';
@import ( reference ) '../../themes/mixins/common.less';

Let's settle on one way: the more explicit, probably clearer from dev experience view and more fail-safe would be to always use reference.

Acceptance criteria for Done

  • Settle on which path to choose: @import ( reference ) as clearer dev experience and more failsafe
    • Unify import statements in code base

Event Timeline

Change 863054 had a related patch set uploaded (by Catrope; author: Catrope):

[design/codex@main] styles: Make mixin imports consistent

https://gerrit.wikimedia.org/r/863054

Change 863054 merged by jenkins-bot:

[design/codex@main] styles: Make mixin imports consistent

https://gerrit.wikimedia.org/r/863054

Change 865151 had a related patch set uploaded (by Eric Gardner; author: Eric Gardner):

[mediawiki/core@master] Update Codex from v0.3.0 to v0.4.0

https://gerrit.wikimedia.org/r/865151

Change 865151 merged by jenkins-bot:

[mediawiki/core@master] Update Codex from v0.3.0 to v0.4.0

https://gerrit.wikimedia.org/r/865151