Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: Hide thumbnails in mobile portrait #967

Merged
merged 7 commits into from
Mar 28, 2019

Conversation

ConradJChan
Copy link
Contributor

@ConradJChan ConradJChan commented Mar 27, 2019

When in a CSS breakpoint of less than 600px, hide the toggle thumbnails button & the sidebar

Todo:

  • e2e tests

@boxcla
Copy link

boxcla commented Mar 27, 2019

Verified that @ConradJChan has signed the CLA. Thanks for the pull request!

jstoffan
jstoffan previously approved these changes Mar 27, 2019
test/integration/document/Thumbnails.e2e.test.js Outdated Show resolved Hide resolved
@ConradJChan ConradJChan dismissed stale reviews from jstoffan and JustinHoldstock via 0fa7c11 March 27, 2019 21:35
@ConradJChan ConradJChan merged commit 8d5cbb8 into box:master Mar 28, 2019
@ConradJChan ConradJChan deleted the no-mobile-thumbnails branch March 28, 2019 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants