Skip to content

Commit 13b855b

Browse files
committed
fix(web): theme-aware hover on filter chips in dark mode
1 parent dcb1d6a commit 13b855b

1 file changed

Lines changed: 7 additions & 6 deletions

File tree

apps/web/src/components/search/floatingChipSx.ts

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,14 @@ import { TEAL } from "@/lib/theme";
66
* (category chips, the category filter bar toggles, the opening-times chip).
77
*
88
* Two variants exist because the rows diverge in a few precise ways:
9-
* - `category` (CategoryChips): scrollable row, so chips get `flexShrink: 0`
10-
* and the inactive hover uses the theme-aware `--omx-chip-hover`.
9+
* - `category` (CategoryChips): scrollable row, so chips get `flexShrink: 0`.
1110
* - `toggle` (CategoryFilterBar): the row wrapper has `pointerEvents: "none"`
12-
* so each chip re-enables `pointerEvents: "auto"`, adds label right-padding,
13-
* and uses `grey.300` for the inactive hover.
11+
* so each chip re-enables `pointerEvents: "auto"` and adds label
12+
* right-padding.
1413
*
15-
* The shared core (sizing, colors, icon styling, active hover) is identical.
14+
* The shared core (sizing, colors, icon styling, hover) is identical — both
15+
* use the theme-aware `--omx-chip-hover` so the inactive hover stays dark in
16+
* dark mode.
1617
*/
1718
export function floatingChipSx(active: boolean, variant: "category" | "toggle"): SxProps<Theme> {
1819
const base = {
@@ -43,7 +44,7 @@ export function floatingChipSx(active: boolean, variant: "category" | "toggle"):
4344
...base,
4445
pointerEvents: "auto",
4546
"& .MuiChip-label": { pr: "10px" },
46-
"&&:hover": { bgcolor: active ? "var(--omx-teal-hover)" : "grey.300" },
47+
"&&:hover": { bgcolor: active ? "var(--omx-teal-hover)" : "var(--omx-chip-hover)" },
4748
};
4849
}
4950

0 commit comments

Comments
 (0)