Closed Bug 734063 Opened 13 years ago Closed 7 years ago

Remove the multicol element from nsContentUtils::IsHTMLBlock

Categories

(Core :: DOM: Core & HTML, defect, P3)

Product:
Core
Component:
DOM: Core & HTML
Platform:
x86_64
Linux
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla64
Tracking Flags:
Tracking Status
firefox64 --- fixed

People

(Reporter: hsivonen, Assigned: dpino)

References

(Blocks 1 open bug)

Details

(Keywords: good-first-bug)

Attachments

(1 file)

Bug-734063-Remove-the-multicol-element-from-nsConten.patch
1.38 KB, patch
hsivonen
: review+
Details | Diff | Splinter Review
The HTML5 parser doesn't know about multicol. The plain text serializer and editor probably shouldn't, either.
Blocks: 1317665
Priority: -- → P3
Keywords: good-first-bug
Here's a patch. I initially thought it'd involve more changes.
Attachment #9016787 - Flags: review?(hsivonen)
Attachment #9016787 - Flags: review?(hsivonen) → review+
Treeherder looks good. OTOH, I cannot commit. I'd need to get the bug assigned so I can edit it and add tag "checkin-needed", or somebody has to commit for me.
Thanks! (Edited the assignment, etc.)
Assignee: nobody → dpino
Status: NEW → ASSIGNED
Keywords: checkin-needed
Pushed by ebalazs@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/c4a64c835820 Remove the multicol element from nsContentUtils::IsHTMLBlock r=hsivonen
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/c4a64c835820
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox64: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: