Closed
Bug 717870
Opened 12 years ago
Closed 12 years ago
Space is incorrectly ignored when adding text to <svg:text>
Categories
(Core :: SVG, defect)
Core
SVG
Tracking
()
RESOLVED
FIXED
mozilla12
People
(Reporter: jruderman, Assigned: longsonr)
References
Details
Attachments
(2 files)
603 bytes,
image/svg+xml
|
Details | |
5.67 KB,
patch
|
dholbert
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Comment 1•12 years ago
|
||
Confirmed. Same behavior at least as far back as Firefox 3.5.16, so not a regression, at least not a recent one. Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1.16) Gecko/20101130 Firefox/3.5.16
OS: Mac OS X → All
Hardware: x86_64 → All
Assignee | ||
Updated•12 years ago
|
Assignee: nobody → longsonr
Assignee | ||
Comment 2•12 years ago
|
||
Attachment #588486 -
Flags: review?(dholbert)
Comment 3•12 years ago
|
||
Comment on attachment 588486 [details] [diff] [review] patch Looks great! One suggestion -- a brief comment would be helpful above or inside SetWhitespaceHandling(), just to be clear up-front what it's trying to do. Something like > // Trim whitespace up to the first non-whitespace frame > // and after the last non-whitespace frame. r=me with that
Attachment #588486 -
Flags: review?(dholbert) → review+
Assignee | ||
Comment 4•12 years ago
|
||
pushed https://hg.mozilla.org/integration/mozilla-inbound/rev/256c57537507
Flags: in-testsuite+
Target Milestone: --- → mozilla12
Comment 5•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/256c57537507
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•