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

Tab navigation doesn't respect removed panels by the user #8352

Closed
Inkrementator opened this issue Aug 11, 2022 · 1 comment · Fixed by #8435
Closed

Tab navigation doesn't respect removed panels by the user #8352

Inkrementator opened this issue Aug 11, 2022 · 1 comment · Fixed by #8435
Labels
Milestone

Comments

@Inkrementator
Copy link

Overview

Using tab to switch focus between elements of the GUI doesn't work as expected if I remove the database tags pane. Pressing in the group pane moves focus somewhere I can't see, presumable the now invisible database tags pane. Similarly, pressing <Shift+Tab> in the Database-Entry List also moves focus off-screen.

Steps to Reproduce

  1. Click on one of the database groups to move focus there
  2. Press to verify that focus is moved to database pane, pressing tab again moves to database entries.
  3. Remove database tags pane by dragging the upper border of it as far down as possible.
  4. Click on one of the database groups to move focus there
  5. Press once. Now it is not visible, where the focus is.
  6. Press again to move to database entries.

Expected Behavior

Pressing tab once from database group pane while database tags pane is removed should move focus directly to database entries. Step 5. should not happen.

Actual Behavior

Pressing tab once from database group pane moves focus to invisible location, presumably to the nonexistant database tags. Pressing again allows to go to database entries.

Context

KeePassXC - Version 2.7.1
Revision: 5916a8f

Operating System: Linux
Desktop Env: i3
Windowing System: X11

@droidmonkey
Copy link
Member

I have fixed this for 2.7.2, will be in upcoming PR

@droidmonkey droidmonkey added this to the v2.7.2 milestone Aug 23, 2022
droidmonkey added a commit that referenced this issue Sep 1, 2022
droidmonkey added a commit that referenced this issue Sep 1, 2022
droidmonkey added a commit that referenced this issue Sep 3, 2022
droidmonkey added a commit that referenced this issue Sep 3, 2022
droidmonkey added a commit that referenced this issue Sep 5, 2022
droidmonkey added a commit that referenced this issue Sep 5, 2022
droidmonkey added a commit that referenced this issue Sep 5, 2022
droidmonkey added a commit that referenced this issue Sep 5, 2022
droidmonkey added a commit that referenced this issue Sep 5, 2022
droidmonkey added a commit that referenced this issue Sep 7, 2022
droidmonkey added a commit that referenced this issue Sep 7, 2022
droidmonkey added a commit that referenced this issue Sep 7, 2022
droidmonkey added a commit that referenced this issue Sep 7, 2022
droidmonkey added a commit that referenced this issue Sep 7, 2022
droidmonkey added a commit that referenced this issue Sep 7, 2022
droidmonkey added a commit that referenced this issue Sep 8, 2022
pull bot pushed a commit to annihilatorrrr/keepassxc that referenced this issue Sep 8, 2022
pull bot pushed a commit to tigerwill90/keepassxc that referenced this issue Sep 8, 2022
droidmonkey added a commit that referenced this issue Sep 11, 2022
droidmonkey added a commit that referenced this issue Sep 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants