Repository navigation
gui: Add keyboard shortcuts to history browser tree - #7423
polucifier wants to merge 3 commits into
Conversation
polucifier
changed the title
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
e6e75ae to
b552d89
Compare
|
Hi! Thanks for implementing this and sorry for the late response (do not be afraid to just call for review :-). |
|
Btw, delete shortcuts would be helpful also in other "removing" situations, mainly in Data Catalog, see Delete point in #5425: • Delete • CTRL + Shift + Delete If you want, you are welcome to implementing these as well :-) |
Implement Ctrl+C for copying commands, the Delete key for removing commands from the history browser tree, and Enter/Return to open a command in the GUI, equivalent to double-clicking it. This improves accessibility and UX for users who prefer keyboard shortcuts over context menus.
Keep only the Delete key, as other software reserves Ctrl+D for duplicating. See OSGeo#5285 for the discussion on shortcut conventions.
4b441f6 to
41d3119
Compare
Move removal of a command to EVT_KEY_UP so that holding the key does not remove several commands in a row. This matches the layer tree and the graphical modeler, which handle removal on key up as well. Opening a command and copying stay on key down.
Description
This Pull Request implements the requested keyboard shortcuts for the history browser tree, improving accessibility and overall user experience. Users who prefer using the keyboard over context menus can now manage their command history much faster.
Key Changes
COMMAND), preventing accidental modification or deletion of day/time period header nodes (TIME_PERIOD).Related PRs / Issues
How to Test
Ctrl + Cand try pasting it into a text editor to verify it was copied.Deleteon a command, confirm the dialog, and verify it was removed.