abhisheknair / keynote-nf

Automatically exported from code.google.com/p/keynote-nf
0 stars 0 forks source link

Esc should close Find box but not also minimize the app #335

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?

1. Open the Find box (Ctrl+F).
2. Press Esc to dismiss the box.
3. The Find box closes, but the entire app is also minimized (if "On Esc key > 
Minimize main window" is set in Config. Options).

What is the expected output? What do you see instead?

This isn't consistent with the original KeyNote (or with most Windows apps that 
use Esc). It's great to be able to press Esc to minimize the app, but it should 
also be possible to press it just to close the Find box. 

What version are you using? On what operating system?

v1.7.8.1 on Windows 7 Home Premium (64-bit)

Please provide any additional information below.

Thanks for fixing this... I'd normally post this kind of thing on a forum, but 
I don't think you have one for this project.

(If you're not able to fix this, I'm afraid I'll have to return to normal 
KeyNote---I can't "retrain" myself after so many years.  :?|

Original issue reported on code.google.com by anderwri...@gmail.com on 22 Jun 2010 at 9:01

GoogleCodeExporter commented 9 years ago
The problem seems to be simply that the Find window hasn't the focus when you 
press Esc. This occurs after the message "Pattern not found ...". To be able to 
show messages in Unicode I had to use another method and with it the focus 
didn't return automatically to the Find Window.
I have corrected it in revision r145

Original comment by dpra...@gmail.com on 19 Aug 2011 at 8:28

GoogleCodeExporter commented 9 years ago
This will also fix my issue #327, so you may close it.

Original comment by herbert.oppmann@siemens.com on 21 Aug 2011 at 9:06

GoogleCodeExporter commented 9 years ago
Revision r145 only corrected the issue in relation to Find window, not with 
Replace window. Now all is corrected with r146.
As issue #327 is more general I will fusionate this issue with that.

Original comment by dpra...@gmail.com on 21 Aug 2011 at 10:48