From 7cace5f160b7ce8d78a6d8386a64ab90a1c516d5 Mon Sep 17 00:00:00 2001
From: Yury Delendik <ydelendik@mozilla.com>
Date: Sat, 6 Oct 2012 08:58:12 -0500
Subject: [PATCH] Escape key closes findbar
---
web/viewer.js | 12 +++++++-----
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/web/viewer.js b/web/viewer.js
index 5c1eda023..b7d815392 100644
--- a/web/viewer.js
+++ b/web/viewer.js
@@ -508,13 +508,15 @@ var PDFFindBar = {
self.dispatchEvent('');
});
- // TODO: Add keybindings CMD-G etc. to go to prev/
- // next match when the findField is selected.
-
- this.findField.addEventListener('keydown', function(evt) {
+ this.bar.addEventListener('keydown', function(evt) {
switch (evt.keyCode) {
case 13: // Enter
- self.dispatchEvent('again', evt.shiftKey);
+ if (evt.target === self.findField) {
+ self.dispatchEvent('again', evt.shiftKey);
+ }
+ break;
+ case 27: // Escape
+ self.close();
break;
}
});