1014229 - Stop using a panelmultiview when displaying subviews as standalone panels

Status: NEW

(Firefox :: Toolbars and Customization, defect)

Description

[:Gijs (he/him)]

We shouldn't really need to have a panelmultiview for this case at all; we never show any other subview in the panel.

Mike, do you remember why we did it this way? Why aren't we just sticking the panelview into a <panel> directly? :-)

Comment 1

[Mike Conley (:mconley) (:⚙️)]

(In reply to :Gijs Kruitbosch from comment #0)
> We shouldn't really need to have a panelmultiview for this case at all; we
> never show any other subview in the panel.
> 
> Mike, do you remember why we did it this way? Why aren't we just sticking
> the panelview into a <panel> directly? :-)

I think because at least at one point, the bookmarks menu was going to work like this:

If the bookmarks menu subview is opened in the menu panel, clicking folders causes us to open the Library.
If the bookmarks menu subview is opened on its own, clicking folders should cause us to open a subview with the folder contents.

That plan eventually got scrapped - so I think what we're seeing here is a fragment of a somewhat baked idea that never shipped, or was bailed out on. We're good to chuck the fragment out.