347374 - SVG: pointer-events="stroke" doesn't respect stroke-width

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Andreas Neumann]

User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8.1b1) Gecko/20060710 Firefox/2.0b1
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.8.1b1) Gecko/20060710 Firefox/2.0b1

In the provided example there are two lines: one "invisible" (stroke="none") which has the pointer-events set to "stroke" and a stroke-width of "20". The second, red  line on top, has the pointer-events set to "none" and is only for reference for where to click.

It seems like MozillaSVG doesn't respect the stroke-width when pointer-events="stroke".

If you click exactly at the center of the red line, the click event is fired, but it should be fired everywhere at the red line, not just at the center.

This bug appears on all platforms and in all version up to now (FF1.5x, FF2 and FF3 trunk)

Reproducible: Always

Steps to Reproduce:
1. load the file
2. click on the red line
3. you should receive an alert wherever you click on the red line

Actual Results:  
the stroke-width of the first line is not respected. It treats it like a line with a stroke-width of 1.

Expected Results:  
the stroke-width should be respected

Comment 1

[tor]

Attachment: set up stroke attributes before hit detection

Comment 2

[tor]

Attachment: testcase of some other interesting cases

Comment 3

[tor]

Attachment: testcase to make sure invalidation isn't broken

Comment 4

[tor]

Attachment: more complete fix

Comment 5

[tor]

Checked in on trunk.

Comment 6

[tor]

Attachment: similar change for 1.8 branch

Comment 7

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

Comment on attachment 232602 [details] [diff] [review]
similar change for 1.8 branch

I don't really understand the code, but from an sr point of view it looks fine.

Comment 8

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 232602 [details] [diff] [review]
similar change for 1.8 branch

a=drivers, please land it on the branch.

Comment 9

[tor]

Checked in on MOZILLA_1_8_BRANCH.

Comment 10

[tor]

*** Bug 349999 has been marked as a duplicate of this bug. ***