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

Use of keyboard shortcut for context menu fails to show context menu #3140

Closed
affinityv opened this issue May 12, 2019 · 6 comments · Fixed by #3409
Closed

Use of keyboard shortcut for context menu fails to show context menu #3140

affinityv opened this issue May 12, 2019 · 6 comments · Fixed by #3409
Labels

Comments

@affinityv
Copy link

affinityv commented May 12, 2019

Mouse can right-click on an entry to get the context menu, but the normal keyboard method of bringing up the context menu doesn't work. This is "<ctrl><f10>" .... it works with other applications.

Expected Behavior

Use keyboard shortcut should show context menu.

Current Behavior

No menu is shown using keyboard shortcut (mouse is okay).

Steps to Reproduce

Select an entry, via keyboard or mouse, it doesn't matter.
Use "<ctrl><f10>" -- no menu

Context

Probably not relevant, but this is on a MacbookPro11,3 -- have to use FN key as well as <ctrl><f10>.
(running Linux, not MacOS as per the debug info)

Debug Info

KeePassXC - Version 2.4.1
Revision: 7bafe65
Distribution: AppImage

Qt 5.10.1
Debugging mode is disabled.

Operating system: Devuan GNU/Linux ascii
CPU architecture: x86_64
Kernel: linux 4.9.0-6-amd64

Enabled extensions:

  • Auto-Type
  • Browser Integration
  • SSH Agent
  • KeeShare (only unsigned sharing)
  • YubiKey

Cryptographic libraries:
libgcrypt 1.8.1

@affinityv affinityv added the bug label May 12, 2019
@affinityv
Copy link
Author

No updates on this? I thought it would have been fixed for 2.4.2, but it is still the same.

@affinityv
Copy link
Author

New debug info:

KeePassXC - Version 2.4.2
Revision: a775031
Distribution: AppImage

Qt 5.10.1
Debugging mode is disabled.

Operating system: Devuan GNU/Linux ascii
CPU architecture: x86_64
Kernel: linux 4.9.0-6-amd64

Enabled extensions:

  • Auto-Type
  • Browser Integration
  • SSH Agent
  • KeeShare (only unsigned sharing)
  • YubiKey

Cryptographic libraries:
libgcrypt 1.8.1

@droidmonkey
Copy link
Member

This was not marked for 2.4.2? I have implemented this in an upcoming PR for accessibility. It's aimed at 2.5.0.

@affinityv
Copy link
Author

Okay, so is 2.5.0 going to be after 2.4.3 or further down the track?

@droidmonkey
Copy link
Member

2.5.0 is coming out next. Probably in 1 month.

@affinityv
Copy link
Author

Great, thanks.

droidmonkey added a commit that referenced this issue Jul 28, 2019
* Add Ctrl+F10 keyboard shortcut to show group/entry context menus. Fixes #3140
* Improve movement between form fields
droidmonkey added a commit that referenced this issue Aug 29, 2019
* Add Ctrl+F10 keyboard shortcut to show group/entry context menus. Fixes #3140
* Improve movement between form fields
droidmonkey added a commit that referenced this issue Aug 31, 2019
* Add application settings reset button
 - Corrects accessibility findings GP.2

* Use icons in addition to color to indicate password mismatch
 - Corrects accessibility finding CN.2

* Announce begin/end of list navigation
 - Corrects accessibility finding KF.4

* Fixes for keyboard navigation
 - Add Ctrl+F10 keyboard shortcut to show group/entry context menus. Fixes #3140
 - Improve movement between form fields

* Fix loading system-defined language in translator
 - Fixes #3202
 - Bypass built-in Qt loading of QLocale for translations. The order of loading languages doesn't consider all file names prior to moving to the next language in the list. This resulted in English being chosen no matter what language is the top priority.

* Improve message box defaults and fix documentation links

* Better support for screen readers

* Add accessible names on form fields

* Prevent changing values during settings widget scrolling
 - Add an event filter to combo boxes and spin boxes on the settings page to prevent the mouse wheel from changing the values without having focus
 - Add horizontal stretch to the security settings to make the spin boxes more manageable.
scoroi pushed a commit to scoroi/keepassxc that referenced this issue Nov 10, 2019
* Add application settings reset button
 - Corrects accessibility findings GP.2

* Use icons in addition to color to indicate password mismatch
 - Corrects accessibility finding CN.2

* Announce begin/end of list navigation
 - Corrects accessibility finding KF.4

* Fixes for keyboard navigation
 - Add Ctrl+F10 keyboard shortcut to show group/entry context menus. Fixes keepassxreboot#3140
 - Improve movement between form fields

* Fix loading system-defined language in translator
 - Fixes keepassxreboot#3202
 - Bypass built-in Qt loading of QLocale for translations. The order of loading languages doesn't consider all file names prior to moving to the next language in the list. This resulted in English being chosen no matter what language is the top priority.

* Improve message box defaults and fix documentation links

* Better support for screen readers

* Add accessible names on form fields

* Prevent changing values during settings widget scrolling
 - Add an event filter to combo boxes and spin boxes on the settings page to prevent the mouse wheel from changing the values without having focus
 - Add horizontal stretch to the security settings to make the spin boxes more manageable.
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