247287 - [FIXr]abs pos rtl block uses left static pos, not right static pos (bidi element with left:auto;right:auto)

Status: RESOLVED FIXED

(Core :: Layout: Positioned, defect, P1)

Description

[Hixie (not reading bugmail)]

An absolutely positioned element with left:auto;right:auto (in static position) 
doesn't take into account the directionality of the element when deciding 
whether to use the 'left' or 'right' property for alignment when they are both 
set to 'auto' as it should according to CSS2.1 section 10.3.7, first paragraph 
after the equation.

TESTCASE: http://www.hixie.ch/tests/adhoc/css/box/absolute/022.html

Tested with FF0.9, sorry if the trunk has already fixed this. Also, I think this 
might be a dupe but I couldn't find the other bug.

(o120248)

Comment 1

[Boris Zbarsky [:bzbarsky]]

Note
http://lxr.mozilla.org/seamonkey/source/layout/html/base/src/nsHTMLReflowState.cpp#984
-- has this changed from CSS2?

Comment 2

[Boris Zbarsky [:bzbarsky]]

Attachment: If the spec _has_ changed, this is the fix

Comment 3

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 153071 [details] [diff] [review]
If the spec _has_ changed, this is the fix

The blame for that comment is troy back in 2000, so it may be that the comment
is just wrong...  My reading of CSS2 certainly doesn't agree with that comment.

Comment 4

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Comment on attachment 153071 [details] [diff] [review]
If the spec _has_ changed, this is the fix

I've removed a whole bunch of other instances of that pattern.

Comment 5

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 153071 [details] [diff] [review]
If the spec _has_ changed, this is the fix

In that case, I think this is worth taking on the 1.7 branch.

Comment 6

[Boris Zbarsky [:bzbarsky]]

Checked in for 1.8a3 or 1.8b or whatever this next milestone is.

Comment 7

[Asa Dotzler [:asa]]

Comment on attachment 153071 [details] [diff] [review]
If the spec _has_ changed, this is the fix

If this didn't make it into Firefox 1.0 then we probably don't want it in 1.7.x
which is supposed to be a gecko equivalent for Firefox 1.0. Please re-request
approval if I'm wrong. Thanks.

Comment 8

[Boris Zbarsky [:bzbarsky]]

No, that's correct. This really needed to go into both 1.7 and aviary, in my
opinion, but approvals just took a long time.  :(

Comment 9

[(no longer active)]

Mass-assigning the new rtl keyword to RTL-related (see bug 349193).