Cmd-R is "Refresh" AND "Run" #2652

Open
michaelkanis opened this Issue Jan 9, 2017 · 8 comments

Projects

None yet

5 participants

@michaelkanis

Depending on where the focus is, Cmd-R does a refresh or runs a query.

Run should definitely have a different shortcut than refresh, or else you may submit "drop database foobar" where you meant "refresh". ;-)

@javisperez

Or maybe add a preference Panel with custom shortcuts, so we can add our own Run shortcut which by default can be Cmd-R.

@elliotfiske

@javisperez OS X lets you define custom shortcuts for any application as described over here

@javisperez

@elliotfiske Yeap, i just can't find the menu entry for Run Sequence, so don't know what to put in the Menu Title option, leaving it blank didn't work for me, ideas?

@abhibeckert
Collaborator

@javisperez the menu items are here:

image

image

image

@javisperez

Thx @abhibeckert i've tried with that before, but seems like i tried with Run Current > Run Current Query and didnt work. Now i've tried with Run Selected Text and worked like a charm, thank you! :)

@abhibeckert
Collaborator

@javisperez I think there might be some glitches editing shortcuts for popup menus.

Try it out for a few days and we'll find workarounds for any bugs you find.

@michaelkanis michaelkanis reopened this Jan 19, 2017
@michaelkanis

Sorry, I unintentionally clicked on "Close" when I meant to comment. Personally, I would consider this a bug, because it might put user data at risk. To my knowledge basically all Mac applications use Cmd-R for refresh and Sequel Pro does too. Running database queries is a potentially "dangerous" operation, because they may not be easily revertable. Using the same shortcut as a harmless refresh might be "surprising" to users.

@michaelkanis

By the way, here is what MySQL Workbench does:
bildschirmfoto 2017-01-19 um 08 05 55

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