"Show Content Messages" checkbox is reinitialized in the Omniscient Browser Toolbox after every restart
Categories
(DevTools :: Console, task, P3)
Tracking
(firefox73 fixed)
Tracking | Status | |
---|---|---|
firefox73 | --- | fixed |
People
(Reporter: jdescottes, Assigned: jdescottes)
References
(Blocks 1 open bug)
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
text/x-phabricator-request
|
Details |
STRs:
- open OBT, open console
- in the Console cog options, disable "Show Content Messages"
- close the OBT
- open OBT, open console
Expected Result: The setting should be persisted, "Show Content Messages" should be disabled
Actual Result: The setting is enabled again.
Note: this only applies to this setting. Timestamp and grouping options are correctly persisted across restarts.
Assignee | ||
Comment 1•5 years ago
|
||
Probably from this code that initializes the state of the console
showContentMessages: webConsoleUI.isBrowserConsole
? getBoolPref(PREFS.UI.CONTENT_MESSAGES)
: true,
Assignee | ||
Comment 2•5 years ago
|
||
To be clear this should be updated to check webConsoleUI.isBrowserConsole || isBrowserToolbox
(the catch is that we don't have a good isBrowserToolbox
helper available I think)
Assignee | ||
Comment 3•5 years ago
|
||
Actually the console has webConsoleUI.isBrowserToolboxConsole
which should be perfect here!
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 4•5 years ago
|
||
Nicolas, what do you think should be the default behavior of the Browser Toolbox Console? Should it show content messages by default or not?
Today, with the current code, the setting is always true on startup. If we change it to have the same logic as the Browser Console, then by default content messages will be hidden.
We can either:
- keep it disabled by default and update browser_toolbox_console_new_process.js to set the pref in the BT profile
- update the default value of devtools.browserconsole.contentMessages to be true (will then change the default behavior of the Browser Console)
- default it to true only for the Browser Toolbox Console (would be consistent with the current situation, but might be hard to justify)
Assignee | ||
Comment 5•5 years ago
|
||
Assignee | ||
Comment 6•5 years ago
|
||
Updated•5 years ago
|
Comment 7•5 years ago
|
||
I'd say let's keep it disabled to match the Browser Console logic.
Assignee | ||
Comment 8•5 years ago
|
||
sounds good, thanks!
Updated•5 years ago
|
Comment 10•5 years ago
|
||
bugherder |
Description
•