854221 - marquee direction change makes it improperly scroll on 2nd turn

Status: RESOLVED WORKSFORME

(Core :: General, defect)

Description

[Mr_KrzYch00]

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:22.0) Gecko/20130324 Firefox/22.0
Build ID: 20130324031024

Steps to reproduce:

I changed on my site how marquee should be working with noscript as well as with proper javascript support. Without noscript it uses ' ::: ' as a sepearator for messages and scroll with dection="left" behavior="alternate".
To properly support javascript browser [without text overlapping outside of marquee while page is loading] I added javascript function to body onload getting marquee content, doing replace(/ ::: /g,'<br>');, running other function to replace links, put it back to marquee element, and as last line, change direction of marquee to 'up' as setAttribute('direction','up').


Actual results:

The marquee jumps a bit [like frame skips] when going down first, then it properly scrolls back up (may be some lag at my side). After getting back up it goes a bit too high, by like 1 line, and scrolls 1 line down and 1 line up over and over again.

Address to see improper behaviour: http://www.4my.eu/ptp1/mrkrzych00


Expected results:

It should respect scrolldelay and scrollamount settings as well as properly read number of lines that were generated by javascript replace command, replacing separator " ::: " with <br>.

Open it with Google Chrome for comparison. There is works like it should.

Comment 1

[(mostly gone) XtC4UaLL [:xtc4uall]]

FWIW, this works as expected with Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.8.1.20) Gecko/20081217 Firefox/2.0.0.20 ID:2008121709

with Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.0.19) Gecko/2010031422 Firefox/3.0.19 ID:2010031422 it is stuck from the Beginning on

with Mozilla/5.0 (Windows; U; Windows NT 6.1; en-US; rv:1.9.2.28) Gecko/20120306 Firefox/3.6.28 ID:20120306064154 CSet: b2576dbcd614 the Behavior changed as to what can still be seen in Trunk.

Comment 2

[Martijn Wargers (dead)]

Attachment: testcase

I can see an issue that is basically bug 314445.

Comment 3

[Martijn Wargers (dead)]

One problem the testcase is also exposing is that it is changing the direction to up on onload.

When direction=up is set dynamically, we should make sure the constructor code isn't being run another time at least.

Comment 4

[Karl Dubost💡 :karlcow]

This is working the same in Firefox, Chrome and Safari.