This commit fixes an issue where there were multiple buttons were inaccessible using only keyboard.
Two main problems caused this:
Relying on :hover to change behavior is never accessible
Toggling the action buttons using display: none removes the ability to focus on them.
The simplest "fix" was using opacity:0 and opacity: 100% to control if the icon buttons are visible. This way, a keyboard user can tab through the buttons... they would just be invisible.
The next fix was to use the &:has(:focus) .NodeActions to grant visibility to the buttons if there is any focus inside of the .NodeParent.
This commit fixes an issue where there were multiple buttons were inaccessible using only keyboard.
Two main problems caused this:
:hover
to change behavior is never accessibledisplay: none
removes the ability to focus on them.The simplest "fix" was using
opacity:0
andopacity: 100%
to control if the icon buttons are visible. This way, a keyboard user can tab through the buttons... they would just be invisible.The next fix was to use the
&:has(:focus) .NodeActions
to grant visibility to the buttons if there is any focus inside of the.NodeParent
.FIXED=259452291, 259452367, 259452413