@brightspace-ui/core 3.331.1 → 3.331.3

package/components/dialog/dialog-mixin.js

@@ -104,7 +104,7 @@ export const DialogMixin = superclass => class extends superclass {
 		window.removeEventListener('d2l-mvc-dialog-open', this.#handleMvcDialogOpen);
 
 		// If the dialog is disconnected before the close animation finishes
-		if (this.opened) {
+		if (!this.#closeHandled) {
 			this._handleClose();
 		}
 	}
@@ -168,6 +168,7 @@ export const DialogMixin = superclass => class extends superclass {
 
 	#useNativeInitialized = false;
 	#ifrauDialogService = null;
+	#closeHandled = true;
 
 	#handleMvcDialogOpen = () => {
 		// native dialogs on top layer will be stacked on non-native dialogs regardless of z-index
@@ -391,6 +392,7 @@ export const DialogMixin = superclass => class extends superclass {
 				detail: { action: this._action }
 			}
 		));
+		this.#closeHandled = true;
 	}
 
 	_handleDialogCloseWithinNative(e) {
@@ -449,6 +451,7 @@ export const DialogMixin = superclass => class extends superclass {
 
 	async _open() {
 		if (!this.opened) return;
+		this.#closeHandled = false;
 
 		this._opener = getComposedActiveElement();
 		this._dismissibleId = setDismissible(() => {

package/components/toolbar/toolbar-dropdown.js

@@ -90,19 +90,18 @@ class ToolbarDropdown extends FocusMixin(DropdownOpenerMixin(SlottedIconMixin(To
 		return html`
 			<button
 				aria-disabled="${this.disabled ? 'true' : 'false'}"
-				aria-label="${this.text}"
 				id="opener"
 				@keydown="${this.#handleKeyDown}"
 				tabindex="${this._activeFocusable ? 0 : -1}"
 				type="button">
 				<div class="background"></div>
 				${this._renderIcon()}
-				${this.valueText ? html`<div class="value-text">${this.valueText}</div>` : nothing}
+				${this.valueText ? html`<div aria-hidden="true" class="value-text">${this.valueText}</div>` : nothing}
 				<d2l-icon icon="tier1:chevron-down-small"></d2l-icon>
 			</button>
-			<d2l-tooltip class="${classMap(tooltipClasses)}" for="opener">
-				<span aria-hidden="true">${this.text}</span>
-				<span class="d2l-offscreen">${this.valueText}</span>
+			<d2l-tooltip class="${classMap(tooltipClasses)}" for="opener" for-type="label">
+				<span>${this.text}</span>
+				<d2l-offscreen>${this.valueText}</d2l-offscreen>
 			</d2l-tooltip>
 			<slot></slot>
 		`;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@brightspace-ui/core",
-  "version": "3.331.1",
+  "version": "3.331.3",
   "description": "A collection of accessible, free, open-source web components for building Brightspace applications",
   "type": "module",
   "repository": "https://github.com/BrightspaceUI/core.git",