Bug 11911

Summary: SVG exhibits double-text-selection behavior
Product: WebKit Reporter: Eric Seidel (no email) <eric>
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: a.neumann, justin.garcia
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
URL: http://svg-whiz.com/svg/HideShow.svg
Attachments:
Description Flags
First attempt oliver: review+

Eric Seidel (no email)
Reported 2006-12-21 06:08:33 PST
SVG exhibits double-text-selection behavior Try selecting the explanation text in this example SVG: http://svg-whiz.com/svg/HideShow.svg You'll see that the text is selected *twice* by two separate selection regions. Very strange. It makes the text very difficult to read during selection. I'm not sure if this falls under "editing" or "SVG"
Attachments
First attempt (16.03 KB, patch)
2007-01-25 13:50 PST, Rob Buis
oliver: review+
Eric Seidel (no email)
Comment 1 2006-12-21 06:10:10 PST
Accidentally closed.
Andreas Neumann
Comment 2 2007-01-04 13:55:51 PST
Here is another very strange behavior with text selection: In this very simple example http://www.carto.net/papers/svg/manipulating_svg_with_dom_ecmascript/basic_doc_tree.svg if you double click on the rectangle, portions of the text in if you click further, the whole text seems to be selected, even if the mouse cursor does not hit the text element.
Eric Seidel (no email)
Comment 3 2007-01-18 04:37:45 PST
*** Bug 12003 has been marked as a duplicate of this bug. ***
Rob Buis
Comment 4 2007-01-25 13:50:36 PST
Created attachment 12673 [details] First attempt This patch works quite well, also with transformed text, just may have a slight issue with overlapping boxes. Cheers, Rob.
Oliver Hunt
Comment 5 2007-01-25 14:15:26 PST
Comment on attachment 12673 [details] First attempt in SVGInlineTextBox you should indent the class definition. Apparently this is expected in header files now :-/
Oliver Hunt
Comment 6 2007-01-25 14:36:14 PST
Comment on attachment 12673 [details] First attempt r+ pending style fixes :p
Oliver Hunt
Comment 7 2007-01-25 14:39:15 PST
Ooh, make sure any layout tests that need to be updated, are This should only effect pixelresults of selection tests in svg
Rob Buis
Comment 8 2007-01-26 01:39:44 PST
Landed in r19140.
Note You need to log in before you can comment on or make changes to this bug.