Skip to content

Advanced Digitiser: remove Ctrl modifier from X/Y/Z lock shortcuts#65133

Open
Geojim wants to merge 2 commits intoqgis:masterfrom
Geojim:digitiser-taking-back-ctrl
Open

Advanced Digitiser: remove Ctrl modifier from X/Y/Z lock shortcuts#65133
Geojim wants to merge 2 commits intoqgis:masterfrom
Geojim:digitiser-taking-back-ctrl

Conversation

@Geojim
Copy link
Contributor

@Geojim Geojim commented Feb 27, 2026

Description

  • Removes Ctrl as a modifier for locking X/Y/Z coordinates in the advanced sketcher dock widget, leaving only Alt+X/Y/Z
  • Restores standard Ctrl+X (cut) and Ctrl+Y (redo) behavior when editing coordinate text fields. Ctrl+Z was never bound so this brings it inline with existing behavior
  • Updates tooltips to reflect the Alt-only shortcut

Fixes #64558

…ard Ctrl+X/Y/Z behavior

The Advanced Digitizing panel was intercepting Ctrl+X and Ctrl+Y for
coordinate locking, preventing standard Cut (Ctrl+X) and Redo (Ctrl+Y)
from working in the panel's text input fields. This matches the existing
approach for Key_Z which already used Alt-only to avoid conflicting with
Undo (Ctrl+Z).

Also fixes the Z lock tooltip which incorrectly referenced Ctrl+Z when
the code only responds to Alt+Z.

Fixes qgis#64558

https://claude.ai/code/session_01Y7C8ZoZvubrHdfSsxG7BE2
@github-actions github-actions bot added this to the 4.0.0 milestone Feb 27, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 27, 2026

🍎 MacOS Qt6 builds

Download MacOS Qt6 builds of this PR for testing.
This app is not notarized, run sudo xattr -d com.apple.quarantine /Applications/QGIS*.app to avoid the warning
(Built from commit 372c613)

🪟 Windows Qt6 builds

Download Windows Qt6 builds of this PR for testing.
(Built from commit 372c613)

@elpaso elpaso closed this Mar 2, 2026
@elpaso elpaso reopened this Mar 2, 2026
@elpaso elpaso added GUI/UX Related to QGIS application GUI or User Experience Digitizing Related to feature digitizing map tools or functionality labels Mar 2, 2026
@elpaso
Copy link
Contributor

elpaso commented Mar 2, 2026

Test failure seems unrelated.

@lbartoletti is this change ok with you or is there any reason to maintain the previous shortcuts?

@lbartoletti
Copy link
Member

sorry, I can't review/test at the moment, but the proposal seems reasonable to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Digitizing Related to feature digitizing map tools or functionality GUI/UX Related to QGIS application GUI or User Experience

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Advanced Digitizing - Ctrl X doesn't cut....

4 participants