RESOLVED FIXED147094
Web Inspector: wrong cursor shown when selecting time interval from timeline overview
https://bugs.webkit.org/show_bug.cgi?id=147094
Summary Web Inspector: wrong cursor shown when selecting time interval from timeline ...
Brian Burg
Reported 2015-07-19 14:48:47 PDT
STEPS TO REPRODUCE: 1. Recording some stuff in a timeline 2. Mouse down in the bars area 3. Mouse move to select an interval 4. Mouse up to finish selecting For the entirety of step 3, the cursor should change to the vertical resizer >|<. It does this correctly for the grabbers, but not when directly selecting an interval.
Attachments
[Patch] Proposed Fix (5.24 KB, patch)
2015-07-22 19:39 PDT, Matt Baker
no flags
[Patch] Proposed Fix (5.17 KB, patch)
2015-07-22 21:46 PDT, Matt Baker
no flags
Radar WebKit Bug Importer
Comment 1 2015-07-19 14:48:58 PDT
Matt Baker
Comment 2 2015-07-22 19:39:36 PDT
Created attachment 257325 [details] [Patch] Proposed Fix
Timothy Hatcher
Comment 3 2015-07-22 20:46:28 PDT
Comment on attachment 257325 [details] [Patch] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=257325&action=review > Source/WebInspectorUI/UserInterface/Views/TimelineRuler.js:718 > + if (!this._element.classList.contains(WebInspector.TimelineRuler.CreatingSelectionStyleClassName)) You can just call add. It won't add it multiple times.
Matt Baker
Comment 4 2015-07-22 21:46:47 PDT
Created attachment 257336 [details] [Patch] Proposed Fix
WebKit Commit Bot
Comment 5 2015-07-23 07:01:21 PDT
Comment on attachment 257336 [details] [Patch] Proposed Fix Clearing flags on attachment: 257336 Committed r187227: <http://trac.webkit.org/changeset/187227>
WebKit Commit Bot
Comment 6 2015-07-23 07:01:25 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.