We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
It looks like the only aria tag you are providing is the the ariaDescribedBy.
ariaDescribedBy
This makes it difficult to access the component when using React Testing Library.
Pass the full list of aria props as listed here on to the draft js component:
https://github.com/facebook/draft-js/blob/master/src/component/base/DraftEditorProps.js
It looks like you aren't support arbitrary passing of props to base elements, like Material-UI often does.
(Write your answer here.)
The text was updated successfully, but these errors were encountered:
Update DraftailEditor.js
07e765d
Suuport springload#436
Hey @dwjohnston, thank you for the report, and the PR. Not having those props seems like oversight from my part more than anything.
I’ll triple check how the Draft.js 0.10 -> 0.11 situation might affect this and review your PR.
Sorry, something went wrong.
efc0c9c
Implemented in #438.
No branches or pull requests
Is your proposal related to a problem?
It looks like the only aria tag you are providing is the the
ariaDescribedBy
.This makes it difficult to access the component when using React Testing Library.
Describe the solution you’d like
Pass the full list of aria props as listed here on to the draft js component:
https://github.com/facebook/draft-js/blob/master/src/component/base/DraftEditorProps.js
Describe alternatives you’ve considered
It looks like you aren't support arbitrary passing of props to base elements, like Material-UI often does.
Additional context
(Write your answer here.)
The text was updated successfully, but these errors were encountered: