Skip to content

Commit

Permalink
fix RDM-1 (RDM-35): Bring back broken content-height limit for huge s…
Browse files Browse the repository at this point in the history
…creens ( was lost by moving from css to scripted height )
  • Loading branch information
vue-dude committed Oct 12, 2020
1 parent 6367e3b commit 45a7790
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions src/App.vue
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,11 @@ export default {
updateDevice() {
new DeviceHandler(this.$store).updateDevice()
},
updateAppHeight(height) {
updateAppHeight() {
let height = this.$store.state.innerHeight
if (height > 1250 && !this.$store.state.isMobile) {
height = 1250
}
const yOffset = this.$store.state.isMobile ? 100 : 150
const dy = this.$store.state.isMobile ? 50 : 30
$('.default-view .content-container .content .layers .scroll-area').height(height - yOffset - dy)
Expand All @@ -78,12 +82,14 @@ export default {
watch: {
'$store.state.innerWidth'() {
this.updateAppWidth()
this.updateAppHeight()
},
'$store.state.innerHeight'(height) {
this.updateAppHeight(height)
this.updateAppHeight()
},
'$store.state.isMobile'() {
this.updateAppWidth()
this.updateAppHeight()
}
}
}
Expand Down

0 comments on commit 45a7790

Please sign in to comment.