Closed Bug 1446584 Opened 2 years ago Closed 2 years ago

Should use HTML/CSS definition of whitespace

Categories

(Core :: SVG, enhancement)

enhancement
Not set

Tracking

()

RESOLVED FIXED
mozilla61
Tracking Status
firefox61 --- fixed

People

(Reporter: longsonr, Assigned: longsonr)

Details

Attachments

(1 file)

per https://www.w3.org/TR/SVG2/changes.html#types

Make whitespace include form feed (U+000C) to align with CSS and HTML.
Assignee: nobody → longsonr
Attachment #8959752 - Flags: review?(jwatt)
Comment on attachment 8959752 [details] [diff] [review]
use IsHTMLWhitespace everywhere

Review of attachment 8959752 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good, thanks!

(I wonder if nsContentUtils::IsHTMLWhitespace should be inline.)
Attachment #8959752 - Flags: review?(jwatt) → review+
Pushed by longsonr@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/a8b3ac1e9f25
Make SVG use the same definition of whitespace as HTML/CSS r=jwatt
https://hg.mozilla.org/mozilla-central/rev/a8b3ac1e9f25
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
You need to log in before you can comment on or make changes to this bug.