Closed Bug 1435090 Opened 3 years ago Closed 3 years ago

Hide the 'persist logs' checkbox when in the Browser Console

Categories

(DevTools :: Console, enhancement, P1)

Product:
DevTools
Component:
Console
Type:
enhancement

Tracking

(firefox60 fixed)

Status:
RESOLVED FIXED
Milestone:
Firefox 60
Tracking Flags:
Tracking Status
firefox60 --- fixed

People

(Reporter: bgrins, Assigned: nchevobbe)

References

Details

(Whiteboard: [newconsole-mvp])

Attachments

(3 files)

Bug 1435090 - Allow to pass a hidePersistLogsCheckbox prop to the FilterBar; .
59 bytes, text/x-review-board-request
Honza
: review+
Details
Bug 1435090 - Hide the persist logs checkbox on the browser console; .
59 bytes, text/x-review-board-request
Honza
: review+
Details
Screenshot at Feb 08 15-32-11.png
235.67 KB, image/png
Details 
There's no reloads happening in the Browser Console, so the checkbox doesn't do anything
Depends on: 1435084
Whiteboard: [newconsole-mvp]
There's a patch with the basic idea in https://reviewboard.mozilla.org/r/217368/diff/1#index_header, but it needs tests
Priority: -- → P2
Assignee: nobody → nchevobbe
Status: NEW → ASSIGNED
Priority: P2 → P1
Here's how it looks (I really like having the new frontend in the browser console :) )

Not sure we should add a mochitest for that since it's pretty simple, and we do have a mocha test for the filterBar rendering.
Comment on attachment 8949366 [details]
Bug 1435090 - Allow to pass a hidePersistLogsCheckbox prop to the FilterBar; .

https://reviewboard.mozilla.org/r/218718/#review224540

Looks good, thanks!

R+, assuming try is green

Honza

::: devtools/client/webconsole/new-console-output/components/FilterBar.js:224
(Diff revision 1)
>      const {
>        filter,
>        filterBarVisible,
>        persistLogs,
>        filteredMessagesCount,
> +      hidePersistLogsCheckbox = false,

nit: you might want to use defaultProps, so it's consistent with other components.

static get defaultProps() {
  return {
    hidePersistLogsCheckbox: false,
  };
}
Attachment #8949366 - Flags: review?(odvarko) → review+
Comment on attachment 8949367 [details]
Bug 1435090 - Hide the persist logs checkbox on the browser console; .

https://reviewboard.mozilla.org/r/218720/#review224542

R+

Honza
Attachment #8949367 - Flags: review?(odvarko) → review+
Pushed by nchevobbe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/28bcec297b6c
Allow to pass a hidePersistLogsCheckbox prop to the FilterBar; r=Honza.
https://hg.mozilla.org/integration/autoland/rev/dfd0afe71bb5
Hide the persist logs checkbox on the browser console; r=Honza.
https://hg.mozilla.org/mozilla-central/rev/28bcec297b6c
https://hg.mozilla.org/mozilla-central/rev/dfd0afe71bb5
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
status-firefox60: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 60
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.