brittneybrinsfield / sequel-pro

Automatically exported from code.google.com/p/sequel-pro
Other
0 stars 0 forks source link

[REQ] Run All useless and dangerous... #1610

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
There are related requests (1584 and 1604). I just wanted to throw in another 
suggestion--remove or optionally disable the "Run All" button. 

I know you rearranged the buttons to be more in line with the Apple way of 
doing things. So maybe I just need to get over it. But the "Run All" button is 
useless. CMD-A followed by "Run Selection" is equivalent and much, much safer.

My current work around to avoid hitting "Run All" accidentally is to have a SQL 
statement at the top that fails. This allows me to press "Stop" in the 
sequel-pro dialog box. I must accidentally hit "Run All" about 3 times more 
often than I get it correct. Of course, after I press stop, the program focuses 
on the broken SQL and I have to move my cursor to what I really wanted to 
run--also kind of a nuisance. 

You knew you'd get bugs/requests on this issue, yes? :)

thanks
jason.

Original issue reported on code.google.com by underlig...@gmail.com on 11 Feb 2013 at 5:06

GoogleCodeExporter commented 9 years ago
I agree.  I never cared for Run and Run All sitting right there.   I blindly 
mash one when my mind is on other stuff.  I'd prefer a single button (that 
changes to "Run All" when opt is pressed, or whatever the outcome of that 
argument is)

Original comment by mainstre...@gmail.com on 12 Feb 2013 at 12:39

GoogleCodeExporter commented 9 years ago
I agree with mainstreetmark. I already raised that point when discussing this 
issue with Rowan.

Original comment by schlabbe...@gmail.com on 16 Feb 2013 at 8:22

GoogleCodeExporter commented 9 years ago
This just bit me after I upgraded, as well. Just get rid of the Run All button, 
or hide it behind option, or have a confirmation dialog. But honestly, select 
all + run selection is good enough.

The fact that the keyboard shortcuts were also swapped is downright dastardly.

Original comment by qsm...@gmail.com on 1 Mar 2013 at 1:03

GoogleCodeExporter commented 9 years ago
I agree... this button just screwed me over...

Original comment by jfi...@hyperxmedia.com on 13 Mar 2013 at 10:12

GoogleCodeExporter commented 9 years ago
Version 1.0.2 has implemented a single button.

Original comment by schlabbe...@gmail.com on 22 Jun 2013 at 12:21