Summary: | Web Inspector: Timelines - Improve handling of past recordings (readonly) | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Joseph Pecoraro <joepeck> | ||||
Component: | Web Inspector | Assignee: | Joseph Pecoraro <joepeck> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | commit-queue, hi, inspector-bugzilla-changes, joepeck, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | All | ||||||
OS: | All | ||||||
Attachments: |
|
Description
Joseph Pecoraro
2019-03-11 18:20:41 PDT
Created attachment 364330 [details]
[PATCH] Proposed Fix
Comment on attachment 364330 [details] [PATCH] Proposed Fix View in context: https://bugs.webkit.org/attachment.cgi?id=364330&action=review rs=me > Source/WebInspectorUI/UserInterface/Views/TimelineTabContentView.js:487 > + if (WI.timelineManager.activeRecording.readonly) > + shouldCreateNewRecording = true; Should we also change the tooltip of the record button to remove the non-`shouldCreateNewRecording` text? e.g. `"Start recording (%s)\nCreate new recording (%s)"` becomes `"Create new recording (%s)"` when the active recording is readonly (and vice versa). (In reply to Devin Rousso from comment #2) > Comment on attachment 364330 [details] > [PATCH] Proposed Fix > > View in context: > https://bugs.webkit.org/attachment.cgi?id=364330&action=review > > rs=me > > > Source/WebInspectorUI/UserInterface/Views/TimelineTabContentView.js:487 > > + if (WI.timelineManager.activeRecording.readonly) > > + shouldCreateNewRecording = true; > > Should we also change the tooltip of the record button to remove the > non-`shouldCreateNewRecording` text? e.g. `"Start recording (%s)\nCreate > new recording (%s)"` becomes `"Create new recording (%s)"` when the active > recording is readonly (and vice versa). Naw, I don't think this is common enough to warrant it. I think the current tooltip is still reasonably accurate even in the readonly case, it will start a recording! Comment on attachment 364330 [details] [PATCH] Proposed Fix Clearing flags on attachment: 364330 Committed r242785: <https://trac.webkit.org/changeset/242785> All reviewed patches have been landed. Closing bug. |