This might actually be a little more complicated when a lightweight theme is selected. At the moment, lightweight themes cause us to set `-moz-appearance: none; background-color: var(--lwt-accent-color);` on the root element. With those properties, we can't have any vibrancy in the window under the new model. If we could set those properties on the toolbox inset, and leave the window with `-moz-appearance: dialog`, then we can still have vibrant sidebars. But maybe we don't want vibrant sidebars if a theme is selected anyway.
Bug 1592739 Comment 1 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
This might actually be a little more complicated when a lightweight theme is selected. At the moment, lightweight themes cause us to set `-moz-appearance: none; background-color: var(--lwt-accent-color);` on the root element. With those properties, we can't have any vibrancy in the window under the new model. If we could set those properties on the toolbox instead, and leave the window with `-moz-appearance: dialog`, then we can still have vibrant sidebars. But maybe we don't want vibrant sidebars if a theme is selected anyway.