Skip to content
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

Unexpected RichTextBox cmp behaviour #445

Open
andrew-bury opened this issue Aug 10, 2022 · 4 comments
Open

Unexpected RichTextBox cmp behaviour #445

andrew-bury opened this issue Aug 10, 2022 · 4 comments

Comments

@andrew-bury
Copy link

This component doesn't work properly with allowDefaultTabNavigation={true}. Instead of changing interactive element on the Tab key, it's adds indent to the RichTextBox component (ex: the same like in Code Editor).

This issue occurred a week or two ago.

@leiwsf
Copy link

leiwsf commented Aug 16, 2022

Thanks for reporting this. Can you clarify your use case? what are you trying to do with this component?

@andrew-bury
Copy link
Author

Thanks for reporting this. Can you clarify your use case? what are you trying to do with this component?

Official documentation (ref: https://quip.com/dev/liveapps/1.x.x/reference/react-ui/richtextbox#allowdefaulttabnavigation)

Imagine you have an app with some interactive elements like buttons, links, custom elements/components (which contains something with tabindex="0") etc... Then for example y wanna navigate trough the app by Tab key, all works fine until RichTextComponent not in focus. When focus appears on RichTextComponent and you press on the "Tab" key on your keyboard (This action should change focus to the next interactive element) it's adding an indent to the content inside editable div (this div placed inside RichTextComponent).

@leiwsf
Copy link

leiwsf commented Aug 16, 2022

Thanks for clarifying the use case!

@zhouhao
Copy link
Contributor

zhouhao commented Feb 20, 2023

BTW: This issue should be fixed in Q4 of 2022.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants