Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Focus issue when keyboard navigating with backtick #797

Open
flavorjones opened this issue Jan 23, 2024 · 2 comments
Open

Focus issue when keyboard navigating with backtick #797

flavorjones opened this issue Jan 23, 2024 · 2 comments

Comments

@flavorjones
Copy link

Reproduction:

  • start in bundled inbox view with "priority inbox" configuration
  • have a bundle as the first entry in one of the inbox sections, which is out of the viewport (needs to be scrolled to)
  • use the backtick key to navigate between sections
  • when focus is given to the bundle, the viewport does not scroll to that section

I'm attaching a gif which attempts to show what's going on. I'm happy to re-record or explain more if what's happening isn't clear.

Peek 2024-01-23 11-53 gmail simplify focus


Simplify v.3.1.6 configuration: aAX boldHighlight bundleInbox caA cfF cfQ cfR cfS composeFab dateGroup defaultTheme gaArchive gaRemove groupHideFirst groupInbox groupOtherLists hideAds hideEmptySections hideExtWarn hideFileIcons hideInboxHideAllLink hideLabelChips hideListCount hideMsgCount hideMsgReactions hideMsgReplies hideNavLabels hideNavUnreads hideSearchChips hideSelectRefresh hideSignatures hideTabIcons highDensity inInbox inList invertAddons invertCompose izBgOff lightTheme listWidthLg maArchive maDelete maDiv maDone maLabel maMark maMove maPin maSnooze maSpam maTask matYou msgWidthSm newNav newUI reminders rightImportance rightLabels simplify simplifyPrefsOpen- System: Mozilla/5.0 (X11 Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/120.0.0.0 Safari/537.36 - gmail.pinto-server_20240115.06_p1 - Window: 1200 x 1209 (85pct zoom) - Language: en - Errors: / 11549:16: Uncaught TypeError: Cannot read properties of null (reading 'classList') / 2186:54: clickOnWhenReady(tbody.bundled tr.zA[id=":gh"]) timed out

@leggett
Copy link
Owner

leggett commented Jan 24, 2024

I had NO idea that backtick jumped between sections... I thought it only jumped between tabs in the default inbox.

I need to detect and intercept this shortcut (like I do with so many others) in a bundled inbox so I can make it work as expected. Thanks for the report!

@flavorjones
Copy link
Author

@leggett I just noticed that this also happens if I use j to go down one message to a bundled set of messages that's out of view.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants