2011-02-17 Ojan Vafai <ojan@chromium.org>
authorojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 21 Feb 2011 02:38:53 +0000 (02:38 +0000)
committerojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 21 Feb 2011 02:38:53 +0000 (02:38 +0000)
        Reviewed by Adam Barth.

        fix escape handling in code review tool
        https://bugs.webkit.org/show_bug.cgi?id=54722

        This broke with http://trac.webkit.org/changeset/78948.
        Apparently jquery events get a wrapped object that
        does not expose keyIdentifier.

        * code-review.js:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@79165 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Websites/bugs.webkit.org/ChangeLog
Websites/bugs.webkit.org/code-review.js

index a2c5235..62936c5 100644 (file)
@@ -2,6 +2,19 @@
 
         Reviewed by Adam Barth.
 
+        fix escape handling in code review tool
+        https://bugs.webkit.org/show_bug.cgi?id=54722
+
+        This broke with http://trac.webkit.org/changeset/78948.
+        Apparently jquery events get a wrapped object that
+        does not expose keyIdentifier.
+
+        * code-review.js:
+
+2011-02-17  Ojan Vafai  <ojan@chromium.org>
+
+        Reviewed by Adam Barth.
+
         create comments when hitting enter with a focused diff
         https://bugs.webkit.org/show_bug.cgi?id=54719
 
index bbfeba5..1065188 100644 (file)
@@ -1387,8 +1387,8 @@ var CODE_REVIEW_UNITTEST;
   }
 
   $('textarea').live('keydown', function(e) {
-    var unicode_escape = 'U+001B';
-    if (e.keyIdentifier == unicode_escape)
+    var escape_key_code = 27;
+    if (e.keyCode == escape_key_code)
       handleEscapeKeyDownInTextarea(this);
   });