Closed
Bug 407011
Opened 17 years ago
Closed 4 months ago
Remove plaintext rules in quirks.css
Categories
(Core :: Layout, defect, P3)
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: martijn.martijn, Unassigned)
Details
Attachments
(2 files)
7.32 KB,
patch
|
dbaron
:
review-
|
Details | Diff | Splinter Review |
271 bytes,
text/html
|
Details |
<multicol> isn't really supported by any real browser, so it seems to me that those rules can be removed from quirks.css. I think the same applies for <plaintext>. Also, I think that the margin rule for <plaintext> can be removed in html.css, because in IE, <plaintext> always seems to have margin:0.
Reporter | ||
Comment 1•17 years ago
|
||
Reporter | ||
Updated•17 years ago
|
Attachment #291703 -
Flags: review?(dbaron)
Comment on attachment 291703 [details] [diff] [review] patch So... I've certainly had long enough to think about this. Sorry for the delay. I guess I think if the rationale is that nobody supports this, we should be making a decision to remove it completely. Removing a small percentage of the code to support these elements doesn't make much sense to me; there's still a good bit in the HTML parser. (I'd also probably want to coordinate with any HTML5 spec work that's going on as far as actually doing that.) So I think I prefer not to do this. (So review- as a policy decision; not because there's anything actually wrong with the patch.)
Attachment #291703 -
Flags: review?(dbaron) → review-
Comment 3•9 years ago
|
||
There's also bug 734063. And in https://www.w3.org/Bugs/Public/show_bug.cgi?id=27672 I got the request to remove it from HTML, which I think I'll grant once a patch is submitted since Firefox is the only browser that supports it.
Comment 4•8 years ago
|
||
Per HTML currently, plaintext has default margins and the quirk. multicol has no default styles or quirks. https://html.spec.whatwg.org/multipage/rendering.html#margin-collapsing-quirks multicol was removed from the spec in https://github.com/whatwg/html/pull/238
Updated•8 years ago
|
Priority: -- → P3
Reporter | ||
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WORKSFORME
![]() |
||
Updated•8 years ago
|
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Updated•2 years ago
|
Severity: normal → S3
Comment 5•4 months ago
|
||
multicol was removed in bug 1893342, updating this bug to only be about plaintext.
Summary: Remove multicol and plaintext rules in quirks.css → Remove plaintext rules in quirks.css
Comment 6•4 months ago
|
||
WONTFIX because the current spec includes plaintext
https://html.spec.whatwg.org/multipage/rendering.html#margin-collapsing-quirks
Status: REOPENED → RESOLVED
Closed: 8 years ago → 4 months ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•