Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Remove role attribute from recharts-wrapper (#4386)
<!--- Provide a general summary of your changes in the Title above --> <!--- Describe your changes in detail --> Before this PR, there was a `role` attribute on both the `.recharts-wrapper` parent DIV, as well as the SVG element. This is unnecessary, since the SVG element is enough. That extra role also caused problems, as I outlined in #4384 This PR removes the `role` attribute from the `.recharts-wrapper` element. The SVG element still retains its role. <!--- This project only accepts pull requests related to open issues --> <!--- If suggesting a new feature or change, please discuss it in an issue first --> <!--- If fixing a bug, there should be an issue describing it with steps to reproduce --> <!--- Please link to the issue here: --> #4384 <!--- Why is this change required? What problem does it solve? --> <!--- Please describe in detail how you tested your changes. --> <!--- Include details of your testing environment, and the tests you ran to --> <!--- see how your change affects other areas of the code, etc. --> <!--- What types of changes does your code introduce? Put an `x` in all the boxes that apply: --> - [x] Bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) <!--- Go over all the following points, and put an `x` in all the boxes that apply. --> <!--- If you're unsure about any of these, don't hesitate to ask. We're here to help! --> - [ ] My change requires a change to the documentation. - [ ] I have updated the documentation accordingly. - [ ] I have added tests to cover my changes. - [ ] I have added a storybook story or extended an existing story to show my changes
- Loading branch information