fix(ui5-dialog): correctly restore body scrolling on ESC

[dimovpetar]

Body scrolling is now blocked only on first call of blockBodyScrolling and restored on the last call of unblockBodyScrolling.

Fixes #3690

[alexandar-mitsev]

also fix OpenedPopupsRegistry.js, line 38 should be
openedRegistry[openedRegistry.length - 1].instance.close(true);
instead of
openedRegistry.pop().instance.close(true);

this on its own is also fixing the issue, but can be fixed in combination

[dimovpetar]

also fix OpenedPopupsRegistry.js, line 38 should be
openedRegistry[openedRegistry.length - 1].instance.close(true);
instead of
openedRegistry.pop().instance.close(true);

this on its own is also fixing the issue, but can be fixed in combination

You are right, I've changed the OpenedPopupsRegistry to not implicitly remove the popup.
But the logic in Popup.js is still needed, because there might be a popup that is not blocking the scrolling. Therefore, we need to maintain information about those blockers.

[Lukas742]

Hi colleagues,

could you kindly also include this in the next release of the 0.31 sf branch. Thanks :)

[ilhan007]

Hi @Lukas742 yes, we will do that