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

[needs-docs] Avoid duplicate use of Ctrl-D shorcut #7196

Closed
wants to merge 1 commit into from

Conversation

DelazJ
Copy link
Contributor

@DelazJ DelazJ commented Jun 6, 2018

Fixes #16702
This shortcut, also used to constrain the distance in the CAD panel, is never triggered in that case and instead tries to save the edits and remove the layer in edit.

Fixes qgis#16702
This shortcut, also used to constrain the distance in the CAD panel, is never triggered in that case and instead tries to save the edits and remove the layer in edit.
@nyalldawson
Copy link
Collaborator

I'm -1 on this change. Remove layer is a very common task (much more common then using the distance constraint) and there's lots of resources, books, etc listing this shortcut. Plus it's baked into my muscle memory!

Can we change the constraint shortcut instead?

@DelazJ
Copy link
Contributor Author

DelazJ commented Jun 7, 2018

hmmmm... For consistency, we would need to find a combination that would work with (x, y, a, d) without concurrence. It happens that "Alt" is enabled for the same feature and works well except for d, where the Database menu is instead triggered --> impossible to use ctrl-D, Alt-D.
But I wonder if it does not have with some missing context instructions (or whatever it's called) on this button because pressing Ctrl-A does not select all features and as expected locks the angle while Ctrl-D tries to remove the layer instead of locking the distance (as well as Alt-D opens database menu).

Otherwise, the alternative will be to move the lock to Shift+(adxy) and use ctrl to toggle the relative placement (which currently uses Shift). @3nids?

@nyalldawson
Copy link
Collaborator

Superseded by #7280

@DelazJ DelazJ deleted the patch-2 branch June 20, 2018 13:08
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.

2 participants