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

Shortcut Instead Brings Up Keyboard Preferences Dialog #1358

Closed
JohnColket opened this issue Jul 24, 2021 · 8 comments · Fixed by #1583
Closed

Shortcut Instead Brings Up Keyboard Preferences Dialog #1358

JohnColket opened this issue Jul 24, 2021 · 8 comments · Fixed by #1583
Assignees
Labels
bug An error, undesired behaviour, or missed functionality P2 High priority bugs regression Bug is a regression against previous version
Milestone

Comments

@JohnColket
Copy link
Contributor

JohnColket commented Jul 24, 2021

There are two reports on the forum for MacOS users where trying to use a keyboard shortcut will instead bring up the Keyboard Preferences Dialog.

See: https://forum.audacityteam.org/viewtopic.php?f=70&t=119343
and: https://forum.audacityteam.org/viewtopic.php?p=410469#p410469

@petersampsonaudacity
Copy link

petersampsonaudacity commented Jul 25, 2021

I do not get this on SW10 with 3.0.2 or 3.0.3 RC3

@petersampsonaudacity petersampsonaudacity added the bug An error, undesired behaviour, or missed functionality label Jul 25, 2021
@JohnColket
Copy link
Contributor Author

JohnColket commented Jul 25, 2021

Note that this bug was originally reported on Bugzilla under Linux, but was closed April 26th of this year as "Works For Me" after not seeing it for over a year. See https://bugzilla.audacityteam.org/show_bug.cgi?id=2253

Note that most the most recent reports are under 3.0.2 on MacOS and also on Windows.

@JamesCrook JamesCrook added the P2 High priority bugs label Jul 25, 2021
@petersampsonaudacity
Copy link

petersampsonaudacity commented Jul 26, 2021

This was logged originally on Bugzilla, closed and then reopened>
https://bugzilla.audacityteam.org/show_bug.cgi?id=2253
Bug 2253 - Mac/Linux: Keyboard Shortcut Preferences may open unexpectedly

Paul Licameli wrote in the big thread.

I think it's because of the code controlled by EXPERIMENTAL_EASY_CHANGE_KEY_BINDINGS

The intent was to take you to keyboard preferences only when you shift+click in a menu item of the the main toolbar.

I don't understand how you get there without opening a menu.

If this is it, we could consider disabling this feature on Linux, or we could make the code implementing this test
that control is also not down.

James Crook wrote later in the bug thread

If not a Heisenbug on Macbook Air macOS 11.4, then it's P2 not P4, and RM should decide whether it should
block release or not.
The problem for sure is quite unnecessary, given Paul's comment

@petersampsonaudacity petersampsonaudacity added the regression Bug is a regression against previous version label Jul 26, 2021
@petersampsonaudacity
Copy link

petersampsonaudacity commented Jul 26, 2021

This bug is a regression on 2.2.1

@petersampsonaudacity petersampsonaudacity added this to To do (Dev) in Release 3.1 via automation Jul 26, 2021
@petersampsonaudacity petersampsonaudacity added this to the Audacity 3.1 milestone Jul 26, 2021
@AnitaBats AnitaBats added this to To do in Sprint 5 - Release 3.1 via automation Aug 19, 2021
@Paul-Licameli
Copy link
Member

Paul-Licameli commented Aug 19, 2021

Remember this feature: you can hold Shift while releasing a toolbar menu item, and go to Keyboard Preferences to change the hotkey of the item (instead of doing the command).

I added that in 2018, but it is mis-firing on Mac sometimes when there are shortcuts with Shift in them.

Quick fix might be just to eliminate my little known feature.

@Tantacrul
Copy link

Tantacrul commented Aug 19, 2021

@Paul-Licameli believes this has been caused by his inclusion of a Shift modifier to help users jump to a keyboard shortcut preference quickly.

After discussing it, neither of us feels this modifier is especially discoverable and perhaps we can simply remove it (to fix this bug) and focus more on improving the usability of shortcuts in the medium term

@petersampsonaudacity
Copy link

petersampsonaudacity commented Aug 19, 2021

I would vote for removing this little-know feature

Is that the cause of this P2 Bugzilla Bug ?
https://bugzilla.audacityteam.org/show_bug.cgi?id=2253
Bug 2253 - Mac/Linux: Keyboard Shortcut Preferences may open unexpectedly

@petersampsonaudacity
Copy link

petersampsonaudacity commented Aug 26, 2021

I have closed Bugzilla 2253 in favor of this GitHub Issue

Paul-Licameli added a commit to Paul-Licameli/audacity that referenced this issue Aug 31, 2021
…Dialog...

... Happened at least on macOS because the easy change of key bindings was
mis-firing in other cases than picks on menu items.

So we just disable that little valued feature.

Disable the easy key rebinding feature
Release 3.1 automation moved this from To do (Dev) to Done Aug 31, 2021
Sprint 5 - Release 3.1 automation moved this from To do to Ready for QA Aug 31, 2021
Paul-Licameli added a commit that referenced this issue Aug 31, 2021
... Happened at least on macOS because the easy change of key bindings was
mis-firing in other cases than picks on menu items.

So we just disable that little valued feature.

Disable the easy key rebinding feature
@Penikov Penikov moved this from Ready for QA to In QA in Sprint 5 - Release 3.1 Sep 3, 2021
@Penikov Penikov moved this from In QA to Done in Sprint 5 - Release 3.1 Sep 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An error, undesired behaviour, or missed functionality P2 High priority bugs regression Bug is a regression against previous version
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

5 participants