fix(@clayui/popover): LPD-15615 Popover: focus is not redirected to popover when disableScroll prop is true

liferay / clay

A web implementation of the Lexicon Experience Language

http://clayui.com

Other

207 stars 470 forks source link

fix(@clayui/popover): LPD-15615 Popover: focus is not redirected to popover when disableScroll prop is true #5833

Closed ilzamcmed closed 3 months ago

ilzamcmed commented 3 months ago

fix(@clayui/popover): LPD-15615 Add focus on popover when disableScroll is enabled
fix(@clayui/css): LPD-15615 Add focus style on popover when disableScroll is enabled

Jira issue: https://liferay.atlassian.net/browse/LPD-15615

ilzamcmed commented 3 months ago

Hey @pat270 @matuzalemsteles This PR is ready for review! Thanks

pat270 commented 3 months ago

@ilzamcmed I removed border radius from the focus styles and added focus styles to Cadmin. I also removed the $focus variable from clay-popover-variant. Those are present in links to maintain backward compatibility with the 2.x Sass API. It was never existed in clay-popover-variant.

ilzamcmed commented 3 months ago

Thanks, @pat270! :tada: