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
Bidi text model doesn't take into account wrapped lines #1757
Comments
Well... it seems there as a huge glaring hole in the current handling of right-to-left text -- in that my code assumes that the text is displayed entirely inverted, whereas it'll only be inverted per-line. Currently, the editor code leaves line breaking entirely up to the browser, and doesn't observe where the line breaks end up, but getting this right would involve changing that somehow. Which is going to be messy. I am not sure how to go about fixing this. Will think about it a bit more. #1756 might simply be a different symptom of this problem. |
rtlMoveVisually helped me. |
This should be fixed in master since #4463 got merged. |
Applause. Thanks for taking care of this! |
(Tested everywhere)
<dd>
and move rightBoth in chrome and firefox on linux, in a textarea in this case the cursor never jumps (just like on a non-wrapped line), but in CodeMirror with
rtlMoveVisually
set to true, the cursor moves to the lower part of the rtl text first. I don't know "how wrong" this would seem to a speaker of a rtl language, though.The text was updated successfully, but these errors were encountered: