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

[Accessibility] Info Page: Zooming disabled in mobile view #830

Closed
nwanduka opened this issue Mar 20, 2024 · 1 comment · Fixed by #900
Closed

[Accessibility] Info Page: Zooming disabled in mobile view #830

nwanduka opened this issue Mar 20, 2024 · 1 comment · Fixed by #900
Labels

Comments

@nwanduka
Copy link
Contributor

nwanduka commented Mar 20, 2024

Describe the bug
Zooming is disabled on mobile view of the Info Page.
User Impact: Critical
Tool used: Axe

To Reproduce
Steps to reproduce the behavior:

  1. Go to http://localhost:4010/apps/Info.html
  2. Right-click on the page and click "inspect" to open the developer tools window
  3. Zoom in on the Info page while it's in desktop view. The page zooms in and out.
  4. Then switch to mobile view in the inspect panel and try to zoom. You'd notice that zoom doesn't work in mobile view

Expected behavior
The info page contains a table that fills a desktop view. In mobile view, only a small part of the table is shown. So to view the rest of the table, users need to be able to scale the table up or down by zooming.

Screenshots
image

Zoom accessibility issue

Desktop (please complete the following information):

  • OS: [Windows
  • Browser: Microsoft Edge
  • Version [e.g. 22]

Why is this important?
The user-scalable="no" parameter inside the content attribute of element disables zooming on a page. The maximum-scale parameter limits the amount the user can zoom. This is problematic for people with low vision who rely on screen magnifiers to properly see the contents of a web page.

How to Fix the Problem
Remove the user-scalable="no" parameter from the content attribute of the element in order to allow zooming and ensure the maximum-scale parameter is not less than 2.

@Ayesh-07
Copy link
Contributor

@nwanduka
Fixed disabled in mobile #830
I would appreciate a review of my work to ensure it meets the project's standards.

birm added a commit that referenced this issue Mar 26, 2024
Fixed disabled in mobile view #830
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants