Skip to content

Commit

Permalink
Merge pull request #1679 from Sefaria/bug/sc-21134/mobile-web-display…
Browse files Browse the repository at this point in the history
…-jumbled-because-of-banner

Banners don't jumble up the mobile navigation menu anymore!
  • Loading branch information
edamboritz authored Oct 30, 2023
2 parents a4bd282 + 8130803 commit 0ecd68e
Showing 1 changed file with 13 additions and 1 deletion.
14 changes: 13 additions & 1 deletion static/css/s2.css
Original file line number Diff line number Diff line change
Expand Up @@ -607,7 +607,7 @@ input.noselect {
}
.mobileNavMenu {
position: fixed;
height: calc(100% - 60px);
height: calc(100vh - 60px);
box-sizing: border-box;
top: 60px;
width: 100%;
Expand All @@ -616,6 +616,18 @@ input.noselect {
z-index: 1000;
overflow-y: scroll;
}
div:has(#bannerMessage) + .readerApp.singlePanel .mobileNavMenu {
position: fixed; /*This takes the 60px of the header plus 120px of the banner into account */
height: calc(100vh - 180px);
top: 180px;
}
@supports not selector(:has(a, b)) {
/* Fallback for when :has() is unsupported */
.mobileNavMenu {
position: absolute;
}
}

.mobileNavMenu.closed {
display: none;
}
Expand Down

0 comments on commit 0ecd68e

Please sign in to comment.