Skip to content

Commit

Permalink
[EuiFacetButton] Remove euiLinkFocusCSS usage
Browse files Browse the repository at this point in the history
- just write out the style manually 🤷 honestly, it might need a redesign at some point anyway

+ clean up styles and how they're written
  • Loading branch information
cee-chen committed Feb 23, 2024
1 parent 8753ca1 commit 33e8448
Showing 1 changed file with 12 additions and 15 deletions.
27 changes: 12 additions & 15 deletions src/components/facet/facet_button.styles.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,33 +14,30 @@ import {
logicalTextAlignCSS,
} from '../../global_styling';

import { euiLinkFocusCSS } from '../link/link.styles';

export const euiFacetButtonStyles = ({ euiTheme }: UseEuiTheme) => ({
// Base
euiFacetButton: css`
${logicalTextAlignCSS('left')}
&:hover,
&:focus {
/* Make sure the quantity doesn't get an underline on hover */
&:not(:disabled) [class*='euiFacetButton__text'] {
text-decoration: underline;
&:not(:disabled) {
&:hover,
&:focus {
/* Make sure the quantity doesn't get an underline on hover */
text-decoration: none;
.euiFacetButton__text {
text-decoration: underline;
}
}
}
&:focus:not(:disabled) [class*='euiFacetButton__text'] {
${euiLinkFocusCSS(euiTheme)}
&:focus .euiFacetButton__text {
text-decoration-thickness: ${euiTheme.border.width.thick};
}
}
&:disabled {
color: ${euiTheme.colors.disabledText};
pointer-events: none;
&:hover,
&:focus {
text-decoration: none;
}
}
`,
});
Expand Down

0 comments on commit 33e8448

Please sign in to comment.