Skip to content

feat(ui5-product-switch): Implement accessibility specification #2865

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

Merged
merged 3 commits into from
May 10, 2021

Conversation

dobrinyonkov
Copy link
Contributor

@dobrinyonkov dobrinyonkov commented Feb 24, 2021

FIXES: #2707

Additionally:

UP and DOWN arrow keys navigation is updated, and limited to the product switch borders.

@dobrinyonkov dobrinyonkov marked this pull request as draft February 24, 2021 14:18
@ilhan007 ilhan007 added the ACC label Feb 24, 2021
ilhan007
ilhan007 previously approved these changes Feb 25, 2021
Copy link
Member

@ilhan007 ilhan007 left a comment

Choose a reason for hiding this comment

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

It looks good to me, but let's wait for @elenastoyanovaa to review from a11y perspective.
Just a reminder to update the a11y page afterwards:
https://wiki.wdf.sap.corp/wiki/pages/viewpage.action?pageId=2382053438

@dobrinyonkov
Copy link
Contributor Author

I'm keeping this as a draft as it needs some further discussion in regards of the grid like navigation (arrows - up, down, left, right). The role menu -> menuitem and aria-setsize and posinset attributes may not be the most appropriate ones. I will discuss this with accessibility experts on the next round.

@dobrinyonkov dobrinyonkov force-pushed the product-switch-acc branch 2 times, most recently from e855efe to 57e3ac9 Compare April 13, 2021 13:04
@dobrinyonkov dobrinyonkov marked this pull request as ready for review April 14, 2021 05:56
@nnaydenow
Copy link
Contributor

Hi @ilhan007,

For me the change looks okey. Can you approve it?

@ilhan007 ilhan007 merged commit 8c0225c into SAP:master May 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ui5-product-switch: Screen Reader Support Implementation
3 participants