Skip to content

Commit

Permalink
Don't regard 1024px width viewport as mobile
Browse files Browse the repository at this point in the history
Currently we treat 1024px as mobile in some places (e.g. `isMobile` in
nextcloud-vue) and as not mobile yet in others (e.g. in mobile.css).

This commit changes `mobile.scss` to treat viewports smaller than 1024px
as mobile.

This PR accompanies nextcloud-libraries/nextcloud-vue#3768

Both are required to fix
nextcloud-libraries/nextcloud-vue#3758.

Signed-off-by: Jonas <jonas@freesources.org>
  • Loading branch information
mejo- authored and summersab committed Jun 26, 2023
1 parent 1372ec9 commit 302bb62
Show file tree
Hide file tree
Showing 5 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion core/css/mobile.css

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion core/css/mobile.css.map

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion core/css/mobile.scss
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
@use 'variables';

@media only screen and (max-width: variables.$breakpoint-mobile) {
@media only screen and (width < variables.$breakpoint-mobile) {

/* position share dropdown */
#dropdown {
Expand Down
2 changes: 1 addition & 1 deletion core/css/server.css

Large diffs are not rendered by default.

Loading

0 comments on commit 302bb62

Please sign in to comment.