@brightspace-ui/core 2.0.4 → 2.1.2

package/components/dialog/dialog-mixin.js

@@ -124,7 +124,12 @@ export const DialogMixin = superclass => class extends RtlMixin(superclass) {
 	}
 
 	resize() {
-		return this._updateSize();
+		return new Promise(resolve => {
+			setTimeout(async() => {
+				await this._updateSize();
+				resolve();
+			}, 0);
+		});
 	}
 
 	_addHandlers() {
@@ -330,7 +335,7 @@ export const DialogMixin = superclass => class extends RtlMixin(superclass) {
 		// focus first focusable child prior to auto resize (fixes screen reader hiccups)
 		this._focusInitial();
 
-		requestAnimationFrame(async() => {
+		setTimeout(async() => {
 
 			this.shadowRoot.querySelector('.d2l-dialog-content').scrollTop = 0;
 			// scrollbar is kept hidden while we update the scroll position to avoid scrollbar flash
@@ -340,6 +345,7 @@ export const DialogMixin = superclass => class extends RtlMixin(superclass) {
 
 			await this._updateSize();
 			this._state = 'showing';
+			await this._updateComplete;
 
 			// edge case: no children were focused, try again after one redraw
 			const activeElement = getComposedActiveElement();
@@ -354,7 +360,7 @@ export const DialogMixin = superclass => class extends RtlMixin(superclass) {
 			this.dispatchEvent(new CustomEvent(
 				'd2l-dialog-open', { bubbles: true, composed: true }
 			));
-		});
+		}, 0);
 
 	}
 

package/components/dialog/dialog-styles.js

@@ -11,6 +11,15 @@ export const dialogStyles = css`
 		display: block;
 	}
 
+	:host([opened]:not([_state="showing"])) {
+		visibility: hidden;
+	}
+
+	:host([opened][_state="showing"]),
+	:host([opened][_state="hiding"]) {
+		visibility: visible;
+	}
+
 	.d2l-dialog-outer {
 		background-color: white;
 		border: 1px solid var(--d2l-color-mica);

package/components/tabs/tab-panel-mixin.js

@@ -71,6 +71,7 @@ export const TabPanelMixin = superclass => class extends superclass {
 				}
 			} else if (prop === 'text') {
 				this.setAttribute('aria-label', this.text);
+				/** Dispatched when the text attribute is changed */
 				this.dispatchEvent(new CustomEvent(
 					'd2l-tab-panel-text-changed', { bubbles: true, composed: true, detail: { text: this.text } }
 				));

package/custom-elements.json

@@ -9707,7 +9707,8 @@
           "description": "Dispatched when a tab is selected"
         },
         {
-          "name": "d2l-tab-panel-text-changed"
+          "name": "d2l-tab-panel-text-changed",
+          "description": "Dispatched when the text attribute is changed"
         }
       ],
       "slots": [

package/mixins/localize-dynamic-mixin.js

@@ -2,11 +2,17 @@ import { getLocalizeOverrideResources } from '../helpers/getLocalizeResources.js
 import { LocalizeMixin } from './localize-mixin.js';
 
 const fallbackLang = 'en';
+const supportedLangpacks = ['ar', 'cy', 'da', 'de', 'en', 'es', 'es-es', 'fr', 'fr-fr', 'fr-on', 'hi', 'ja', 'ko', 'nl', 'pt', 'sv', 'tr', 'zh-cn', 'zh-tw'];
 
 export const LocalizeDynamicMixin = superclass => class extends LocalizeMixin(superclass) {
 
 	static async getLocalizeResources(langs, { importFunc, osloCollection }) {
 
+		// in dev, don't request unsupported langpacks
+		if (!importFunc.toString().includes('switch')) {
+			langs = langs.filter(lang => supportedLangpacks.includes(lang));
+		}
+
 		for (const lang of [...langs, fallbackLang]) {
 
 			const resources = await importFunc(lang).catch(() => {});

package/mixins/localize-mixin.md

@@ -83,7 +83,9 @@ class MyComponent extends LocalizeDynamicMixin(LitElement) {
 }
 ```
 
-If your build system does not support variable dynamic imports, you'll need to manually set up imports for each language:
+When using this method, depending on various user settings, it's possible that a language file that does not exist will be requested, resulting in a network error (404). In production, your build system should prevent this by transpiling the variable dynamic import into a `switch` statement.
+
+If your build system does not support variable dynamic imports, you'll need to manually set up imports for each supported language:
 
 ```javascript
 static get localizeConfig() {

package/package.json

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