Closed Bug 370757 Opened 14 years ago Closed 14 years ago

Vertical RTL marquee appears outside its container

Categories

(Core :: Layout, defect)

PowerPC
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: uriber, Assigned: uriber)

References

()

Details

(Keywords: regression, rtl, testcase)

Attachments

(2 files)

Attached file testcase
In the URL above, notice that the news ticker on the left appears to the left of its intended position (which is inside a table cell).
The testcase shows that this also happens when a container is an RTL div. 

This regressed between 2007-01-27 and 2007-01-28, almost certainly due to the fix for bug 163504.

Note also that horizontal RTL marquees have been broken for a while (bug 336736).
Flags: blocking1.9?
Probably margin-right: 100%; at:
http://lxr.mozilla.org/seamonkey/source/layout/style/xbl-marquee/xbl-marquee.xml#667
needs to be -moz-margin-end: 100%;
Yeah... patch coming right up.
Attached patch patchSplinter Review
What Martijn said.
Assignee: nobody → uriber
Status: NEW → ASSIGNED
Attachment #255467 - Flags: superreview?(dbaron)
Attachment #255467 - Flags: review?(dbaron)
Comment on attachment 255467 [details] [diff] [review]
patch

r+sr=dbaron
Attachment #255467 - Flags: superreview?(dbaron)
Attachment #255467 - Flags: superreview+
Attachment #255467 - Flags: review?(dbaron)
Attachment #255467 - Flags: review+
Checked in:
Checking in mozilla/layout/style/xbl-marquee/xbl-marquee.xml;
/cvsroot/mozilla/layout/style/xbl-marquee/xbl-marquee.xml,v  <--  xbl-marquee.xml
new revision: 1.27; previous revision: 1.26
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Flags: blocking1.9?
Resolution: --- → FIXED
Mass-assigning the new rtl keyword to RTL-related (see bug 349193).
Keywords: rtl
You need to log in before you can comment on or make changes to this bug.