Skip to content

Commit a1a3e7b

Browse files
committed
refactor: use static class names
This isn’t a 1:1 as there is now a slight hover effect when `aria-disabled`. I think it’s fine though.
1 parent 697326b commit a1a3e7b

File tree

1 file changed

+1
-8
lines changed

1 file changed

+1
-8
lines changed

app/components/Compare/FacetSelector.vue

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -77,14 +77,7 @@ function isCategoryNoneSelected(category: string): boolean {
7777
:disabled="facet.comingSoon"
7878
:aria-checked="isFacetSelected(facet.id)"
7979
:aria-label="facet.label"
80-
class="gap-1 px-1.5 rounded transition-colors focus-visible:outline-accent/70"
81-
:class="
82-
facet.comingSoon
83-
? 'text-fg-subtle/50 bg-bg-subtle border-border-subtle cursor-not-allowed'
84-
: isFacetSelected(facet.id)
85-
? 'text-fg-muted bg-bg-muted'
86-
: 'text-fg-subtle bg-bg-subtle border-border-subtle hover:text-fg-muted hover:border-border'
87-
"
80+
class="gap-1 px-1.5 rounded transition-colors text-fg-subtle bg-bg-subtle border-border-subtle enabled:hover:(text-fg-muted border-border) aria-checked:(text-fg-muted bg-bg-muted) focus-visible:outline-accent/70 disabled:(text-fg-subtle/50 bg-bg-subtle border-border-subtle cursor-not-allowed)"
8881
@click="!facet.comingSoon && toggleFacet(facet.id)"
8982
:classicon="
9083
facet.comingSoon

0 commit comments

Comments
 (0)