Closed Bug 15432 Opened 25 years ago Closed 24 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: 24 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. ***
You need to log in before you can comment on or make changes to this bug.