fixed #143 by trying to give focus to first enabled input field for m…

…odal dialogs when they are opened
tmplat-extension · Apr 22, 2013 · 1b7a4b1 · 1b7a4b1
1 parent 26162f9
commit 1b7a4b1
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/chrome/bin/lib/options.js b/chrome/bin/lib/options.js
@@ -1762,6 +1762,9 @@
         return analytics.track('Footer', 'Clicked', 'Donate');
       });
       load();
+      $('.modal').on('shown', function() {
+        return $(this).find(':input:enabled:first').focus();
+      });
       $('#template_shortcut_modifier').html(ext.isThisPlatform('mac') ? ext.SHORTCUT_MAC_MODIFIERS : ext.SHORTCUT_MODIFIERS);
       $('[popover]').each(function() {
         var $this, placement, trigger;

diff --git a/chrome/src/lib/options.coffee b/chrome/src/lib/options.coffee
@@ -1435,6 +1435,10 @@ options = window.options = new class Options extends utils.Class
     $('#donation').submit -> analytics.track 'Footer', 'Clicked', 'Donate'
     # Load the current option values.
     load()
+    # Ensure first enabled input field in modals get focus when opened.
+    $('.modal').on 'shown', ->
+      $(this).find(':input:enabled:first').focus()
+    # Apply OS-specific keyboard shortcut modifiers.
     $('#template_shortcut_modifier').html if ext.isThisPlatform 'mac'
       ext.SHORTCUT_MAC_MODIFIERS
     else