274322 - bug with MARQUEE alternate, shows nothing

Status: RESOLVED FIXED

(Core :: Layout, defect)

Description

[Marc Falesse]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7) Gecko/20040803 Firefox/0.9.3
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7) Gecko/20040803 Firefox/0.9.3

My MARQUEE used to work with previous version of Mozilla, now it doesn't anymore.
I've tried out several HTML syntaxe but it couldn't help ! when i use TABLE i
get very tiny pictures that desapere very quick

Reproducible: Always
Steps to Reproduce:
http://www.sunfactory.fr
Actual Results:  
doesn't show the MARQUEE, but used to work fine (just like IE) before i updated
Mozilla.

Expected Results:  
the MARQUEE to go from left to right, and right to left.

Comment 1

[Sean Hayward]

Confirm behavour with latest build... 

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8a6) Gecko/20041209
Firefox/1.0+

Comment 2

[Marc Falesse]

well if you like i confirm it doesn't work with Firefox 1.0, the reason of my
been on previous version was downgrading to do tests.
MARQUEE used to work with Mozilla (i've never seen it working on Firefox) but
now it doesn't anymore.

Comment 3

[Boris Zbarsky [:bzbarsky]]

Looks like a regression from bug 167224 -- it broke the day that was checked in.
 Removing the "alternate" makes things happy.

Comment 4

[Mats Palmgren (inactive)]

Attachment: Distilled from http://www.sunfactory.fr

Comment 5

[Mats Palmgren (inactive)]

*** Bug 276969 has been marked as a duplicate of this bug. ***

Comment 6

[Mats Palmgren (inactive)]

There's a better testcase in duped bug 276969:
https://bugzilla.mozilla.org/attachment.cgi?id=170235

Comment 7

[Martijn Wargers (dead)]

*** Bug 297908 has been marked as a duplicate of this bug. ***

Comment 8

[j.j.]

Seems bug 244264 has the same problem, see testcase 
https://bugzilla.mozilla.org/attachment.cgi?id=149029

Comment 9

[j.j.]

Shouldn't this (and duplicate? bug 244264) have higher priority?
This is a regresson, and content is lost.

Comment 10

[Martijn Wargers (dead)]

Attachment: patch

wip patch, heavily polluted with patch from bug 313642.
This is sort of the logic that is needed to get it working. direction="up" and direction="down" also needs to be fixed.
This sort of regresses https://bugzilla.mozilla.org/attachment.cgi?id=170235 , but I suspect it is currently not working well either (only for that specific case, sometimes).

Comment 11

[Doron Rosenberg (IBM)]

can't believe I missed this bug.

Just to be clear, what alternate does is start with all content visible, and then it scrolls as much as it can untill a part of the content would be hidden, and then switches to the next direction (bouncing text for example).

Comment 12

[Martijn Wargers (dead)]

Attachment: patch2

Comment 13

[Martijn Wargers (dead)]

Attachment: testcase2

Comment 14

[Martijn Wargers (dead)]

Attachment: testcase3

Comment 15

[Martijn Wargers (dead)]

Attachment: patch3

Ok, this patch fixes testcase2 for me. Testcase3 is also working mostly, except marquee2 is too slow. I'll look at that some other time.

Comment 16

[Martijn Wargers (dead)]

Attachment: testcase4

I also tested with this one, it's bit messy though.
With this testcase I discovered that DOMAttrModified isn't working well anymore (I filed bug 314343 about that).

Comment 17

[Boris Zbarsky [:bzbarsky]]

Fixed on trunk.

Comment 18

[Martijn Wargers (dead)]

*** Bug 244264 has been marked as a duplicate of this bug. ***

Comment 19

[Ria Klaassen (not reading all bugmail)]

*** Bug 331761 has been marked as a duplicate of this bug. ***

Comment 20

[Martijn Wargers (dead)]

*** Bug 339157 has been marked as a duplicate of this bug. ***

Comment 21

[kennexc]

The bug happened when the text inside the marquee need to be displayed not only in one line. It is long text like this: 

<marquee behavior="ALTERNATE">This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text This is text</marquee>

Comment 22

[Patrick Dark]

Why is this even here at all; isn't the marquee element a proprietary (non-standards) creation of Microsoft? I would think that would mean that Mozilla shouldn't even support it. If you really want marquees, get Mozilla to implement section 18 of the CSS3 Box Model Module (http://www.w3.org/TR/css3-box/#marquee).