springload / draftail

📝🍸 A configurable rich text editor built with Draft.js

Home Page:https://www.draftail.org/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Support full aria controls provided by draftJS

dwjohnston opened this issue · comments

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.)

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.

Implemented in #438.