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

Lines reversed in right-to-left alphabets #538

Open
romanHerolo opened this issue Dec 4, 2020 · 4 comments
Open

Lines reversed in right-to-left alphabets #538

romanHerolo opened this issue Dec 4, 2020 · 4 comments
Labels
bug Something isn't working regression Something that was working before rtl Issues related to right-to-left alphabets

Comments

@romanHerolo
Copy link

It appears that when using Hebrew/ Arabic strings, the output comes out in the correct direction, but when the sentence is long enough, the line breaks in reverse, i.e - first line (logic) appears as last line (visual).

Example:
expected output:
לווייתן מצוי (לעתים רורקל או רורקל ענק); שם מדעי: הוא יונק ימי ממשפחת לווייתני ענק שבסדרת הלווייתנאים. זהו הלווייתן השני בגודלו, ובעל החיים השני בגודלו החי כיום אחרי הלווייתן הכחול. הלווייתן המצוי עשוי להגיע לאורך של כמעט 27 מטר ומשקל של 120 טון אך אורכו נע בדרך כלל בין 18 ל-21 מטר ומשקלו הממוצע הוא כ-70 טון.

received output:
לאורך של כמעט 27 מטר ומשקל של 120 טון אך אורכו נע בדרך כלל בין 18 ל-21 מטר ומשקלו הממוצע הוא כ-70 טון.
הלווייתנאים. זהו הלווייתן השני בגודלו, ובעל החיים השני בגודלו החי כיום אחרי הלווייתן הכחול. הלווייתן המצוי
לווייתן מצוי (לעתים רורקל או רורקל ענק); שם מדעי: הוא יונק ימי ממשפחת לווייתני ענק שבסדרת

@timobrembeck
Copy link
Collaborator

What's the status of this issue? PR #541 claims to fix this issue, so can this be closed?

At least for Arabic and Farsi, we still experience this problem, see #662.

@seluianova
Copy link

seluianova commented Jan 23, 2023

@timoludwig I believe it's a different one.
Here the text order was broken.
In #662 the order is correct, the only problem is formatting.

@timobrembeck
Copy link
Collaborator

@seluianova Ah okay, thanks for clearing that up!

@timobrembeck timobrembeck changed the title Lines Reversed Lines reversed in right-to-left alphabets Feb 1, 2023
@timobrembeck timobrembeck added bug Something isn't working rtl Issues related to right-to-left alphabets labels Feb 1, 2023
@timobrembeck
Copy link
Collaborator

I'm not 100% sure, but I think this issue might have been resolved in the mean time and was re-introduced by f51bf87?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working regression Something that was working before rtl Issues related to right-to-left alphabets
Projects
None yet
Development

No branches or pull requests

3 participants