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

Make demo a bit more mobile friendly #44

Merged
merged 1 commit into from
Sep 10, 2024
Merged

Conversation

BGR360
Copy link
Contributor

@BGR360 BGR360 commented Sep 10, 2024

  1. Change ui.horizontal() to ui.horizontal_wrapped() for the PlotDemo selector so it doesn't go off screen.
  2. Wrap several of the demo's "settings" UI's in a ScrollArea::horizontals so they don't go off screen and make the plots go off screen as well.
  3. Wrap the plots in the LinkedAxes demo in a ScrollArea::horizontal so they don't go off screen.

Before and after screenshots

Lines

Before:
Screenshot 2024-09-09 at 7 15 24 PM

After:
Screenshot 2024-09-09 at 7 17 13 PM

Legend

Before:
Screenshot 2024-09-09 at 7 43 05 PM

After:
Screenshot 2024-09-09 at 7 22 37 PM

Charts

Before:
Screenshot 2024-09-09 at 7 43 12 PM

After:
Screenshot 2024-09-09 at 7 25 15 PM

Linked Axes

Before:
Screenshot 2024-09-09 at 7 43 29 PM

After:
Screenshot 2024-09-09 at 7 29 50 PM

Copy link
Contributor Author

@BGR360 BGR360 left a comment

Choose a reason for hiding this comment

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

Doing a self review as suggested in CONTRIBUTING.md. LGTM 😅

@emilk emilk added exclude from changelog This change will NOT be included in the changelog examples Examples and demo labels Sep 10, 2024
@emilk emilk changed the title [demo] Fix UI overflowing the screen on mobile. Make demo a bit more mobile friendly Sep 10, 2024
@emilk emilk merged commit 1e7fd71 into emilk:main Sep 10, 2024
8 of 9 checks passed
@emilk
Copy link
Owner

emilk commented Sep 10, 2024

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
examples Examples and demo exclude from changelog This change will NOT be included in the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants