Closed Bug 407011 Opened 17 years ago Closed 4 months ago

Remove plaintext rules in quirks.css

Categories

(Core :: Layout, defect, P3)

x86
Windows XP
defect

Tracking

()

RESOLVED WONTFIX

People

(Reporter: martijn.martijn, Unassigned)

Details

Attachments

(2 files)

Attached patch patchSplinter Review
<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.
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-
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.
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
Blocks: 1317665
Priority: -- → P3
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WORKSFORME
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Severity: normal → S3

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
No longer blocks: 1317665

WONTFIX because the current spec includes plaintext
https://html.spec.whatwg.org/multipage/rendering.html#margin-collapsing-quirks

Status: REOPENED → RESOLVED
Closed: 8 years ago4 months ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: