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.

Sign up for GitHub

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

Jump to bottom

sidebar width property screws up mobile rendering #177

Closed
lukewp opened this issue Jan 19, 2017 · 1 comment · Fixed by #204
Closed

sidebar width property screws up mobile rendering #177

lukewp opened this issue Jan 19, 2017 · 1 comment · Fixed by #204
Assignees
@bborgesr

Comments

@lukewp
Copy link

lukewp commented Jan 19, 2017 •
edited by bborgesr
Loading

If you set width for a sidebar, when you call the page from mobile, part of the sidebar may never go away, obscuring part of the content like this:
image

One solution that gets the dashboard rendering properly on mobile is to remove the sidebar width property from the code.
@wch wch added the targeted label Jan 26, 2017
@wch
Copy link
Contributor

wch commented Jan 26, 2017

Probably related to #127.

bborgesr added a commit that referenced this issue Apr 21, 2017
@bborgesr
fixes sidebar complete showing/hiding on mobile when width paramete…
42328e3 
…r is passed to dashboardSidebar() (fixes #177 and fixes #127)
@bborgesr bborgesr mentioned this issue Apr 21, 2017
Merged
@bborgesr bborgesr self-assigned this Apr 21, 2017
@wch wch closed this as completed in #204 Apr 21, 2017
wch pushed a commit that referenced this issue Apr 21, 2017
@bborgesr @wch
fixes sidebar complete showing/hiding on mobile when width paramete…
6fdc564 
…r is passed to dashboardSidebar() (fixes #177 and fixes #127) (#204)
@wch wch removed the in progress label Apr 21, 2017
bborgesr added a commit that referenced this issue Apr 21, 2017
@bborgesr
fixes sidebar complete showing/hiding on mobile when width paramete…
a580033 
…r is passed to dashboardSidebar() (fixes #177 and fixes #127) (#204)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bborgesr bborgesr

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix sidebar complete showing/hiding on mobile rstudio/shinydashboard
3 participants
@wch @lukewp @bborgesr