Closed
Bug 719385
Opened 14 years ago
Closed 14 years ago
Stroke hit-testing is buggy
Categories
(Core :: SVG, defect)
Core
SVG
Tracking
()
RESOLVED
DUPLICATE
of bug 676001
People
(Reporter: rustam_faizr, Unassigned)
Details
(Keywords: regression)
Attachments
(2 files, 1 obsolete file)
|
310 bytes,
image/svg+xml
|
Details | |
|
2.55 KB,
patch
|
Details | Diff | Splinter Review |
test.svg
User Agent: Mozilla/5.0 (Windows NT 5.1; rv:9.0.1) Gecko/20100101 Firefox/9.0.1
Build ID: 20111220165912
Steps to reproduce:
Highlight SVG Line and SVG Polyline on 'mouseover' event
Actual results:
SVG event 'mouseover' not work for 'nearly horizontal polyline and line'
Expected results:
event 'onmouseover'
This sample work correctly in all browsers.
Severity: normal → critical
|
||
Comment 2•14 years ago
|
||
Confirmed on Mozilla/5.0 (Windows NT 5.1; rv:12.0a1) Gecko/20120119 Firefox/12.0a1 ID:20120119031105
Only middle line changes color on mouseover. Tested Safari and Opera and all lines correctly change color.
Status: UNCONFIRMED → NEW
Component: Untriaged → SVG
Ever confirmed: true
Product: Firefox → Core
QA Contact: untriaged → general
Version: 9 Branch → Trunk
|
|
||
Updated•14 years ago
|
Severity: critical → major
|
|
||
Updated•14 years ago
|
Assignee: nobody → jwatt
|
|
||
Updated•14 years ago
|
OS: Windows XP → All
Hardware: x86 → All
Summary: SVG line and polyline mouseover → Hit testing for stroke is buggy
|
|
||
Comment 3•14 years ago
|
||
Mousing over the line, you'll see that there are small patches on the line which are not returning true for hit testing. The same problem exists on the right hand side of the circle.
Attachment #589822 -
Attachment is obsolete: true
|
|
||
Updated•14 years ago
|
Attachment #593146 -
Attachment is patch: false
Attachment #593146 -
Attachment mime type: text/plain → image/svg+xml
|
|
||
Updated•14 years ago
|
Assignee: jwatt → nobody
|
|
||
Comment 4•14 years ago
|
||
Seems like gfxContext::PointInStroke can't be relied upon to tell the truth.
|
|
||
Comment 5•14 years ago
|
||
If you use this mochitest to debug and put a breakpoint in nsSVGPathGeometryFrame::GetFrameForPoint it'll be hit four times. Twice for the first elementFromPoint() (once for the circle, once for the line), then twice for the second elementFromPoint() (once for the circle, once for the line). Both time's we're in nsSVGPathGeometryFrame::GetFrameForPoint for the circle, gfxContext::PointInStroke should return true. It only returns true the second time though.
|
|
||
Updated•14 years ago
|
Keywords: regression
Summary: Hit testing for stroke is buggy → Stroke hit-testing is buggy
|
||
Updated•14 years ago
|
Keywords: regressionwindow-wanted
|
|
||
Comment 6•14 years ago
|
||
It regressed some time between 3.6 and 4.0 it seems, but I didn't narrow the range any further than that.
|
||
Comment 7•14 years ago
|
||
Is this the same issue as bug 676001 it seems similar
|
|
||
Comment 8•14 years ago
|
||
bug 676001 has a regression range
|
|
||
Comment 9•14 years ago
|
||
Sure looks like it. Thanks, Robert.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → DUPLICATE
|
||
Updated•10 years ago
|
Keywords: regressionwindow-wanted
You need to log in
before you can comment on or make changes to this bug.
Description
•