Skip to content

fix: Guidelines Tool duplicated the Scale Bar Tool shortcut - #17

Open
doha-230 wants to merge 1 commit into
akhuettel:masterfrom
doha-230:pr/fix-duplicate-shortcut
Open

doha-230 wants to merge 1 commit into
akhuettel:masterfrom
doha-230:pr/fix-duplicate-shortcut

Conversation

@doha-230

Copy link
Copy Markdown

Bug:

  • Both the Scale Bar Tool and the Guidelines Tool were assigned Shift+F8 in CreateActions::createDigitize, and both QActions belong to the same QActionGroup, so one of the two tools could not be reached from the keyboard. The Scale Bar Tool keeps Shift+F8; the Guidelines Tool moves to Shift+F9, which is the next free shortcut in the digitize group (Shift+F2..F8 were taken).

The translation files need no update: after this change the Guidelines Tool has a new, untranslated shortcut string that falls back to the source text, while the existing Shift+F8 translation still applies to the Scale Bar Tool.

Tested: built and ran with Qt 6.8 (MSVC); Shift+F9 now activates the Guidelines Tool, Shift+F8 still activates the Scale Bar Tool.

Bug:
- Both the Scale Bar Tool and the Guidelines Tool were assigned Shift+F8 in
  CreateActions::createDigitize, and both QActions belong to the same
  QActionGroup, so one of the two tools could not be reached from the keyboard.
  The Scale Bar Tool keeps Shift+F8; the Guidelines Tool moves to Shift+F9,
  which is the next free shortcut in the digitize group (Shift+F2..F8 were
  taken).

The translation files need no update: after this change the Guidelines Tool has
a new, untranslated shortcut string that falls back to the source text, while
the existing Shift+F8 translation still applies to the Scale Bar Tool.

Tested: built and ran with Qt 6.8 (MSVC); Shift+F9 now activates the Guidelines
Tool, Shift+F8 still activates the Scale Bar Tool.

This branch has not been deployed

No deployments
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

Successfully merging this pull request may close these issues.

1 participant