Bug 12391 - SVG containers with viewports should respect overflow when hit testing
Summary: SVG containers with viewports should respect overflow when hit testing
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:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-01-24 02:04 PST by Rob Buis
Modified: 2007-01-25 06:40 PST (History)
0 users

See Also:

Attachments
First attempt (11.95 KB, patch)
2007-01-24 02:28 PST, Rob Buis 		mjs: review-
Details | Formatted Diff | Diff
Improved patch (11.92 KB, patch)
2007-01-25 01:28 PST, Rob Buis 		mjs: review+
Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Rob Buis 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.
Comment 1 Rob Buis 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.
Comment 2 Maciej Stachowiak 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.
Comment 3 Rob Buis 2007-01-25 01:28:29 PST 
Created attachment 12663 [details]
Improved patch

This patch should address Maciej's issue.
Cheers,

Rob.
Comment 4 Maciej Stachowiak 2007-01-25 02:38:47 PST 
Comment on attachment 12663 [details]
Improved patch

r=me
Comment 5 David Kilzer (:ddkilzer) 2007-01-25 06:40:32 PST 
Committed by rwlbuis in r19115.