Return permanent scrollbar (killed by refactoring)

This commit is contained in:
Dmitry Lyzo 2020-09-03 01:04:21 +03:00
parent 388b33a6b4
commit d644193441
3 changed files with 7 additions and 7 deletions

View file

@ -44,6 +44,11 @@ body {
overflow-x: hidden;
background-color: transparent !important;
-webkit-font-smoothing: antialiased;
/* This keeps the scrollbar always present in all pages, so we avoid clipping while switching between pages
that need the scrollbar and pages that don't.
*/
overflow-y: scroll;
}
.mainAnimatedPage {
@ -122,10 +127,6 @@ div[data-role=page] {
padding-bottom: 4em;
}
.force-scroll {
overflow-y: scroll;
}
.hide-scroll {
overflow-y: hidden;
}

View file

@ -492,7 +492,6 @@ export default function (options) {
document.removeEventListener((window.PointerEvent ? 'pointermove' : 'mousemove'), onPointerMove);
// Shows page scrollbar
document.body.classList.remove('hide-scroll');
document.body.classList.add('force-scroll');
}
/**
@ -657,7 +656,6 @@ export default function (options) {
self.show = function () {
createElements(options);
// Hides page scrollbar
document.body.classList.remove('force-scroll');
document.body.classList.add('hide-scroll');
};

View file

@ -107,7 +107,7 @@ function tryRemoveElement(elem) {
const animatedPage = document.querySelector('.page:not(.hide)');
animatedPage.classList.add('hide');
// At this point, we must hide the scrollbar placeholder, so it's not being displayed while the item is being loaded
document.body.classList.remove('force-scroll');
document.body.classList.add('hide-scroll');
}
function zoomIn(elem) {
@ -679,6 +679,7 @@ function tryRemoveElement(elem) {
destroyFlvPlayer(this);
appRouter.setTransparency('none');
document.body.classList.remove('hide-scroll');
const videoElement = this.#mediaElement;