Bug 12391

Summary: SVG containers with viewports should respect overflow when hit testing
Product: WebKit Reporter: Rob Buis <rwlbuis>
Component: SVGAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 420+   
Hardware: Mac   
OS: OS X 10.4   
Attachments:
Description Flags
First attempt
mjs: review-
Improved patch mjs: review+

Rob Buis
Reported 2007-01-24 02:04:20 PST
The summary says it all. You can note this effect in carto.net/window.svg on the buttons.
Attachments
First attempt (11.95 KB, patch)
2007-01-24 02:28 PST, Rob Buis 		mjs: review-
DetailsFormatted DiffDiff
Improved patch (11.92 KB, patch)
2007-01-25 01:28 PST, Rob Buis 		mjs: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Rob Buis
Comment 1 2007-01-24 02:28:02 PST
Created attachment 12642 [details] First attempt This patch should fix the problem. I'll look whether there are problems too for other svg elements that create a viewport. Cheers, Rob.
Maciej Stachowiak
Comment 2 2007-01-24 18:33:38 PST
Shouldn't you respect the overflow property in every case, not just on the SVG element? It's a little weird to have a tag name check like that in the render tree. r- to consider this, otherwise patch and test case look great.
Rob Buis
Comment 3 2007-01-25 01:28:29 PST
Created attachment 12663 [details] Improved patch This patch should address Maciej's issue. Cheers, Rob.
Maciej Stachowiak
Comment 4 2007-01-25 02:38:47 PST
Comment on attachment 12663 [details] Improved patch r=me
David Kilzer (:ddkilzer)
Comment 5 2007-01-25 06:40:32 PST
Committed by rwlbuis in r19115.
Note You need to log in before you can comment on or make changes to this bug.