Skip to content

Conversation

adolgachev
Copy link
Contributor

@adolgachev adolgachev commented Sep 10, 2025

Fixes b/361783174 by addressing the incorrect placement of tabpanel roles inside the tab list.

For horizontal, can move the tablist down to the div which only wraps the actual tabs.

For vertical, it is triickier as the tab content is placed inline with the tab headers. For that, had to change to no longer use the tab roles.

@adolgachev adolgachev added the Accessibility This issue is related to accessibility (a11y) label Sep 10, 2025
@adolgachev adolgachev added target: minor This PR is targeted for the next minor release dev-app preview When applied, previews of the dev-app are deployed to Firebase action: review The PR is still awaiting reviews from at least one requested reviewer action: global presubmit The PR is in need of a google3 global presubmit requires: TGP This PR requires a passing TGP before merging is allowed labels Sep 10, 2025
Copy link

github-actions bot commented Sep 10, 2025

Deployed dev-app for e319459 to: https://ng-dev-previews-comp--pr-angular-components-31844-dev-p3vqpt53.web.app

Note: As new commits are pushed to this pull request, this link is updated after the preview is rebuilt.

@adolgachev adolgachev marked this pull request as ready for review September 11, 2025 20:35
@adolgachev adolgachev requested a review from a team as a code owner September 11, 2025 20:35
@adolgachev adolgachev requested review from crisbeto and ok7sai and removed request for a team September 11, 2025 20:35
@adolgachev adolgachev removed the action: global presubmit The PR is in need of a google3 global presubmit label Sep 11, 2025
@adolgachev adolgachev requested review from wagnermaciel and removed request for crisbeto September 15, 2025 23:45
@adolgachev adolgachev removed the request for review from wagnermaciel September 16, 2025 19:39
@adolgachev adolgachev added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Sep 16, 2025
@adolgachev adolgachev merged commit ff10f04 into angular:main Sep 17, 2025
25 checks passed
@adolgachev adolgachev deleted the stepper-tweak branch September 17, 2025 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accessibility This issue is related to accessibility (a11y) action: merge The PR is ready for merge by the caretaker area: material/stepper dev-app preview When applied, previews of the dev-app are deployed to Firebase requires: TGP This PR requires a passing TGP before merging is allowed target: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants