Closed Bug 1048 Opened 26 years ago Closed 26 years ago

CSS property text-indent not supported

Categories

(Core :: Layout, defect, P2)

x86
Other
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: dbaron, Assigned: buster)

References

()

Details

Attachments

(1 file)

(deleted), application/octet-stream
Details
text-indent is not supported. I think this is one of the more commonly used and better supported properties.‰
Status: NEW → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
I implemented it today.
Status: RESOLVED → REOPENED
Percentage values for text-indent should be based on the width of the parent element, not based on the width of the element itself. See the URL for the BLOCKQUOTE test within it. ‰
Resolution: FIXED → ---
‰
Fixed by repairing nsInlineReflow's text-indent handling to use the correct reflow-state for finding the containing block.
Status: REOPENED → ASSIGNED
Status: ASSIGNED → RESOLVED
Closed: 26 years ago26 years ago
Resolution: --- → FIXED
Status: RESOLVED → REOPENED
The percentages still aren't computed based on the width of the PARENT element. This is what is required by both CSS1 and CSS2.
Status: REOPENED → RESOLVED
Closed: 26 years ago26 years ago
It's working correctly. Look again. The spec says that the parent elements width is used. The parent of the blockquote is the body. The width of the indent is half the width of the page (used a ruler held up to my screen). here is my test case (also checked in as mozilla/layout/html/tests/block/text-indent.html) --CUT HERE-- <HTML> <HEAD> <STYLE type="text/css"> .in4 { width: 4in; border: solid 1px green; } .in3 { width: 3in; text-indent: 50%; border: solid 1px yellow; } .bq { margin-left: 20%; margin-right: 20%; border: solid 1px blue; text-indent: 50%; } </STYLE> </HEAD> <BODY> <DIV CLASS=in4> <DIV CLASS=in3> This text should be indented two inches because the parent element (the outer div) has a content-width of 4 inches and the text-indent for this element is set to 50%. </DIV> </DIV> <TABLE CLASS=in4 border=1> <TR> <TD> <DIV CLASS=in3> This text should be indented two inches because the parent element (the outer div) has a content-width of 4 inches and the text-indent for this element is set to 50%. </DIV> </TD> </TR> </TABLE> <BLOCKQUOTE CLASS=bq> This is text-indented by 50%. Because the blockquote has substantial margin values, the indent will place the text closer to the right edge of the blockquote (the blockquote is smaller than its parents because of its margin, but the text-indent is relative to the elements parent which is the BODY not the blockquote itself). </BLOCKQUOTE> </BODY> </HTML>
I looked again, and it does work correctly on the 12-07 build. I looked again at the 11-24 build, and it wasn't then. So I wasn't hallucinating, but everything is fine now.
QA Contact: 4110
setting QA_assigned to field.... dbaron, can you mark this VERIFIED-FIXED if its not a problem anymore in recent builds? thanks!
Status: RESOLVED → VERIFIED
Verified fixed
Attached file sdsd (deleted) —
tes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: