Closed
Bug 717870
Opened 13 years ago
Closed 13 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 |
testcase (clicking should have no effect)
No description provided.
|
||
Comment 1•13 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•13 years ago
|
Assignee: nobody → longsonr
|
|
Assignee | |
Comment 2•13 years ago
|
||
Attachment #588486 -
Flags: review?(dholbert)
|
|
||
Comment 3•13 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•13 years ago
|
||
Flags: in-testsuite+
Target Milestone: --- → mozilla12
|
||
Comment 5•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•