-
Notifications
You must be signed in to change notification settings - Fork 939
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
Provide tabIndex prop to allow for sequential tabbing in forms #232
Comments
I think However, I can't see how the same could be done for the toolbar, without either providing a custom toolbar, or monkey patching the elements (which I won't do). I suggest pushing your proposal upstream to Quill itself. In the meantime, you can accomplish both, in a slightly more verbose way, this by providing a custom editing area and a custom toolbar. |
Added tabIndex prop for built-in editing area. Closes #232
The new I could provide a little PR for that although I guess that should be handled by Quill itself instead. |
Quill unfortunately doesn't provide an API to set the editor's tabindex, so we have to override it directly on its dom node. Also see zenoamaro#232 (comment)
* Propagate tabindex prop to editor's scroll container Quill unfortunately doesn't provide an API to set the editor's tabindex, so we have to override it directly on its dom node. Also see #232 (comment) * Update component.js
* Propagate tabindex prop to editor's scroll container Quill unfortunately doesn't provide an API to set the editor's tabindex, so we have to override it directly on its dom node. Also see zenoamaro/react-quill#232 (comment) * Update component.js
* Propagate tabindex prop to editor's scroll container Quill unfortunately doesn't provide an API to set the editor's tabindex, so we have to override it directly on its dom node. Also see zenoamaro/react-quill#232 (comment) * Update component.js
When quill is part of a form where previous/next inputs have
tabindex
> 0, sequential tabbing is lost, as elements of quill do not allow for tabIndex to be specified. It would be great if all elements generated by quill had their tabIndex explicitly set to a defined prop that defaulted to 0.Quill version
The text was updated successfully, but these errors were encountered: