guberm / tortoisegit

Automatically exported from code.google.com/p/tortoisegit
0 stars 0 forks source link

Support keyboard shortcuts in yes/no prompts #2507

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Trigger any yes/no dialog, eg edit/split commit during a rebase
2. Press Y/N
3. Dialog does nothing.

What is the expected output? What do you see instead?
Most Windows dialogs would allow the user to press Y for the affirmative, or N 
for the negative response.

What version of TortoiseGit and msysgit are you using? On
what operating system?
1.8.14, 1.9.5, Windows 8.1 x64

Please provide any additional information below.
This would be incredibly useful for those of us who are familiar with keyboard 
shortcuts. Thanks!

Original issue reported on code.google.com by j...@joelsplace.sg on 14 May 2015 at 11:47

GoogleCodeExporter commented 9 years ago

Original comment by sstrickr...@googlemail.com on 16 May 2015 at 10:03

GoogleCodeExporter commented 9 years ago
Can you please point out one dialog where this does not work?

Original comment by sstrickr...@googlemail.com on 16 May 2015 at 10:48

GoogleCodeExporter commented 9 years ago
When rebasing:

"The current working tree is not clean. Do you want to stash the changes?" It 
says "Stash" and "Abort", but Y/N should still map to "Stash" and "Abort"

After editing a commit in the rebase dialog, and the commit is being edited, I 
check Edit/Split commit. I click on Amend, and split the commit in 2. "Add 
another commit?" It says "Yes" and "No", and Y/N has no effect

There probably are other dialogs, but these are those I can think of offhand.

Original comment by j...@joelsplace.sg on 16 May 2015 at 11:10

GoogleCodeExporter commented 9 years ago
The "Add another commit?" dialog needs to be updated, right.

But for the other dialogs where no Yes/No buttons are shown Y/N doesn't make 
sense. Here, however, you can use the memnorics (shown when pressing the ALT 
key).

Original comment by sstrickr...@googlemail.com on 17 May 2015 at 10:43

GoogleCodeExporter commented 9 years ago
This issue was closed by revision a1ed91ea9696.

Original comment by sstrickr...@googlemail.com on 17 May 2015 at 10:52