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;
       }
     });