Fix: Perform HTMLDialogElement.close on drawer before animating closed (fixes #628)

[oliverfoster]

Additional fixes to pr #601

fixes #628

When plp navigates to a different page it causes both the drawer to close and the pip notify to open. The drawer dialog should be closed before the notify is open such that the drawer close doesn't accidentally close the new notify at

adapt-contrib-core/js/a11y/popup.js

Lines 168 to 176 in 230a2fa

	_removeLastPopupLayer() {
	// the body layer is the first element and must always exist
	if (this._floorStack.length <= 1) {
	return;
	}
	const $popupElement = this._floorStack.pop();
	if ($popupElement.is('dialog')) {
	$popupElement[0].close();
	return this._focusStack.pop();

Reverting to previous behaviour in v6.60.2

adapt-contrib-core/js/views/drawerView.js

Lines 194 to 198 in a1b6c57

	hideDrawer($toElement) {
	if (!this._isVisible) return;
	this._useMenuPosition = false;
	const direction = {};
	a11y.popupClosed($toElement);

Fix

• Perform HTMLDialogElement.close on drawer before animating closed

[cahirodoherty-learningpool]

👀