css: Fix last commit overflow should not apply to inner-editors (ethe…

…r#3914)

* css: Fix last commit overflow should not apply to inner-editors

* css: hide toolbar popup and nice-select when clicking on pad

src/static/css/pad/form.css

@@ -158,6 +158,9 @@ select, .nice-select {
 .nice-select .option:hover,.nice-select .option.focus,.nice-select .option.selected.focus {
   background-color: #f6f6f6;
 }
+.nice-select .option[data-value="dummy"] {
+  display: none;
+}
 .nice-select .option.selected {
   font-weight: bold;
 }

src/static/css/pad/layout.css

@@ -1,10 +1,12 @@
 html, body {
   width: 100%;
   height: 100%;
-  overflow: hidden;
   margin: 0;
   padding: 0;
 }
+html:not(.inner-editor), html:not(.inner-editor) body {
+  overflow: hidden;
+}
 body {
   display: flex;
   flex-direction: column;

src/static/js/pad_editbar.js

@@ -217,6 +217,9 @@ var padeditbar = (function()
         return;
       }
 
+      $('.nice-select').removeClass('open');
+      $('.toolbar-popup').removeClass("popup-show");
+
       // hide all modules and remove highlighting of all buttons
       if(moduleName == "none")
       {
@@ -235,13 +238,12 @@ var padeditbar = (function()
           var isAForceReconnectMessage = module.find('button#forcereconnect:visible').length > 0;
           if(isAForceReconnectMessage)
             continue;
-
-          if(module.css('display') != "none")
-          {
+          if (module.hasClass('popup-show')) {
             $("li[data-key=" + thisModuleName + "] > a").removeClass("selected");
             module.removeClass("popup-show");
           }
         }
+
         if(!returned && cb) return cb();
       }
       else