-
Notifications
You must be signed in to change notification settings - Fork 6.7k
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(cdk/table): set default role of table to 'table' #22491
Conversation
2eb7279
to
19b2fd5
Compare
25537fe
to
ad1aaaf
Compare
Set cell roles to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Caretaker note: when we sync this into Google, we need to send out a communication that this default has changed |
* fix(cdk/table): set default role of table to 'table' * fixup! fix(cdk/table): set default role of table to 'table' (cherry picked from commit 71d75f4)
This reverts commit 71d75f4.
Related issue: #22122 |
…#22491)" (angular#23150) This reverts commit 71d75f4.
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
role="grid"
with appropriate keyboard interactions