@brightspace-ui/core 3.75.1 → 3.75.3

package/components/dialog/dialog.js

@@ -35,7 +35,7 @@ class Dialog extends PropertyRequiredMixin(LocalizeCoreElement(AsyncContainerMix
 			critical: { type: Boolean },
 
 			/**
-			 * Whether to read the contents of the dialog on open
+			 * Causes screen readers to announce the content of the dialog on open. Only use if the content is concise and contains only text since screen readers ignore HTML semantics and some have a ~250 character limit.
 			 */
 			describeContent: { type: Boolean, attribute: 'describe-content' },
 

package/components/dropdown/dropdown-content-mixin.js

@@ -628,7 +628,7 @@ export const DropdownContentMixin = superclass => class extends LocalizeCoreElem
 
 		// DE44538: wait for dropdown content to fully render,
 		// otherwise this.getContentContainer() can return null.
-		await this.updateComplete;
+		await this.__waitForContentContainer();
 
 		this.__previousFocusableAncestor =
 			newValue === true
@@ -849,6 +849,12 @@ export const DropdownContentMixin = superclass => class extends LocalizeCoreElem
 		this._topOverflow = this.__content.scrollTop !== 0;
 	}
 
+	async __waitForContentContainer() {
+		if (this.getContentContainer() !== null) return;
+		await new Promise(resolve => requestAnimationFrame(resolve));
+		return this.__waitForContentContainer();
+	}
+
 	_constrainSpaceAround(spaceAround, spaceRequired, targetRect) {
 		const constrained = { ...spaceAround };
 		if (this.boundary) {

package/components/filter/filter.js

@@ -49,11 +49,11 @@ function addSpaceListener() {
 	if (spaceListenerAdded) return;
 	spaceListenerAdded = true;
 	document.addEventListener('keydown', e => {
-		if (e.keyCode !== 32) return;
+		if (e.key !== ' ') return;
 		spacePressed = true;
 	});
 	document.addEventListener('keyup', e => {
-		if (e.keyCode !== 32) return;
+		if (e.key !== ' ') return;
 		spacePressed = false;
 	});
 }
@@ -244,6 +244,7 @@ class Filter extends FocusMixin(LocalizeCoreElement(RtlMixin(LitElement))) {
 			onSubscribe: this._updateActiveFiltersSubscriber.bind(this),
 			updateSubscribers: this._updateActiveFiltersSubscribers.bind(this)
 		});
+		this._spacePressedDuringLastSelection = false;
 	}
 
 	static get focusElementSelector() {
@@ -617,7 +618,7 @@ class Filter extends FocusMixin(LocalizeCoreElement(RtlMixin(LitElement))) {
 		return html`
 			<d2l-list-item
 				id="${itemId}"
-				@d2l-list-item-selected="${ifDefined(item.additionalContent ? this._handleListItemSelelcted : undefined)}"
+				@d2l-list-item-selected="${item.additionalContent ? this._handleListItemSelected : undefined}"
 				?selection-disabled="${item.disabled}"
 				?hidden="${item.hidden}"
 				key="${item.key}"
@@ -896,18 +897,23 @@ class Filter extends FocusMixin(LocalizeCoreElement(RtlMixin(LitElement))) {
 	}
 
 	async _handleExpandCollapse(e) {
-		const eventPromise = e.target.expanded ? e.detail.expandComplete : e.detail.collapseComplete;
+		const expanded = e.target.expanded;
+		const eventPromise = expanded ? e.detail.expandComplete : e.detail.collapseComplete;
 		const parentListItem = e.target.closest('d2l-list-item');
 		parentListItem.classList.add('expanding-content');
 
 		await eventPromise;
 		parentListItem.classList.remove('expanding-content');
+
+		if (expanded && !hasDisplayedKeyboardTooltip && this._spacePressedDuringLastSelection) {
+			await new Promise(resolve => requestAnimationFrame(resolve));
+			this._displayKeyboardTooltip = true;
+			hasDisplayedKeyboardTooltip = true;
+		}
 	}
 
-	_handleListItemSelelcted() {
-		if (hasDisplayedKeyboardTooltip || !spacePressed) return;
-		this._displayKeyboardTooltip = true;
-		hasDisplayedKeyboardTooltip = true;
+	_handleListItemSelected() {
+		this._spacePressedDuringLastSelection = spacePressed;
 	}
 
 	_handleSearch(e) {

package/components/inputs/input-time.js

@@ -461,9 +461,17 @@ class InputTime extends InputInlineHelpMixin(FocusMixin(LabelledMixin(SkeletonMi
 		// open and focus dropdown on down arrow or enter
 		if (e.keyCode === 40 || e.keyCode === 13) {
 			if (!this._dropdownFirstOpened) this._dropdownFirstOpened = true;
-			this.opened = true;
 			e.preventDefault();
+			await this._waitForItems();
+			this.opened = true;
 		}
 	}
+
+	async _waitForItems() {
+		const items = this.shadowRoot.querySelectorAll('d2l-menu-item-radio');
+		if (items.length > 0) return;
+		await new Promise(resolve => requestAnimationFrame(resolve));
+		return this._waitForItems();
+	}
 }
 customElements.define('d2l-input-time', InputTime);

package/custom-elements.json

@@ -2229,7 +2229,7 @@
         },
         {
           "name": "describe-content",
-          "description": "Whether to read the contents of the dialog on open",
+          "description": "Causes screen readers to announce the content of the dialog on open. Only use if the content is concise and contains only text since screen readers ignore HTML semantics and some have a ~250 character limit.",
           "type": "boolean",
           "default": "false"
         },
@@ -2275,7 +2275,7 @@
         {
           "name": "describeContent",
           "attribute": "describe-content",
-          "description": "Whether to read the contents of the dialog on open",
+          "description": "Causes screen readers to announce the content of the dialog on open. Only use if the content is concise and contains only text since screen readers ignore HTML semantics and some have a ~250 character limit.",
           "type": "boolean",
           "default": "false"
         },

package/lang/fr.js

@@ -120,7 +120,7 @@ export default {
 	"components.selection-controls.label": "Actions à sélectionner",
 	"components.switch.visible": "Visible",
 	"components.switch.visibleWithPeriod": "Visible.",
-	"components.switch.hidden": "Masqué(e)",
+	"components.switch.hidden": "Masqué",
 	"components.switch.conditions": "Les conditions doivent être remplies",
 	"components.table-col-sort-button.addSortOrder": "Sélectionner pour ajouter un ordre de tri",
 	"components.table-col-sort-button.changeSortOrder": "Sélectionner pour modifier l'ordre de tri",

package/lang/hi.js

@@ -43,7 +43,7 @@ export default {
 	"components.form-element.input.number.rangeUnderflow": "{minExclusive, select, true {संख्या {min} से ज़्यादा होनी चाहिए.} other {संख्या {min} से ज़्यादा या उसके बराबर होनी चाहिए.}}",
 	"components.form-element.input.text.tooShort": "{label} कम से कम {minlength} वर्णों का होना चाहिए",
 	"components.form-element.input.url.typeMismatch": "URL मान्य नहीं है",
-	"components.form-element.valueMissing": "{label} ज़रूरी है",
+	"components.form-element.valueMissing": "{label} ज़रूरी है",
 	"components.form-error-summary.errorSummary": "{count, plural, one {आपके द्वारा सबमिट की गई जानकारी में {count} त्रुटियाँ पाई गईं} other {आपके द्वारा सबमिट की गई जानकारी में {count} त्रुटियाँ पाई गईं}}",
 	"components.form-error-summary.text": "त्रुटि विवरण टॉगल करें",
 	"components.input-color.backgroundColor": "पृष्ठभूमि का रंग",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@brightspace-ui/core",
-  "version": "3.75.1",
+  "version": "3.75.3",
   "description": "A collection of accessible, free, open-source web components for building Brightspace applications",
   "type": "module",
   "repository": "https://github.com/BrightspaceUI/core.git",
@@ -47,16 +47,13 @@
   "devDependencies": {
     "@brightspace-ui/stylelint-config": "^1",
     "@brightspace-ui/testing": "^1",
-    "@eslint/compat": "^1",
-    "@eslint/eslintrc": "^3",
-    "@eslint/js": "^9",
     "@rollup/plugin-dynamic-import-vars": "^2",
     "@rollup/plugin-node-resolve": "^15",
     "@rollup/plugin-replace": "^6",
     "@web/dev-server": "^0.4",
     "chalk": "^5",
     "eslint": "^9",
-    "eslint-config-brightspace": "^1.2.1",
+    "eslint-config-brightspace": "^2.0.0",
     "glob-all": "^3",
     "messageformat-validator": "^2",
     "rollup": "^4",