You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This is a fun one, we received a bug report that hovering on a menu item causes a focus-ring to flicker into existence:
20250926-1922-16.6927621.mp4
After a fun debugging session this is caused due to a timing issue where :focus-visible is applied by the browser due to focusOnHover being true before :hover is applied which then removes the focus-ring style
Latest version
Description
This is a fun one, we received a bug report that hovering on a menu item causes a focus-ring to flicker into existence:
20250926-1922-16.6927621.mp4
After a fun debugging session this is caused due to a timing issue where :focus-visible is applied by the browser due to
focusOnHoverbeing true before :hover is applied which then removes the focus-ring stylehttps://github.com/jpmorganchase/salt-ds/blame/3ad2fc43d7f069ead1d493ee8ea9ea67f9ac5841/packages/core/src/menu/MenuItem.css#L24-L26
Steps to reproduce
.
Expected behavior
No response
Package name(s)
Core (@salt-ds/core)
Package version(s)
No response
Browser
Operating system
Are you a JPMorgan Chase & Co. employee?