Bug 45107

Summary: SVG atlas map slider doesn't work properly
Product: WebKit Reporter: Nikolas Zimmermann <zimmermann>
Component: SVGAssignee: Nikolas Zimmermann <zimmermann>
Status: RESOLVED FIXED    
Severity: Normal CC: aestes, krit, svg, zimmermann
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: PC   
OS: OS X 10.5   
Attachments:
Description Flags
Partly reduced testcase
none
Patch krit: review+

Description Nikolas Zimmermann 2010-09-02 05:00:00 PDT
Created attachment 66360 [details]
Partly reduced testcase

The SVG Atlas Map from IE9 Test Center (vis.uell.net) contains a slider, which doesn't respond to move events as expected.
A mousemove listener is attached to the <svg> root element, and events are only fired, when moving the mouse over areas that contain graphical elements.

SVG 1.1 2nd edition also allows to fire events when hovering over container elements (like <svg>). The old behaviour implemented in WebKit now, also doesn't make any sense.

This was brought to attention @ SVG Open 2010, Michael Neutze the author of atlas map was so kind to reduce the problem, attaching his testcase (not really reduced, but I'm containing two reduced testcases in my patch)
Comment 1 Nikolas Zimmermann 2010-09-03 07:16:05 PDT
*** Bug 45170 has been marked as a duplicate of this bug. ***
Comment 2 Nikolas Zimmermann 2010-09-03 07:17:10 PDT
Created attachment 66490 [details]
Patch

Fixes all problems with the SVG "Bevoelkerungspyramide".
Comment 3 Dirk Schulze 2010-09-03 07:22:30 PDT
Comment on attachment 66490 [details]
Patch

I'd write the comment before the if, but that's  matter of opinion. r=me.
Comment 4 Nikolas Zimmermann 2010-09-03 07:28:05 PDT
Thanks Dirk, landed in r66731.

Michael, please retry with a nightly >= 66731, once it's available.
Comment 5 Michael N. 2010-09-06 10:03:10 PDT
ok, this looks very good and is a huge improvement.
tested in r66819 on OSX 10.6.4

I am still not sure if text that's set to "pointer-events: none" should be selectable but I now know how to cover that with prventDefault.
Comment 6 Nikolas Zimmermann 2010-09-07 12:59:03 PDT
(In reply to comment #5)
> ok, this looks very good and is a huge improvement.
> tested in r66819 on OSX 10.6.4
Good to hear, glad it works fine.

> I am still not sure if text that's set to "pointer-events: none" should be selectable but I now know how to cover that with prventDefault.
Hm, can you try to look up a relevant section in the SVG spec? My feeling is that you're right, if you can find any part of the spec that supports this view, please file another bug.
Happy to fix, it you find evidence that we're wrong :-) (despite what other browsers do)
Comment 7 Andy Estes 2011-01-04 20:24:57 PST
<rdar://problem/8432245>