Bug 1876198 Comment 0 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Here's the [Figma link](https://www.figma.com/file/JhKhHqJJ9bjzU1TITUpiWi/Review-Checker---Experiments?type=design&node-id=762-23740&mode=design&t=PPTe8L9lpC0glxgE-4) (screenshots to be added)

Notes:
* The auto-open toggle should go below the ads toggle
* also need to default the settings card to be open, not closed, by default
* if the auto-open toggle is flipped from true to false, we need to immediately close the current sidebar and flip the browser.shopping.experience2023.autoOpen pref

We might be able to replace the existing pref listener that calls ShoppingSidebarParent.updateSidebarVisibility when the 'active' pref is updated, to one that calls the same function when the 'autoOpen' pref is updated. This may touch on the big bug, so ask whoever's working on that.
Here's the [Figma link](https://www.figma.com/file/JhKhHqJJ9bjzU1TITUpiWi/Review-Checker---Experiments?type=design&node-id=762-23740&mode=design&t=PPTe8L9lpC0glxgE-4) (screenshots to be added)

Notes:
* The auto-open toggle should go below the ads toggle
* also need to default the settings card to be open, not closed, by default
* if the auto-open toggle is flipped from true to false, we need to immediately close the current sidebar and flip the browser.shopping.experience2023.autoOpen pref
* because we are past string freeze for 123, we will need to land the new strings in the preview location for shopping, then land a followup patch in a followup bug to move the strings to the permanent location in 124 nightly.

We might be able to replace the existing pref listener that calls ShoppingSidebarParent.updateSidebarVisibility when the 'active' pref is updated, to one that calls the same function when the 'autoOpen' pref is updated. This may touch on the big bug, so ask whoever's working on that.
Here's the [Figma link](https://www.figma.com/file/JhKhHqJJ9bjzU1TITUpiWi/Review-Checker---Experiments?type=design&node-id=762-23740&mode=design&t=PPTe8L9lpC0glxgE-4) (screenshots to be added)

Notes:
* The auto-open toggle should go below the ads toggle
* also need to default the settings card to be open, not closed, by default
* if the auto-open toggle is flipped from true to false, we need to immediately close the current sidebar and flip the browser.shopping.experience2023.autoOpen pref
* because we are past string freeze for 123, we will need to land the new strings in the preview location for shopping, then land a followup patch in a followup bug to move the strings to the permanent location in 124 nightly.
Here's the [Figma link](https://www.figma.com/file/JhKhHqJJ9bjzU1TITUpiWi/Review-Checker---Experiments?type=design&node-id=762-23740&mode=design&t=PPTe8L9lpC0glxgE-4) (screenshots to be added)

Notes:
* The auto-open toggle should go below the ads toggle
* also need to default the settings card to be open, not closed, by default
* if the auto-open toggle is flipped from true to false, we should flip the browser.shopping.experience2023.autoOpen pref and also flip the 'active' pref to false
* because we are past string freeze for 123, we will need to land the new strings in the preview location for shopping, then land a followup patch in a followup bug to move the strings to the permanent location in 124 nightly.

Back to Bug 1876198 Comment 0