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

Drop the conflicting Ctrl+Alt+A shortcut #38608

Closed

Conversation

borysiasty
Copy link
Member

A few months ago (24e1a82) the Ctrl+Shift+A shortcut (Deselect all features from all layers) was replaced with Ctrl+Alt+A, as the old one has been used for a new action, Deselect features only from the current layer (452ca91).

Unfortunately in Windows Ctrl+Alt+A is an alias to GrAlt+A, what is used for national characters in some localisations (Polish for example, but according to Wikipedia also in Belgian, Latvian, Turkish, UK, Irish and Dutch), so that change broke the Locator bar and possibly other text inputs in scope of the map canvas. By the way, in Polish Debian/KDE the new shortcut seems to not work at all.

I propose to just drop that troublesome shortcut, leaving the old action (for all layers) without any. I can't find any other reasonable and safe shortcut for that action, and if there is only Ctrl+Alt+A, the new action (current layer only) probably deserves more for it. It's also consistent with its use in the Attribute table.

@suricactus is it ok for you?

…used for diacritical characters in some localizations. Reverts 24e1a82
@borysiasty borysiasty added GUI/UX Related to QGIS application GUI or User Experience Regression Something which used to work, but doesn't anymore Bug Either a bug report, or a bug fix. Let's hope for the latter! backport release-3_14 labels Sep 4, 2020
@github-actions github-actions bot added this to the 3.16.0 milestone Sep 4, 2020
@suricactus
Copy link
Contributor

suricactus commented Sep 4, 2020

@borysiasty aah, didn't know that it is an alias to GrAlt. :( It is fine with me, but I will reference the intitial discussion #35189 . Maybe the others have objections?

@borysiasty
Copy link
Member Author

@suricactus me too, just win users told me they can't type ą in Locator...

I didn't know that discusssion, let's go there.

@agiudiceandrea
Copy link
Contributor

agiudiceandrea commented Sep 5, 2020

@borysiasty @suricactus Please see this #29021 similar oldstanding unresolved issue with Ctrl+Alt++ on Windows systems with italian or spanish keyboard.

@borysiasty
Copy link
Member Author

So we have 13 Ctrl+Alt shortcuts. Almost all of them are used in one or more localisations (the last column seem to be the least troublesome):

Ctrl+Alt+A    Ctrl+Alt++    Ctrl+Alt+/
Ctrl+Alt+M    Ctrl+Alt+-    Ctrl+Alt+[
Ctrl+Alt+P    Ctrl+Alt+=    Ctrl+Alt+]
Ctrl+Alt+V    Ctrl+Alt+;    Ctrl+Alt+{
                            Ctrl+Alt+}

One solution is to drop all Ctrl+Alt shortcuts (what will hurt users from not affected languages).
Another is to only change or drop them in translation for affected languages.

Personally I'm ok with both.

@agiudiceandrea
Copy link
Contributor

One solution is to drop all Ctrl+Alt shortcuts (what will hurt users from not affected languages).
Another is to only change or drop them in translation for affected languages.

Would it be possible to remove the Ctrl+Alt+ shortcuts only when the Python console is active or, more generally, when the cursor is in an active editable field/place?

@nyalldawson
Copy link
Collaborator

Another is to only change or drop them in translation for affected languages.

I'm +1 to this -- I'm not too keen on loss of usability as a result of the specific locale handling.

@nirvn
Copy link
Contributor

nirvn commented Sep 7, 2020

+1 too.

@agiudiceandrea
Copy link
Contributor

One solution is to drop all Ctrl+Alt shortcuts (what will hurt users from not affected languages).

Anyway, the discriminant is not the interface translation language, but the keyboard layout: on a system with an italian or spanish keyboard layout the issue is present non only when the italian or spanish user interface translation is set, but also when the system locale is overridden and the American English user interface is set.

@borysiasty
Copy link
Member Author

Would it be possible to remove the Ctrl+Alt+ shortcuts only when the Python console is active or, more generally, when the cursor is in an active editable field/place?

Not easily. They are main window action shortcuts and work globally. Theoretically we could capture them in Py console but I guess it would be much easier to use them for something new than to revert to the OS-default action. And we could need to implement it everywhere: in Locator, Browser filter etc.

I'm +1 to this -- I'm not too keen on loss of usability as a result of the specific locale handling.

It's probably a dozen locales affected, but only one OS. So I'd rather say: as a result of supporting that particular OS ;)

Well, I'm also not too keen to hurt 80% of users for those 20 (those values are totally guessed), but it seems Ctrl+Alt shortcuts are in general not good for Windows.

Anyway, the discriminant is not the interface translation language, but the keyboard layout: on a system with an italian or spanish keyboard layout the issue is present non only when the italian or spanish user interface translation is set, but also when the system locale is overridden and the American English user interface is set.

Good point. It's not a big problem here in PL as almost anyone use localised interface, but there was a discussion about translation quality recently and if I remember correctly many users in Spain or Portugal (please forgive my poor memory) use the EN version...

@stale
Copy link

stale bot commented Oct 4, 2020

The QGIS project highly values your contribution and would love to see this work merged! Unfortunately this PR has not had any activity in the last 14 days and is being automatically marked as "stale". If you think this pull request should be merged, please check

  • that all unit tests are passing

  • that all comments by reviewers have been addressed

  • that there is enough information for reviewers, in particular

    • link to any issues which this pull request fixes

    • add a description of workflows which this pull request fixes

    • add screenshots if applicable

  • that you have written unit tests where possible
    In case you should have any uncertainty, please leave a comment and we will be happy to help you proceed with this pull request.
    If there is no further activity on this pull request, it will be closed in a week.

@stale stale bot added the stale Uh oh! Seems this work is abandoned, and the PR is about to close. label Oct 4, 2020
@stale
Copy link

stale bot commented Oct 11, 2020

While we hate to see this happen, this PR has been automatically closed because it has not had any activity in the last 21 days. If this pull request should be reconsidered, please follow the guidelines in the previous comment and reopen this pull request. Or, if you have any further questions, just ask! We love to help, and if there's anything the QGIS project can do to help push this PR forward please let us know how we can assist.

@stale stale bot closed this Oct 11, 2020
@agiudiceandrea
Copy link
Contributor

@borysiasty @suricactus @nirvn why don't go ahead with this PR? The issue is still present in addition to #29021.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! GUI/UX Related to QGIS application GUI or User Experience NOT FOR MERGE Don't merge! Regression Something which used to work, but doesn't anymore stale Uh oh! Seems this work is abandoned, and the PR is about to close.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants