Skip to content

Commit 7a98682

Browse files
committed
Fix mobile select selected-state highlighting
1 parent 2b8d1ac commit 7a98682

2 files changed

Lines changed: 8 additions & 0 deletions

File tree

packages/web/src/styles/components.css

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -269,6 +269,10 @@
269269
background: color-mix(in srgb, var(--accent-blue) 10%, var(--bg-surface));
270270
}
271271

272+
.mobile-select-sheet__list > [data-selected="true"] > .mobile-select-sheet__item {
273+
background: color-mix(in srgb, var(--accent-blue) 10%, var(--bg-surface));
274+
}
275+
272276
.mobile-select-sheet__item-row:last-child {
273277
border-bottom: none;
274278
}

packages/web/src/styles/components.theme.test.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -298,6 +298,9 @@ describe("components.css theme-sensitive surfaces", () => {
298298
it("keeps mobile select row-side actions lightweight and token-driven", () => {
299299
const row = getLastRuleBlock(".mobile-select-sheet__item-row");
300300
const rowSelected = getLastRuleBlock('.mobile-select-sheet__item-row[data-selected="true"]');
301+
const plainSelected = getLastRuleBlock(
302+
'.mobile-select-sheet__list > [data-selected="true"] > .mobile-select-sheet__item'
303+
);
301304
const commandSelected = getLastRuleBlock(
302305
'.mobile-select-sheet--command .mobile-select-sheet__list > [data-selected="true"] > .mobile-select-sheet__item'
303306
);
@@ -307,6 +310,7 @@ describe("components.css theme-sensitive surfaces", () => {
307310
expect(row).toContain("display: flex");
308311
expect(row).toContain("padding: var(--sp-1) var(--sp-2)");
309312
expect(rowSelected).toContain("var(--accent-blue)");
313+
expect(plainSelected).toContain("var(--accent-blue)");
310314
expect(commandSelected).toContain("var(--accent-blue) 12%");
311315
expect(commandSelected).toContain("inset 2px 0 0");
312316
expect(sideAction).toContain("width: 40px");

0 commit comments

Comments
 (0)