craftcms / cms

Build bespoke content experiences with Craft.
https://craftcms.com
Other
3.23k stars 630 forks source link

Bugfix/15209 disclosure menu and shortcuts #15215

Closed i-just closed 3 months ago

i-just commented 3 months ago

Description

When you open a slideout via a disclosure menu, the order of events is:

When hiding the disclosure menu, ensure we’re removing that menu’s UI layer and not the very top layer, as at that point, the very top layer might be for something else (e.g. a slideout).

Related issues

15209

brandonkelly commented 3 months ago

The same bug could have potentially affected Craft 4, so fixed it there instead. We’re already passing the disclosure menu container into addLayer() so the only change needed was passing the same container to removeLayer().