Skip to content

Commit

Permalink
Don' regard 1024px width viewport as mobile
Browse files Browse the repository at this point in the history
* In IsMobileState.js we treat viewports smaller 1024px as mobile.
* In NcAppContent.vue we treat viewports starting with 1024px as mobile.

This results in weird behaviour with screens that have exactly 1024px
width. E.g., on three-pane apps, NcAppContentList is hidden but the
NcAppDetailsToggle button is not displayed yet.

This commit changes NcAppContent.vue to treat viewports smaller 1024px
as mobile, to be in sync with IsMobileState.js.

Fixes: #3758

Signed-off-by: Jonas <jonas@freesources.org>
  • Loading branch information
mejo- committed Feb 16, 2023
1 parent 61409e1 commit 0b5e4d2
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/components/NcAppContent/NcAppContent.vue
Original file line number Diff line number Diff line change
Expand Up @@ -381,15 +381,15 @@ export default {
position: sticky;
top: var(--header-height);

@media only screen and (max-width: $breakpoint-mobile) {
@media only screen and (width < $breakpoint-mobile) {
display: none;
}
}

&-details {
overflow-y: auto;

@media only screen and (max-width: $breakpoint-mobile) {
@media only screen and (width < $breakpoint-mobile) {
min-width: 100%;
}
}
Expand Down

0 comments on commit 0b5e4d2

Please sign in to comment.