1373592 - Ensure the resizing architecture for Photon panels can work asynchronously

Status: VERIFIED WORKSFORME

(Firefox :: Toolbars and Customization, enhancement)

Description

[:Paolo Amadini]

I was experimenting with how we can use async functions instead of callback-based functions in panelmultiview, and came across an issue with views initializing asynchronously.

I added a simple "await Promise.resolve();" after the point in PanelMultiView.jsm where we wait for the "blockers" registered by the ViewShowing event.

If I do this, there are glitches when opening the "More" subview of the panel menu, that are more noticeable the first time, but still present afterwards.

This will likely influence subviews that are asynchronous, like the History subview when it's used from the new Library menu in the toolbar, and should be fixed. This is difficult see right now because of an issue with the main view starting too small in the Library menu.

Comment 1

[:Paolo Amadini]

Mike, do you see this too?

Comment 2

[:Gijs (he/him)]

Paolo, I think this is a dupe of https://bugzilla.mozilla.org/show_bug.cgi?id=1370580, see discussion there, particularly comment 7 and later, and https://reviewboard.mozilla.org/r/147596/diff/3-4/ .

Comment 3

[Mike de Boer [:mikedeboer]]

(In reply to :Gijs from comment #2)
> Paolo, I think this is a dupe of
> https://bugzilla.mozilla.org/show_bug.cgi?id=1370580, see discussion there,
> particularly comment 7 and later, and
> https://reviewboard.mozilla.org/r/147596/diff/3-4/ .

Indeed. The updated patch in there should kick things back into gear.

Comment 4

[:Gijs (he/him)]

Is this still an issue, Paolo, or should we dupe this to bug 1370580?

Comment 5

[:Paolo Amadini]

I think it's still an issue for the main panel if you follow the steps in comment 0. In other words, it's a bug in the resizing architecture that may hit us in the future, but does not currently happen. It seems related to views that are smaller than the main view, asynchronous, and is particularly visible the first time they are shown.

Comment 6

[:Gijs (he/him)]

I think this was fixed by the most recent animation changes. Mike or Paolo, can you verify?