Bug 11911 - SVG exhibits double-text-selection behavior
Summary: SVG exhibits double-text-selection behavior
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 420+
Hardware: Mac OS X 10.4
: P2 Normal
Assignee: Nobody
URL: http://svg-whiz.com/svg/HideShow.svg
Keywords:
: 12003 (view as bug list)
Depends on:
Blocks:
 
Reported: 2006-12-21 06:08 PST by Eric Seidel (no email)
Modified: 2007-01-26 01:39 PST (History)
2 users (show)

See Also:


Attachments
First attempt (16.03 KB, patch)
2007-01-25 13:50 PST, Rob Buis
oliver: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Seidel (no email) 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"
Comment 1 Eric Seidel (no email) 2006-12-21 06:10:10 PST
Accidentally closed.
Comment 2 Andreas Neumann 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.
Comment 3 Eric Seidel (no email) 2007-01-18 04:37:45 PST
*** Bug 12003 has been marked as a duplicate of this bug. ***
Comment 4 Rob Buis 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.
Comment 5 Oliver Hunt 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 :-/
Comment 6 Oliver Hunt 2007-01-25 14:36:14 PST
Comment on attachment 12673 [details]
First attempt

r+ pending style fixes :p
Comment 7 Oliver Hunt 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
Comment 8 Rob Buis 2007-01-26 01:39:44 PST
Landed in r19140.