Closed Bug 15432 Opened 26 years ago Closed 26 years ago

'run-in' and 'compact' should not be recognised by CSS parser

Categories

(Core :: CSS Parsing and Computation, defect, P3)

Product:
Core
Component:
CSS Parsing and Computation
Type:
defect

Tracking

()

Status:
VERIFIED FIXED

People

(Reporter: ian, Assigned: pierre)

References

Details

(Keywords: css1)

If 'display:run-in' and 'display:compact' are not implemented, then the entire 'display' declaration in which they occur should be ignored. That is so that authors can easily work around this lack of feature. NOTE! THIS IS CURRENTLY A BUG! i.e., currently, the following: * { display: inline; } H2 { display: run-in; } DL[compact] { display: compact; } ...displays <H2> and <DL compact> elements as block level, not inline level, elements. It should not, since these values are not supported. We should either support these values, or totally ignore the declarations using them. This is due to the forward-compatability parsing rules: if a feature is not supported, then the declaration should be ignored. If run-in and compact are going to be implemented, then this bug is invalid. If they are not going to be implemented, though, then this should be fixed before release. 'display:compact' support is covered by bug 2055. 'display:run-in' support is covered by bug 2056.
Depends on: 2056
Depends on: 2055
Status: NEW → ASSIGNED
Target Milestone: M20
Good point. Deferring until just before release. I don't want to turn things off just yet...
Reassigning peterl's bugs to myself.
Accepting peterl's bugs that have a Target Milestone
Keywords: css1
Migrating from {css1} to css1 keyword. The {css1}, {css2}, {css3} and {css-moz} radars should now be considered deprecated in favour of keywords. I am *really* sorry about the spam...
Summary: {css1} 'run-in' and 'compact' should not be recognised by CSS parser → 'run-in' and 'compact' should not be recognised by CSS parser
This blocks bug #15432 (a *very* important bug IMO).
Blocks: 14983
OK... now that I look at my diffs more closely, I see you meant bug 14983.
Yes, sorry. They're both important though ... :)
Fix checked in nsCSSParser.cpp. Display:marker/run-in/compact are not supported. The parser drops any declaration that contains one of these display types.
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
Th related bugs were: bug 2055: compact bug 2056: run-in bug 14983: marker They have been closed as Later.
Using the following builds: Win: 07_20_08 Mac: 07_19_12 Linux: 07_20_09 verified 'run-in' and 'compact' not recognized. Marking bug verified fixed.
Status: RESOLVED → VERIFIED
*** Bug 189468 has been marked as a duplicate of this bug. ***
Blocks: 2055, 2056
No longer depends on: 2055, 2056
You need to log in before you can comment on or make changes to this bug.