WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
11661
SVG: stroke not sensitive to mouse events (hit testing fails)
https://bugs.webkit.org/show_bug.cgi?id=11661
Summary
SVG: stroke not sensitive to mouse events (hit testing fails)
Andreas Neumann
Reported
2006-11-21 03:30:46 PST
In this example I don't get alerts when clicking on the red line or the black stroke of the rectangle. On rare occasions I get alerts on the black stroke of the rectangle, but not reliable/reproducable. This might be related to bug
http://bugs.webkit.org/show_bug.cgi?id=11096
Attachments
First attempt
(1.86 KB, patch)
2006-11-22 12:29 PST
,
Rob Buis
eric
: review-
Details
Formatted Diff
Diff
Now with testcase
(19.01 KB, patch)
2006-11-22 12:46 PST
,
Rob Buis
eric
: review+
Details
Formatted Diff
Diff
Show Obsolete
(1)
View All
Add attachment
proposed patch, testcase, etc.
Rob Buis
Comment 1
2006-11-21 12:23:51 PST
Hi Andreas, So nice to see you entering svg bugs! (In reply to
comment #0
)
> In this example I don't get alerts when clicking on the red line or the black > stroke of the rectangle. On rare occasions I get alerts on the black stroke of > the rectangle, but not reliable/reproducable. > > This might be related to bug
http://bugs.webkit.org/show_bug.cgi?id=11096
That one really seems different, its about unclosed polylines IIRC. I investigated this one, the transforms/translations seem to mess it up. My bet is that it is due to the viewBox calculations. I'll work on it! Cheers, Rob.
Rob Buis
Comment 2
2006-11-22 12:29:02 PST
Created
attachment 11614
[details]
First attempt This makes the testcase work and also works with the existing tests. Cheers, Rob.
Eric Seidel (no email)
Comment 3
2006-11-22 12:31:23 PST
Comment on
attachment 11614
[details]
First attempt This needs a test case. (or needs to have fixed an existing one). Otherwise it looks fine.
Rob Buis
Comment 4
2006-11-22 12:46:51 PST
Created
attachment 11615
[details]
Now with testcase MacDome correctly pointed out the patch lacked a testcase, so there :) Cheers, Rob.
Eric Seidel (no email)
Comment 5
2006-11-22 13:24:43 PST
Comment on
attachment 11615
[details]
Now with testcase Well, it's not the worlds simplest test case. But assuming it failed before and now succeeds, your patch looks good!
David Kilzer (:ddkilzer)
Comment 6
2006-11-23 14:51:51 PST
rwlbuis committed
r17876
.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug