Merge pull request #12805 from mansimarkaur/var_for_modal_in_dlg

zaggino · web-flow · commit 9f501637d9ba · 2016-09-28T16:29:48.000+10:00
Caching $(".modal-body", $dlg) in to its own variable
diff --git a/src/extensibility/ExtensionManagerDialog.js b/src/extensibility/ExtensionManagerDialog.js
@@ -309,6 +309,7 @@ define(function (require, exports, module) {
             views   = [],
             $search,
             $searchClear,
+            $modalDlg,
             context = { Strings: Strings, showRegistry: !!brackets.config.extension_registry },
             models  = [];
 
@@ -335,6 +336,7 @@ define(function (require, exports, module) {
             $search.val("");
             views.forEach(function (view, index) {
                 view.filter("");
+                $modalDlg.scrollTop(0);
             });
 
             if (!updateSearchDisabled()) {
@@ -360,14 +362,15 @@ define(function (require, exports, module) {
         $dlg = dialog.getElement();
         $search = $(".search", $dlg);
         $searchClear = $(".search-clear", $dlg);
+        $modalDlg = $(".modal-body", $dlg);
 
         function setActiveTab($tab) {
             if (models[_activeTabIndex]) {
-                models[_activeTabIndex].scrollPos = $(".modal-body", $dlg).scrollTop();
+                models[_activeTabIndex].scrollPos = $modalDlg.scrollTop();
             }
             $tab.tab("show");
             if (models[_activeTabIndex]) {
-                $(".modal-body", $dlg).scrollTop(models[_activeTabIndex].scrollPos || 0);
+                $modalDlg.scrollTop(models[_activeTabIndex].scrollPos || 0);
                 clearSearch();
             }
         }
@@ -434,7 +437,7 @@ define(function (require, exports, module) {
             $(".spinner", $dlg).remove();
 
             views.forEach(function (view) {
-                view.$el.appendTo($(".modal-body", $dlg));
+                view.$el.appendTo($modalDlg);
             });
 
             // Update search UI before new tab is shown
@@ -454,6 +457,7 @@ define(function (require, exports, module) {
                 var query = $(this).val();
                 views.forEach(function (view) {
                     view.filter(query);
+                    $modalDlg.scrollTop(0);
                 });
             }).on("click", ".search-clear", clearSearch);
 
