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

In 200% zoom, Keyboard focus is not properly visible on 'Access token' and 'Got Feedback' tab item button. #1374

Closed
ElinorW opened this issue Jan 25, 2022 · 2 comments
Closed

In 200% zoom, Keyboard focus is not properly visible on 'Access token' and 'Got Feedback' tab item button. #1374

ElinorW opened this issue Jan 25, 2022 · 2 comments
Assignees
@ElinorW
Labels
dependencies Pull requests that update a dependency file priority:p1 High priority but not blocking. Causes major but not critical loss of functionality SLA <=7days. type: accessibility A general accessibility issue
Milestone
Feb 04 - Mar 03

Comments

@ElinorW
Copy link
Contributor

ElinorW commented Jan 25, 2022

Test Environment:
OS Version: Windows 11 Build 22504.1010
Browser: Edge Dev Version 98.0.1093.6 (Official build) dev (64-bit)
URL: Graph Explorer - Microsoft Graph
Screen Reader: Narrator
Username: [email protected]/[email protected]

Pre-Requisite:
1.Setting Display Resolution to '1280*768
2.Select Zoom settings to '200%' for Anaheim dev browser.
3.Refresh the page.

Repro Steps:

  1. Open above URL in Edge browser.
  2. Login with valid credential.
  3. Navigate to the 'Access token' button using tab key.
  4. Observe if keyboard focus is properly visible.

Actual Result:
In 200% zoom, Keyboard focus is not properly visible on 'Access token' and 'Got Feedback' tab item button.

Expected Result:
In 200% zoom, Keyboard focus should be properly visible on 'Access token' and 'Got Feedback' tab item button.

User Impact:
Visual impairment users will face difficulty if any item on the application or keyboard focus on it is not visible on resizing the page to a 200% zoom level

MAS Reference:
Resize Text - Liquid (microsoft.com)
@ElinorW ElinorW added priority:p1 High priority but not blocking. Causes major but not critical loss of functionality SLA <=7days. type: accessibility A general accessibility issue labels Jan 25, 2022
@ghost ghost added the ToTriage label Jan 25, 2022
@ElinorW
Copy link
Contributor Author

ElinorW commented Jan 25, 2022

tracking this here microsoft/fluentui#20664

@Onokaev Onokaev self-assigned this Jan 27, 2022
@ghost ghost removed the ToTriage label Jan 27, 2022
@Onokaev Onokaev added this to the Jan 06 - Feb 03 milestone Jan 27, 2022
@Onokaev Onokaev mentioned this issue Jan 27, 2022
Merged
@adhiambovivian adhiambovivian added the dependencies Pull requests that update a dependency file label Feb 3, 2022
@Onokaev Onokaev assigned ElinorW and unassigned Onokaev Feb 14, 2022
@ElinorW
Copy link
Contributor Author

ElinorW commented Feb 21, 2022

No longer an issue as the Feedback button was moved in this PR #1455

@ElinorW ElinorW closed this as completed Feb 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ElinorW ElinorW

Labels
dependencies Pull requests that update a dependency file priority:p1 High priority but not blocking. Causes major but not critical loss of functionality SLA <=7days. type: accessibility A general accessibility issue
Projects
None yet
Milestone
Feb 04 - Mar 03
Development

No branches or pull requests
4 participants
@RabebOthmani @adhiambovivian @ElinorW @Onokaev