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

Incorrect tab navigation ordering in application security settings #9765

Closed
nelson2tm opened this issue Aug 19, 2023 · 1 comment · Fixed by #10907
Closed

Incorrect tab navigation ordering in application security settings #9765

nelson2tm opened this issue Aug 19, 2023 · 1 comment · Fixed by #10907
Assignees
Milestone

Comments

@nelson2tm
Copy link

nelson2tm commented Aug 19, 2023

Overview

The tab ordering of checkboxes in some settings pages is incorrect.

Steps to Reproduce

  1. Open settings and open the security section.
  2. Use tab to try to navigate to Enable database quick unlock (Touch ID / Windows Hello).
  3. The checkbox below, Lock databases when session is locked or lid is closed, will be reached first.
  4. After reaching the last checkbox, the selection will jump up again and finally select the first checkbox.

Expected Behavior

The checkboxes should be navigated over from top to bottom.

Actual Behavior

The checkboxes are selected in an incorrect order, making it difficult to select these out of order items.

KeePassXC - 2.7.6
Revision: dd21def

Operating System: Windows 10

@nelson2tm nelson2tm added the bug label Aug 19, 2023
@nelson2tm nelson2tm changed the title Incorrect tab navigation ordering in security settings Incorrect tab navigation ordering in settings Aug 19, 2023
@droidmonkey
Copy link
Member

DOH, good catch

@droidmonkey droidmonkey added this to the v2.8.0 milestone Aug 19, 2023
@droidmonkey droidmonkey self-assigned this Aug 19, 2023
@droidmonkey droidmonkey changed the title Incorrect tab navigation ordering in settings Incorrect tab navigation ordering in application security settings Aug 19, 2023
c4rlo added a commit to c4rlo/keepassxc that referenced this issue Jun 16, 2024
droidmonkey pushed a commit that referenced this issue Jun 16, 2024
pull bot pushed a commit to tigerwill90/keepassxc that referenced this issue Jun 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants