-
Notifications
You must be signed in to change notification settings - Fork 2.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
Accessibility tests added to AreaChart, LineChart, VerticalBarChart #30059
Accessibility tests added to AreaChart, LineChart, VerticalBarChart #30059
Conversation
/azp run |
Azure Pipelines successfully started running 4 pipeline(s). |
📊 Bundle size report🤖 This report was generated against a0015d1d41f456bf3ff8a8702a79e54791c5b028 |
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit d00215d:
|
🕵 fluentuiv8 No visual regressions between this PR and main |
Asset size changesSize Auditor did not detect a change in bundle size for any component! Baseline commit: a0015d1d41f456bf3ff8a8702a79e54791c5b028 (build) |
packages/react-charting/src/components/LineChart/LineChart.base.tsx
Outdated
Show resolved
Hide resolved
/azp run |
Azure Pipelines successfully started running 4 pipeline(s). |
/azp run |
Commenter does not have sufficient privileges for PR 30059 in repo microsoft/fluentui |
change/@fluentui-react-charting-209f77d4-492f-4ffe-948f-1bd8220ff566.json
Outdated
Show resolved
Hide resolved
/azp run |
Azure Pipelines successfully started running 4 pipeline(s). |
Previous Behavior
No accessibility tests were added for AreaChart, LineChart, VerticalBarChart under react-charting
New Behavior
Accessibilty test has been added with the help of jest-axe.
![image](https://private-user-images.githubusercontent.com/35366351/290253507-e40d5003-9802-4880-a3e2-723651ec702b.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MzkxMTk4MjMsIm5iZiI6MTczOTExOTUyMywicGF0aCI6Ii8zNTM2NjM1MS8yOTAyNTM1MDctZTQwZDUwMDMtOTgwMi00ODgwLWEzZTItNzIzNjUxZWM3MDJiLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAyMDklMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMjA5VDE2NDUyM1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTE5NjQ5OTlkYjI4NmNmNjQ3YWNlNzMzNGI2NzIwZjMzMGQxYmZjMjU1MjUxYTM4MWRkODcwMTAwOTZkYzQ5MzgmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.tFDs52L8Hxgt1kyGmutYRnJkSHvwUaukhH7JLFD9UEc)
Related Issue(s)