Add a hack to a hack to make it look much less hacky.

This commit is contained in:
shpuld 2017-11-12 20:24:20 +02:00
parent 5813541006
commit 07dc195234
3 changed files with 29 additions and 3 deletions

View file

@ -23,6 +23,18 @@ export default {
style () { return { 'background-image': `url(${this.background})` } },
sitename () { return this.$store.state.config.name }
},
created () {
// this is to detect user zooming mostly
window.addEventListener('resize', this.fixSidebarWidth)
},
mounted () {
// for some reason, at least in dev mode, dom is not ready enough at this point
// in theory calling the function directly here should be enough, but it's not
setTimeout( () => { this.fixSidebarWidth() }, 500)
},
destroyed () {
window.removeEventListener('resize', this.fixSidebarWidth)
},
methods: {
activatePanel (panelName) {
this.mobileActivePanel = panelName
@ -32,6 +44,17 @@ export default {
},
logout () {
this.$store.dispatch('logout')
},
fixSidebarWidth () {
// firefox
let barwidth = window.innerWidth - document.body.offsetWidth
if (document.body.offsetWidth <= 0) {
// chromium
barwidth = window.innerWidth - document.body.scrollWidth
}
// adjust the sidebar size to fit the scrollbar width to keep the gap consistently sized
document.getElementById("sidebar-container").style.width = `${345 + barwidth}px`
document.getElementById("sidebar-container").style.paddingRight = `${barwidth}px`
}
}
}

View file

@ -210,16 +210,19 @@ nav {
}
@media all and (min-width: 960px) {
body {
overflow-y: scroll;
}
.sidebar {
overflow: hidden;
max-height: 100vh;
width: 350px;
width: 345px;
position: fixed;
margin-top: -10px;
.sidebar-container {
height: 96vh;
width: 362px;
width: 365px;
padding-top: 10px;
padding-right: 20px;
overflow-x: hidden;

View file

@ -19,7 +19,7 @@
</div>
<div class="sidebar-flexer" :class="{ 'mobile-hidden': mobileActivePanel != 'sidebar'}">
<div class="sidebar" :class="{ 'mobile-hidden': mobileActivePanel != 'sidebar' }">
<div class="sidebar-container">
<div class="sidebar-container" id="sidebar-container">
<user-panel></user-panel>
<nav-panel></nav-panel>
<notifications v-if="currentUser"></notifications>