Skip to content

Commit 3d44af6

Browse files
authored
Fix combo-box item identation not working (#399)
1 parent 0135e60 commit 3d44af6

File tree

1 file changed

+105
-50
lines changed

1 file changed

+105
-50
lines changed

packages/mercury/src/components/combo-box/_combo-box-styles.scss

Lines changed: 105 additions & 50 deletions
Original file line numberDiff line numberDiff line change
@@ -15,18 +15,83 @@
1515
@include control-font-size-regular();
1616
@include control-padding-inline-block();
1717

18-
&:hover {
18+
&--hover {
1919
@include control-colors-hover();
2020
}
2121

22+
&--focus {
23+
@include focus-border();
24+
}
25+
26+
&--disabled {
27+
@include control-colors-disabled();
28+
}
29+
30+
&--error {
31+
@include control-colors-error();
32+
}
33+
34+
&__window {
35+
@include items-container-border();
36+
@include items-container-border-radius();
37+
@include items-container-colors();
38+
@include items-container-padding();
39+
@include items-container-gap();
40+
@include items-container-max-height();
41+
@include items-container-dropdown();
42+
}
43+
44+
&-group__header {
45+
@include item-colors-enabled();
46+
@include item-font-header();
47+
@include item-padding();
48+
49+
&--expandable {
50+
@include item-colors-enabled();
51+
@include item-font-header();
52+
@include item-padding();
53+
@include item-border-radius();
54+
55+
&-hover {
56+
@include item-colors-hover();
57+
}
58+
}
59+
60+
&--disabled {
61+
@include item-colors-disabled();
62+
}
63+
}
64+
65+
&-group {
66+
@include items-container-gap();
67+
68+
&__content {
69+
@include items-container-gap();
70+
}
71+
}
72+
2273
&__item {
74+
@include item-colors-enabled();
75+
@include item-border();
76+
@include item-border-radius();
77+
@include item-font();
78+
@include item-padding();
79+
2380
&--nested {
2481
padding-inline-start: calc(
2582
var(--ch-combo-box-item-gap) +
2683
var(--ch-combo-box-group__expandable-button-size) +
2784
var(--mer-spacing--2xs)
2885
);
2986
}
87+
88+
&--selected {
89+
@include item-colors-selected();
90+
}
91+
92+
&--disabled {
93+
@include item-colors-disabled();
94+
}
3095
}
3196
}
3297

@@ -47,13 +112,13 @@
47112
/// @param {String} $item--selected-selector [".combo-box::part(item selected)"] -
48113
/// @param {Boolean} $add--disabled [true] -
49114
/// @param {Boolean} $add--placeholder [true] -
50-
/// @param {Boolean} $add--placeholder-var [true] -
51115
/// @param {Boolean} $add--error [true] -
52116
@mixin combo-box(
53117
$combo-box-selector: ".combo-box",
54118
$combo-box--disabled-selector: ".combo-box[disabled]",
55119
$combo-box--error-selector: ".combo-box-error",
56-
$combo-box__placeholder-selector: ".combo-box::part(placeholder)",
120+
$combo-box__placeholder-selector:
121+
".combo-box[part='ch-combo-box--empty-value']",
57122
$window-selector: ".combo-box::part(window)",
58123
$group-selector: ".combo-box::part(group)",
59124
$group__content-selector: ".combo-box::part(group__content)",
@@ -67,17 +132,17 @@
67132
$item--selected-selector: ".combo-box::part(item selected)",
68133
$add--disabled: true,
69134
$add--placeholder: true,
70-
$add--placeholder-var: true,
71135
$add--error: true
72136
) {
73137
#{$combo-box-selector} {
74138
@extend %combo-box;
75139

76-
&:focus {
77-
@include focus-border();
140+
&:hover {
141+
@extend %combo-box--hover;
78142
}
79143

80-
@if $add--placeholder-var {
144+
&:focus {
145+
@extend %combo-box--focus;
81146
}
82147
}
83148

@@ -87,81 +152,71 @@
87152
}
88153
}
89154

90-
@if $add--disabled {
91-
#{$combo-box--disabled-selector} {
92-
@include control-colors-disabled();
93-
}
94-
95-
#{$group__header--disabled-selector} {
96-
@include item-colors-disabled();
97-
}
98-
99-
#{$item--disabled-selector} {
100-
@include item-colors-disabled();
101-
}
102-
}
103-
104-
@if $add--error {
105-
#{$combo-box--error-selector} {
106-
@include control-colors-error();
107-
}
108-
}
109-
110155
// - - - - - - - - - - - - - - - - - - - -
111156
// Window
112157
// - - - - - - - - - - - - - - - - - - - -
113158
#{$window-selector} {
114-
@include items-container-border();
115-
@include items-container-border-radius();
116-
@include items-container-colors();
117-
@include items-container-padding();
118-
@include items-container-gap();
119-
@include items-container-max-height();
120-
@include items-container-dropdown();
159+
@extend %combo-box__window;
121160
}
122161

123162
// - - - - - - - - - - - - - - - - - - - -
124163
// Group
125164
// - - - - - - - - - - - - - - - - - - - -
126165
#{$group__header-selector} {
127-
@include item-colors-enabled();
128-
@include item-font-header();
129-
@include item-padding();
166+
@extend %combo-box-group__header;
130167
}
131168

132169
#{$group__header--expandable-selector} {
133-
@include item-colors-enabled();
134-
@include item-font-header();
135-
@include item-padding();
136-
@include item-border-radius();
170+
@extend %combo-box-group__header--expandable;
137171

138172
&:hover {
139-
@include item-colors-hover();
173+
@extend %combo-box-group__header--expandable-hover;
140174
}
141175
}
176+
142177
#{$group-selector} {
143-
@include items-container-gap();
178+
@extend %combo-box-group;
144179
}
180+
145181
#{$group__content-selector} {
146-
@include items-container-gap();
182+
@extend %combo-box-group__content;
147183
}
148184

149185
// - - - - - - - - - - - - - - - - - - - -
150186
// Item
151187
// - - - - - - - - - - - - - - - - - - - -
152188
#{$item-selector} {
153-
@include item-colors-enabled();
154-
@include item-border();
155-
@include item-border-radius();
156-
@include item-font();
157-
@include item-padding();
189+
@extend %combo-box__item;
158190
}
159191

160192
#{$item--nested-selector} {
161193
@extend %combo-box__item--nested;
162194
}
163195

164196
#{$item--selected-selector} {
165-
@include item-colors-selected();
197+
@extend %combo-box__item--selected;
198+
}
199+
200+
// - - - - - - - - - - - - - - - - - - - -
201+
// Disabled
202+
// - - - - - - - - - - - - - - - - - - - -
203+
@if $add--disabled {
204+
#{$combo-box--disabled-selector} {
205+
@extend %combo-box--disabled;
206+
}
207+
208+
#{$group__header--disabled-selector} {
209+
@extend %combo-box-group__header--disabled;
210+
}
211+
212+
#{$item--disabled-selector} {
213+
@extend %combo-box__item--disabled;
214+
}
215+
}
216+
217+
@if $add--error {
218+
#{$combo-box--error-selector} {
219+
@extend %combo-box--error;
220+
}
166221
}
167222
}

0 commit comments

Comments
 (0)