376952 - pointer-events not registered for elements using a clipPath with no fill

Status: RESOLVED FIXED

(Core :: SVG, defect)

Description

[Doug Schepers]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.3) Gecko/20070309 Firefox/2.0.0.3
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.1.3) Gecko/20070309 Firefox/2.0.0.3

Elements using clip-path should still receive pointer events.  If the element that they are clipping to (in the 'clipPath') has fill="none", then the pointer events do not seem to register on the clipped element.

Reproducible: Always

Steps to Reproduce:
1. In test file, click on the rectangles and circles 
2. an alert box identifying the clicked element should appear
3. 
Actual Results:  
rightmost circle, referencing the rect with no fill, does not fire an event

Expected Results:  
rightmost circle should fire an event

you might look at properties such as stroke, and also at style properties vs. presentation attributes

Comment 1

[Doug Schepers]

Attachment: minimal test case

Comment 2

[tor]

Attachment: ignore pointer-events and early exit for clipPath children

Comment 3

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Comment on attachment 261071 [details] [diff] [review]
ignore pointer-events and early exit for clipPath children

+  // check if we're a clipPath - cheaper than IsClipChild(), and we shouldn't
+  // get in here for other nondisplay children

This sounds fragile. Can you at least assert that it's really a clippath?

Comment 4

[JP Fiset]

Please, consider bug 380053

Comment 5

[Jonathan Watt [:jwatt]]

What happened here?

Comment 8

[tor]

(In reply to comment #5)
> What happened here?

Forgot to check in when it got reviews, then lost in the mists of time.

Comment 9

[tor]

Attachment: update to tip

Comment 10

[tor]

Checked in.

Comment 11

[jonathan chetwynd]

I'd like to reopen this bug for mac os x as this is broken

Comment 12

[Samuel Sidler (old account; do not CC)]

Jonathan, can you please file a new bug for the brokenness on Mac OS X and mark it as a dependency of this bug?

Comment 13

[jonathan chetwynd]

#11 files a separate bug:
Bug 407299