Conversation
| /* width: auto; */ | ||
| flex-grow: 1; | ||
| } | ||
| label[data-component='FormControl.Label']:not([data-control-disabled]) { |
There was a problem hiding this comment.
Ignores the disabled ones:
| position: relative; | ||
| &:hover::before { | ||
| content: ''; | ||
| background-color: #fff2; |
There was a problem hiding this comment.
Is there a transparent hover style? This is an overlay so it must be translucent. If it doesn't exist, I'll use rgba(var(blah blah, fuchsia), 5%)
There was a problem hiding this comment.
Anyway I suggest trying this CSS for a while to make sure we nail the style. I feel this is already ok since it matches the toolbar button hover for example, but maybe there's a similarly-large component elsewhere.
There was a problem hiding this comment.
duhhh it's right here!! I'll copy this
There was a problem hiding this comment.
Copied background and border-radius from that component
| &:hover::before { | ||
| content: ''; | ||
| position: absolute; | ||
| z-index: -1; |
There was a problem hiding this comment.
Actually placed it under for an extremely small improvement:
before/after
There was a problem hiding this comment.
Essentially only visible when the var will fail
Test URLs
https://github.com/refined-github/refined-github/pull/9611/changes
Screenshot