321745 - Bidi: Wrapped text containing inlines with explicit RTL embedding is garbled after resizing the window or the text

Status: RESOLVED WORKSFORME

(Core :: Layout: Text and Fonts, defect)

Description

[lɛʁi לערי ריינהארט]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8) Gecko/20051111 Firefox/1.5
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8) Gecko/20051111 Firefox/1.5

Hallo!

I worked on "wrap-around" issues when I discoved this bug.

*steps to reproduce*
- Please go to
http://wikisource.org/w/index.php?title=User:Gangleri/tests/BiDi/template/InterLingvLigoj&oldid=191409&action=submit
and click on the button "Show preview".
- Please resize the browser window according to the folowing screen shot.
- The list of language names is displayed incorrectly. For example "Suomi" is displayed as "ouS" in one filed and "mi" in the field right to it (together with parts of other language names.

best regards reinhardt [[user:gangleri]]

Reproducible: Always

Steps to Reproduce:
resize de window both increasing an decreasing the size
if the problem does not happen the first time then you should click on "Show preview" once more
Actual Results:  
rendering of individual items in a list of [block a is rtl] [block b is rtl] [block c is rtl] etc. depends on the window size

Expected Results:  
rendering of individual items in a list of [block a is rtl] [block b is rtl] [block c is rtl] etc. should not brake the Bidi direction inside the blocks when the window is resize

Comment 1

[lɛʁi לערי ריינהארט]

Attachment: screen shot for bug 321745

Comment 2

[lɛʁi לערי ריינהארט]

see also
http://f25.aaa.livedoor.jp/~suisui/pukiwiki14/pukiwiki.php?plugin=attach&pcmd=open&file=20051229screen.png&refer=gangleri

where  | is "splitted" at "Lara" |

Comment 3

[Uri Bernstein]

I can't reproduce this on Mac:
Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8) Gecko/20051111 Firefox/1.5

So it might be a Windows-specific thing.

And I can't see a bug in the screenshot linked from comment #2.

Comment 4

[Simon Montagu :smontagu]

I can't reproduce the bug either (Mozilla/5.0 (Windows; U; Win98; en-US; rv:1.8) Gecko/20051111 Firefox/1.5) and it doesn't help that I'm not sure exacly where I should be looking.

Reinhardt, can you create a more minimized testcase?

Comment 5

[lɛʁi לערי ריינהארט]

The url demonstrates also that Japanease and Korean words are wrapped character by character and not word by word. I tried to identify such bugs but could find only an old one:
Bugzilla Bug 41461: Japanese is not wrapped in correct position compared with ascii.
Would be happy to know if this is still an open issue and if workarounds are known to avoid this.

Comment 6

[Simon Montagu :smontagu]

As far as I know, wrapping between ideographic characters is correct.

Comment 7

[lɛʁi לערי ריינהארט]

Attachment: screen shot #2 for bug 321745

steps to reproduce:
please tray to measure the window width in pixels; I do not have a tool for this
please make your browser window same size

please go to
http://wikisource.org/w/index.php?title=User:Gangleri/tests/BiDi/bugzilla.mozilla/312745&oldid=191419
click on the edit tab
click on the "Show preview" button
if the window sizes match you should see this error
else resize the window unless you see similar intermixing of the "language names"

in the history tab you will see more versions; all of them generate this error;
I deleted iteratively more and more "blocks"; if I delete one additional line (from the bottom) the error will not happen; I did not tray all combinations (to delete inbetween);

the url is a test page; the old evision will be preserved whatever you change; once you have seen the error feel free to experiment by deleting more lines; exchanging them; replacing A, B, C, D with A, A, A, A etc.

new screen shot will follow

Comment 8

[lɛʁi לערי ריינהארט]

Attachment: screen shot #3 for bug 321745

url's
http://wikisource.org/w/index.php?title=User:Gangleri/tests/BiDi/bugzilla.mozilla/312745&oldid=191420
http://wikisource.org/w/index.php?title=User:Gangleri/tests/BiDi/bugzilla.mozilla/312745&oldid=191422

Hope that this error would be easy to fix: It seams that the common "pattern" of this bug is that at some point a "block" as "Basa Sunda" or "Lingua Latina" is wraped between lines. The last examples contains only 7 bit ASCII characters. WOW!

Another bug is that the last line is left aligned and not right aligned. This could be a subsequent error of the first.

Comment 9

[lɛʁi לערי ריינהארט]

hints to reproduce / confirm this bug

the screen shots have been made with MS Paint;
download and open the second or third attachment in MS Paint;
use the url associatiated with the screen shot and select the edit tab;
move your browser window above the screenshot;
adjust the width to have the *same* width as the screen shot;
click a few times on the "Sow preview" button unless you see the "mess"; I succeded the second time;
Good luck!

Comment 10

[Uri Bernstein]

Attachment: testcase

OK - I managed to reproduce this, but only using ctrl-plus (or ctrl-minus) to change the text size (in addition to adjusting the window width).

The attached testcase contains instructions at the bottom. Once you follow them, you should see "words" with mixed letters.

Comment 11

[Uri Bernstein]

Attachment: screenshot of testcase

This is after adjusting the window width to break between FFFFF and fffff, and pressing ctrl-+ once. The build ID is in the window title (latest trunk as of today).

Comment 12

[Uri Bernstein]

I can't reproduce this with Firefox 1.0, so this is a regression. I'll look for the regression window tomorrow.

Comment 13

[Uri Bernstein]

This regressed between 2005-01-24 and 2005-01-25, so almost certainly due to bug 248447.

Comment 14

[Jungshik Shin]

(In reply to comment #6)
> As far as I know, wrapping between ideographic characters is correct.
 Yes, there's no problem. However, I note that Korean Hangul is not ideographic. Neither is Japanese kana (although Chinese characters - Hanzi, Kanji and Hanja -  are)

Comment 15

[Uri Bernstein]

This is fixed by the patch which landed for bug 299065.
Because I'm not sure why that patch fixes this bug, I'm marking this WORKSFORME rather than FIXED (some underlying problem might still be there).

Comment 16

[(no longer active)]

Mass-assigning the new rtl keyword to RTL-related (see bug 349193).