Merge pull request #420 from conlanpatrek/visibility-hack-fix

Fix sp-tab-page scrolling bugs.
AbdullahAlfaraj · Nov 25, 2023 · 8d84f17 · 8d84f17
2 parents 75814b0 + 363d22d
commit 8d84f17
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 25 deletions.
diff --git a/index.html b/index.html
@@ -340,30 +340,11 @@
  }
 
  .sp-tab-page {
- visibility: hidden;
- /* display: none; */
- flex: 1 1 auto;
- /* overflow: scroll; */
- overflow-y: scroll;
- padding: 12px 0;
+ display: none;
+ padding: 7px;
  flex-direction: column;
- }
-
- .sp-tab-page.visible-hack {
- display: flex;
- /* overflow: hidden; */
- /* overflow-y: scroll; */
- /* position: fixed; */
- top: 25px;
- visibility: visible;
  max-width: 100%;
- width: 100%;
- /* margin: 0px; */
- position: absolute;
- width: 100%;
- left: 0;
- padding-left: 10px;
- padding-right: 10px;
+ position: relative;
  }
 
  .sp-tab-page.visible {
@@ -1124,7 +1105,7 @@
  </div>
 
  <div
- class="sp-tab-page visible-hack"
+ class="sp-tab-page visible"
  id="sp-stable-diffusion-ui-tab-page"
  >
  <div id="sdTabContainer"></div>

diff --git a/index.js b/index.js
@@ -313,9 +313,9 @@ Array.from(document.querySelectorAll('.sp-tab')).forEach((theTab) => {
  .getAttribute('id')
  .startsWith(theTab.getAttribute('id'))
  ) {
- tabPage.classList.add('visible-hack')
+ tabPage.classList.add('visible')
  } else {
- tabPage.classList.remove('visible-hack')
+ tabPage.classList.remove('visible')
  }
  }
  )