-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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
bug(COMPONENT): Sort Header #30196
Comments
@amysorto sorry for pinging, if want to look into this problem how can is test it locally, like i cloned the repo, installed packages through yarn install, but was not able to run something to test it locally, can you guide me or share some docs for runinng and testing material components locally and fixing problems |
@smil-thakur If you are interested in looking into contributing a fix, here are instructions for our repo to set up your dev environment: https://github.com/angular/components/blob/main/DEV_ENVIRONMENT.md. The dev app has examples for table once it's run locally which you can test with. |
Fixes that the sort header wasn't showing the arrow when it's focused. Fixes angular#30196.
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. |
Is this a regression?
The previous version in which this bug was not present was
v18
Description
When focusing on sort header using "tab" key previously on version 18 the sort headers were visible with less opacity but on version 19 the header icons are not visible
old;:
new:
as you can see on new one the header icons not visible on focus
Reproduction
https://material.angular.io/components/sort/overview
check over here and compare with
https://v18.material.angular.io/components/sort/overview
Expected Behavior
expected behavior should be sort icon should be visible with some opacity on focus and on click or enter press it should be visible with full opacity
Actual Behavior
sort icon is not at all visible on focus
Environment
The text was updated successfully, but these errors were encountered: