milas/jellyfish-web
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Use classes instead

Browse source
This commit is contained in:
ferferga 2020-05-17 00:11:37 +02:00
parent 26267e3827
commit 029c7382db
3 changed files with 7 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

4
src/assets/css/site.css
View file

@ -120,3 +120,7 @@ div[data-role=page] {
.headroom--unpinned {
transform: translateY(-100%);
}
.force-scroll {
overflow-y: scroll;
}

2
src/components/htmlvideoplayer/plugin.js
View file

@ -110,7 +110,7 @@ define(['browser', 'require', 'events', 'apphost', 'loading', 'dom', 'playbackMa
let 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.getElementsByTagName('body')[0].style = null;
document.body.classList.remove('force-scroll');
}
function zoomIn(elem) {

4
src/components/skinManager.js
View file

@ -138,7 +138,7 @@ define(['apphost', 'userSettings', 'browser', 'events', 'pluginManager', 'backdr
function onViewBeforeShow(e) {
if (e.detail && e.detail.type === 'video-osd') {
// This removes the space that the scrollbar takes while playing a video
document.getElementsByTagName('body')[0].style = null;
document.body.classList.remove('force-scroll');
return;
}
@ -159,7 +159,7 @@ define(['apphost', 'userSettings', 'browser', 'events', 'pluginManager', 'backdr
}
// 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.
document.getElementsByTagName('body')[0].style = 'overflow-y: scroll';
document.body.classList.add('force-scroll');
}
document.addEventListener('viewshow', onViewBeforeShow);