-
Notifications
You must be signed in to change notification settings - Fork 123
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
Poor Mobile Experience #18
Comments
The code treats any screen with a wide 897px or smaller as mobile (iPhone Max is 898px wide). When this happens, the code switches from the chart to a static image (lines 674-688). If a better way is possible, I'm open to changing this behavior. |
I've gotten more feedback that the mobile version is not very good. 60% of traffic is on mobile so this is now a high priority issue. |
I updated the image to be the April 2022 version which will make it less bad for awhile |
Making the image a clickable link to itself would help. Right now, on Chrome under Android, there's no way to zoom in on the page - it treats it as fixed width and a two-finger drag motion does nothing. I had to long-press the image and go to "open image in new tab" to read it. |
I adjusted the website template to allow the real chart to horizontally scroll in the mobile version. Remaining issues:
|
This chart is too large to be viewed properly on mobile devices. A smaller option should be made available to those who would like to view the chart without zooming.
If removing certifications to make a smaller chart is selected, proper selection criteria should be developed to decide which certifications to remove.
The text was updated successfully, but these errors were encountered: