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

Mobile usability in full screen mode #2887

Closed
Tracked by #3070
CAYdenberg opened this issue Nov 2, 2023 · 4 comments · Fixed by #3650
Closed
Tracked by #3070

Mobile usability in full screen mode #2887

CAYdenberg opened this issue Nov 2, 2023 · 4 comments · Fixed by #3650

Comments

@CAYdenberg
Copy link

Description

Full screen mode is a nice addition which has the potential to drastically improve the experience for mobile users, however it seems to have the opposite effect, crowding screen real estate with the header and footer and leaving little/no space for the data. Turning the phone to landscape actually makes things worse. This matters because many users are instinctively trained to turn to landscape when in full screen because this is how they watch videos.

Expected behaviour

Full screen mode should emphasize the actual data across a range of device capabilities, possibly with the header and footer accessible if needed. In general though this information isn't going to be critical at this point because the user is aware they can access it by exiting full screen mode.

Steps to reproduce

Steps to reproduce the behavior:

  1. Go to https://ourworldindata.org/grapher/co2-emissions-vs-gdp
  2. Click on "Enter full screen"
  3. Turn device to landscape mode
  4. Attempt to tap, scroll up and down, etc.

Screenshots

image_123986672

Environment

See https://www.whatismybrowser.com/w/54YSQKM

  • iPhone SE
  • OS: iOS 16.6.1
  • Browser safari 16.6

Yes, this browser is out of date. It also occurs in Brave. I'm pretty certain it's a viewport and not a browser issue per se.

Additional context

Loving the redesign, just a suggestion for improvement.

@danyx23
Copy link
Contributor

danyx23 commented Feb 14, 2024

Thank you for reporting this, @CAYdenberg! We agree, there are definitely some desperately needed improvements to be made here. We'll rethink grapher on mobile at some point in the near future and then will make sure to consider improving this area in particular.

For reference, Christian and I think that a reasonable area of investigation would be to drop the full screen button in scenarios where it doesn't make much sense and to introduce a minimum chart height that would make the chart scroll to some degree in cases like these. Just getting rid of the note and subtitle is probably not enough on it's own for some scenarios.

@CAYdenberg
Copy link
Author

If you're interested, I wrote up a small UX case study https://livingpixel.io/blog/improving-web-scatterplots

@danyx23
Copy link
Contributor

danyx23 commented Feb 14, 2024

Oh wow, that is a great critique! I'll share it with the team so we can take it into account when we work on this area. Thanks a lot!

@danyx23
Copy link
Contributor

danyx23 commented Mar 6, 2024

Thinking about the cost/benefit ratio of this feature maybe it is almost worth just hiding the fullscreen button on mobile in landscape? And/or if you are in fullscreen and flip the phone switch out of fullscreen mode. So basically if the grapher height is higher than the available viewport height we would drop out of fullscreen or not allow entering it in the first place.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants