Closed Bug 166591 Opened 17 years ago Closed 11 years ago
<marquee> created dynamically (e
.g . inner HTML) is not rendered
<marquee> in innerHTML is not "rendered"!! testcase: http://bugzilla.mozilla.gr.jp/showattachment.cgi?attach_id=1108 cf. http://www.w3.org/TR/REC-html40/appendix/notes.html#h-B.1 "If a user agent encounters an element it does not recognize, it should try to render the element's content."
To layout... opening the page in DOM Inspector and drilling down to the inner html:div in the anonymous content makes the text show... So we apply the binding. But at no point does the JS run...
Assignee: asa → attinasi
Status: UNCONFIRMED → NEW
Component: Browser-General → Layout
Ever confirmed: true
QA Contact: asa → petersen
I noticed this bug when I set -moz-binding to url('chrome://xbl-marquee/content/xbl-marquee.xml#marquee-horizontal') dynamically. See http://weblogs.mozillazine.org/asa/archives/004372.html. This code in xbl-marquee.xml is suspicious: //init needs to be run after the page has loaded in order to calculate the //correct height/width window.addEventListener("load", lambda, false); Maybe it should somehow check whether load has already fired?
Summary: <marquee> in innerHTML is not "rendered"!! → <marquee> created dynamically (e.g. innerHTML) is not rendered
How about running init() in the constructor and re-initing onload or something?
Has anyone found a way round this problem???
Yes -- don't try using DHTML with marquees.
Assignee: attinasi → nobody
Priority: P3 → --
QA Contact: chrispetersen → core.layout
Target Milestone: Future → ---
Note that the url testcase won't be fixed by following the suggestion in comment 4, because of bug 299231. This testcase would be fixed by following the suggestion in comment 7.
Er, I meant in comment 4. I think removing the addeventlistener and replacing it by a timer would be possible if the patch from bug 235564 got used. It would fix the attached testcase, but not the url testcase. But by adding the margin dynamically in the constructor instead of the xbl:content, the marquee contents would at least be visible. This would also be the case when you have js turned off.
This works at least since Fx3 Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b3pre) Gecko/20081127 Minefield/3.1b3pre ->WFM
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → WORKSFORME
I can still see the bug, using: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b3pre) Gecko/20081127 Minefield/3.1b3pre
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Comment on attachment 357700 [details] [diff] [review] patch >+++ b/layout/reftests/reftest.list >+# mathml/ >+include marquee/reftest.list s/mathml/marquee/ Looks good with that.
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
Assignee: nobody → martijn.martijn
Target Milestone: mozilla1.9.2a1 → ---
You need to log in before you can comment on or make changes to this bug.