Skip to content

Commit

Permalink
Try sliding toolbar for overflow on mobile
Browse files Browse the repository at this point in the history
This PR may fix #2287, at least partially, or in a transitional way. It's a two-line code change, and all it does is ensure that on mobile, overflow can be scrolled.

This is not a good solution for desktop, of course, where the scrollbar when visible (as it is on nearly all computers) would not only cover content and be ugly, but scrolling horizontally is generally not a good experience. However on mobile, the scrollbar fades in briefly when you load the page, and you can just drag with your finger.

Let's discuss whether this is a good solution.
  • Loading branch information
jasmussen committed Aug 17, 2017
1 parent 038f49a commit 1c8d2ec
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions editor/modes/visual-editor/style.scss
Original file line number Diff line number Diff line change
Expand Up @@ -322,6 +322,7 @@
background: $white;
border-top: 1px solid $light-gray-500;
border-bottom: 1px solid $light-gray-500;
overflow: auto; // allow horizontal scrolling on mobile

> div,
> ul {
Expand All @@ -331,6 +332,7 @@

@include break-small() {
width: auto;
overflow: hidden;
}
}

Expand Down

1 comment on commit 1c8d2ec

@BoardJames
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:-(
This broke the drop-down menu.

Please sign in to comment.