429849 - marquee text not on one line , part of this line on top and second part on bottom

Status: RESOLVED FIXED

(Core :: Layout, defect)

Description

[Al-Mothafar A-Hasan]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9pre) Gecko/2008041907 Minefield/3.0pre
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9pre) Gecko/2008041907 Minefield/3.0pre

On latest trunk version only...
I download it today from link on Bug 429825 Comment #2 :

OK , I found another bug on another website (not mine) , and this problem not founded on Firefox 2 or 3 : http://www.kotlah.org/forum/index.php

Look at marquee text , its 2 line not one , this marquee must be one line , the second line its continue of first line , so first line (on top) not competed at same line , the continues of first line on second line , this not founded on another arabic website , i found it on the website only, but on firefox 2 and 3 the marquee completed in on line , no second line..

Sorry for this bad details , but my English not good , i try to joining you to make this product the best on this word.

Note : this marquee was not fully compatible whit version 2 , the problem was if you need to click on a link on this marquee , all text will changed to and you need the link to appear again on second cycle , but on version 3 beta 5 it seem fully compatible and not has any problem.

Reproducible: Always

Steps to Reproduce:
1.
2.
3.

Comment 1

[Al-Mothafar A-Hasan]

other website : http://www.yu-std.com/forum/index.php

Comment 2

[Martijn Wargers (dead)]

Ugh, this might be a regression from the patch from bug 413027.
This seems to be an incremental reflow bug.

Comment 3

[Martijn Wargers (dead)]

Attachment: testcase

Comment 4

[Martijn Wargers (dead)]

Attachment: reference

Comment 5

[Martijn Wargers (dead)]

Attachment: patch

Workaround patch for the workaround patch.
It passes current reftests and I also tested this on some websites.

Comment 6

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Martijn, if you could create a bug with testcase for the underlying incremental reflow issue, that'd be great.

Comment 7

[Martijn Wargers (dead)]

Well, it's probably related/the same as bug 121142.

Comment 8

[Damon Sicore (:damons)]

Comment on attachment 318182 [details] [diff] [review]
patch

a1.9+=damons

Comment 9

[:Gavin Sharp [email: [e-mail]]]

mozilla/layout/style/xbl-marquee/xbl-marquee.xml 	1.33
mozilla/layout/reftests/bugs/429849-1-ref.html 	1.1
mozilla/layout/reftests/bugs/429849-1.html 	1.1
mozilla/layout/reftests/bugs/reftest.list 	1.452