From a82e692bdd39feced285b6bd25c3ccb39b63434b Mon Sep 17 00:00:00 2001
From: John McLear <john@mclear.co.uk>
Date: Tue, 31 Mar 2015 13:21:41 +0100
Subject: [PATCH] dont use nasty settimeouts use a callback instead for focus

---
 src/static/js/pad_editbar.js | 26 ++++++++++++--------------
 1 file changed, 12 insertions(+), 14 deletions(-)

diff --git a/src/static/js/pad_editbar.js b/src/static/js/pad_editbar.js
index aa9f183b8..8df6e693e 100644
--- a/src/static/js/pad_editbar.js
+++ b/src/static/js/pad_editbar.js
@@ -352,32 +352,30 @@ var padeditbar = (function()
     toolbar.registerDropdownCommand("embed");
 
     toolbar.registerCommand("settings", function () {
-      setTimeout(function(){
+      toolbar.toggleDropDown("settings", function(){
         $('#options-stickychat').focus();
-      },100);
-      toolbar.toggleDropDown("settings");
+      });
     });
 
     toolbar.registerCommand("import_export", function () {
-      setTimeout(function(){
-        $('#importfileinput').focus();
-      },100);
-      toolbar.toggleDropDown("import_export");
+      toolbar.toggleDropDown("import_export", function(){
+        setTimeout(function(){
+          $('#importfileinput').focus();
+        }, 100);
+      });
     });
 
     toolbar.registerCommand("showusers", function () {
-      setTimeout(function(){
-        $('#myusernameedit').focus(); // TODO: Not working
-      },100);
-      toolbar.toggleDropDown("users");
+      toolbar.toggleDropDown("users", function(){
+        $('#myusernameedit').focus();
+      });
     });
 
     toolbar.registerCommand("embed", function () {
       toolbar.setEmbedLinks();
-      setTimeout(function(){
+      toolbar.toggleDropDown("embed", function(){
         $('#linkinput').focus().select();
-      }, 100);
-      toolbar.toggleDropDown("embed");
+      });
     });
 
     toolbar.registerCommand("savedRevision", function () {