Changeset 22946

Timestamp:

11/30/2012 08:37:17 AM (12 years ago)

Author:

nacin

Message:

Pressing escape now closes the media modal. props koopersmith, see #22502.

File:

• trunk/wp-includes/js/media-views.js (modified) (3 diffs)

trunk/wp-includes/js/media-views.js

@@ -1617 +1617 @@
         template: media.template('media-modal'),
 
+        attributes: {
+            tabindex: 0
+        },
+
         events: {
-            'click .media-modal-backdrop, .media-modal-close' : 'closeHandler'
+            'click .media-modal-backdrop, .media-modal-close': 'closeHandler',
+            'keydown': 'keydown'
         },
 
@@ -1659 +1664 @@
 
         open: function() {
-            this.$el.show();
+            this.$el.show().focus();
             return this.propagate('open');
         },
@@ -1692 +1697 @@
 
             return this;
+        },
+
+        keydown: function( event ) {
+            // Close the modal when escape is pressed.
+            if ( 27 === event.which ) {
+                event.preventDefault();
+                this.close();
+                return;
+            }
         }
     });