Closed
Bug 1144779
Opened 9 years ago
Closed 9 years ago
Do not use mouse coordinates from onMouseUp to set selection end when dragging in Graph
Categories
(DevTools :: Performance Tools (Profiler/Timeline), defect, P2)
Tracking
(firefox40 fixed)
RESOLVED
FIXED
Firefox 40
Tracking | Status | |
---|---|---|
firefox40 | --- | fixed |
People
(Reporter: bgrins, Assigned: bgrins)
References
Details
Attachments
(1 file, 1 obsolete file)
4.18 KB,
patch
|
vporof
:
review+
|
Details | Diff | Splinter Review |
Right now the coordinates from the mouseup event are used to finalize the selection in the Graph. This can cause weirdness when a mouseup happens outside of the toolbox and then it is triggered from the next mousemove. STR: Start a new selection on graph Move the mouse up to the browser content Release mouse Move the mouse back into the timeline Notice that the selection ends at the location of the mouse currently, when it should stay where it was when the mouse went up. Dragging and resizing an existing selection doesn't have this behavior, because the coordinates are used from mousemove instead.
Assignee | ||
Comment 1•9 years ago
|
||
What do you think?
Assignee: nobody → bgrinstead
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•9 years ago
|
||
Comment on attachment 8579478 [details] [diff] [review] timeline-mouseup.patch Forgot to flag r?
Attachment #8579478 -
Flags: review?(vporof)
Comment 3•9 years ago
|
||
Comment on attachment 8579478 [details] [diff] [review] timeline-mouseup.patch Review of attachment 8579478 [details] [diff] [review]: ----------------------------------------------------------------- Need to fix bug 1066504 first. Sorry!
Attachment #8579478 -
Flags: review?(vporof)
Comment 4•9 years ago
|
||
Moving into the Profiler component. Filter on GUTHRIE'S WAVY CAKES.
Component: Developer Tools: Timeline → Developer Tools: Profiler
Updated•9 years ago
|
Updated•9 years ago
|
Comment 5•9 years ago
|
||
Brian, are you still planning on looking at these bugs? ( https://bugzilla.mozilla.org/showdependencytree.cgi?id=1147941&hide_resolved=1 )
Updated•9 years ago
|
Priority: -- → P2
Assignee | ||
Comment 6•9 years ago
|
||
(In reply to Victor Porof [:vporof][:vp] from comment #5) > Brian, are you still planning on looking at these bugs? ( > https://bugzilla.mozilla.org/showdependencytree. > cgi?id=1147941&hide_resolved=1 ) Yes, I'll take a look at this
Assignee | ||
Comment 7•9 years ago
|
||
I don't think this is as big of a deal anymore with Bug 1145784 fixed. I'm not positive if there is still a case where the mouseup event will be skipped when bound to the top level window. Some quick testing shows that the mouseup event fires even when I release it outside of the browser, but I'm not sure if that's dependent on the platform / window manager. I think this will be a quick, easy fix so I'll proceed with a patch.
Depends on: 1145784
Assignee | ||
Comment 8•9 years ago
|
||
Don't use event coordinates from mouseup. Try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=1ee94157fbe4
Attachment #8579478 -
Attachment is obsolete: true
Attachment #8602287 -
Flags: review?(vporof)
Updated•9 years ago
|
Attachment #8602287 -
Flags: review?(vporof) → review+
Comment 9•9 years ago
|
||
Also, thank you Brian for tackling those selection bugs!
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 10•9 years ago
|
||
remote: https://hg.mozilla.org/integration/fx-team/rev/056ceb0ad5ea
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/056ceb0ad5ea
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox40:
--- → fixed
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 40
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•