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

fix(ui5-carousel): selected state is added to item #9184

Merged
merged 4 commits into from
Jun 14, 2024

Conversation

GerganaKremenska
Copy link
Member

@GerganaKremenska GerganaKremenska commented Jun 11, 2024

Fixes: #9027

@GerganaKremenska GerganaKremenska changed the title fix(ui5-carousel): selected state is announced by screen readers fix(ui5-carousel): selected state is added to item Jun 13, 2024
@GerganaKremenska GerganaKremenska merged commit ced0f35 into main Jun 14, 2024
10 checks passed
@GerganaKremenska GerganaKremenska deleted the carousel_aria_selected branch June 14, 2024 05:03
ilhan007 pushed a commit that referenced this pull request Jun 14, 2024
* fix(ui5-carousel): selected state is announced by screen readers

* chore(ui5-carousel): give correct expected values to tests

* fix(ui5-carousel): aria-hidden not needed
kgogov pushed a commit that referenced this pull request Jun 17, 2024
* fix(ui5-carousel): selected state is announced by screen readers

* chore(ui5-carousel): give correct expected values to tests

* fix(ui5-carousel): aria-hidden not needed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Carousel]: [a11y] aria-selected property not being set for current option
2 participants