Skip to content

Commit

Permalink
Menus (dropdowns) with separator: Fix hidden keyboard focus
Browse files Browse the repository at this point in the history
When opening a menu as in Calc > Home > Conditional menu and then
press tab: we can successfully cycle through the elements but for some
elements user cannot see the focus.
 - The focus was being set but under the separator. So, better to
 assure that the immediate separator neighbor is set to transparent.

Signed-off-by: Pedro Pinto Silva <[email protected]>
Change-Id: Icb87b3173144356efad101f5e2b9b77e0e8dcd06
  • Loading branch information
pedropintosilva committed Oct 29, 2023
1 parent 2b5eb98 commit d6375f8
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions browser/css/jsdialogs.css
Original file line number Diff line number Diff line change
Expand Up @@ -218,6 +218,9 @@ td.jsdialog > [id^='table-box']:not(.sidebar) {

/* Separator */

.jsdialog:focus-visible + .jsdialog.ui-separator.vertical {
background-color: transparent;
}
.jsdialog.ui-separator.vertical {
height: 100%;
padding-right: 6px;
Expand Down

0 comments on commit d6375f8

Please sign in to comment.