@brightspace-ui/core 2.131.0 → 2.131.1

package/components/calendar/calendar.js

@@ -463,8 +463,7 @@ class Calendar extends LocalizeCoreElement(RtlMixin(LitElement)) {
 				const month = day.getMonth();
 				const date = day.getDate();
 				const weekday = calendarData.descriptor.calendar.days.long[calendarData.daysOfWeekIndex[index]];
-				const description = `${weekday} ${date}. ${selected ? this.localize(`${this._namespace}.selected`) : this.localize(`${this._namespace}.notSelected`)} ${formatDate(day, { format: 'monthYear' })}`;
-				// role="gridcell" used for screen reader (e.g., JAWS and VoiceOver) behavior to work properly
+				const description = `${weekday} ${date} ${formatDate(day, { format: 'monthYear' })}`;
 				return html`
 					<td
 						aria-selected="${selected ? 'true' : 'false'}"
@@ -519,7 +518,7 @@ class Calendar extends LocalizeCoreElement(RtlMixin(LitElement)) {
 								icon="tier1:chevron-right">
 							</d2l-button-icon>
 						</div>
-						<table aria-labelledby="${labelId}" role="presentation">
+						<table aria-labelledby="${labelId}">
 							${summary}
 							<thead aria-hidden="true">
 								<tr>${weekdayHeaders}</tr>

package/components/more-less/more-less.js

@@ -11,7 +11,6 @@ import { styleMap } from 'lit/directives/style-map.js';
 /**
  * A component used to minimize the display of long content, while providing a way to reveal the full content.
  * @slot - Default content placed inside of the component
- * @fires d2l-more-less-render - Dispatched when the component finishes rendering
  */
 class MoreLess extends LocalizeCoreElement(LitElement) {
 
@@ -147,12 +146,6 @@ class MoreLess extends LocalizeCoreElement(LitElement) {
 	}
 
 	render() {
-		requestAnimationFrame(
-			() => this.dispatchEvent(new CustomEvent('d2l-more-less-render', {
-				bubbles: false,
-				composed: false
-			}))
-		);
 		const contentClasses = {
 			'd2l-more-less-content': true,
 			'd2l-more-less-transition': this.__transitionAdded

package/custom-elements.json

@@ -9875,12 +9875,6 @@
           "default": "\"getDocumentLocaleSettings()\""
         }
       ],
-      "events": [
-        {
-          "name": "d2l-more-less-render",
-          "description": "Dispatched when the component finishes rendering"
-        }
-      ],
       "slots": [
         {
           "name": "",

package/mixins/labelled/labelled-mixin.js

@@ -126,6 +126,16 @@ export const LabelledMixin = superclass => class extends superclass {
 
 	}
 
+	_dispatchChangeEvent() {
+		/** @ignore */
+		this.dispatchEvent(new CustomEvent(
+			'd2l-labelled-mixin-label-change', {
+				bubbles: false,
+				composed: false
+			}
+		));
+	}
+
 	_throwError(err) {
 		if (!this.labelRequired || this._missingLabelErrorHasBeenThrown) return;
 		this._missingLabelErrorHasBeenThrown = true;
@@ -134,9 +144,14 @@ export const LabelledMixin = superclass => class extends superclass {
 
 	_updateLabelElem(labelElem) {
 
+		const oldLabelVal = this.label;
+
 		// setting textContent doesn't change labelElem but we do need to refetch the label
 		if (labelElem === this._labelElem && this._labelElem) {
 			this.label = getLabel(this._labelElem);
+			if (oldLabelVal !== this.label) {
+				this._dispatchChangeEvent();
+			}
 			return;
 		}
 
@@ -180,13 +195,9 @@ export const LabelledMixin = superclass => class extends superclass {
 		}
 
 		this.label = getLabel(this._labelElem);
-		/** @ignore */
-		this.dispatchEvent(new CustomEvent(
-			'd2l-labelled-mixin-label-elem-change', {
-				bubbles: false,
-				composed: false
-			}
-		));
+		if (oldLabelVal !== this.label) {
+			this._dispatchChangeEvent();
+		}
 
 	}
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@brightspace-ui/core",
-  "version": "2.131.0",
+  "version": "2.131.1",
   "description": "A collection of accessible, free, open-source web components for building Brightspace applications",
   "type": "module",
   "repository": "https://github.com/BrightspaceUI/core.git",
@@ -47,7 +47,6 @@
     "@brightspace-ui/stylelint-config": "^0.8",
     "@brightspace-ui/testing": "^0.9",
     "@open-wc/semantic-dom-diff": "^0.20",
-    "@open-wc/testing": "^3",
     "@rollup/plugin-dynamic-import-vars": "^2",
     "@rollup/plugin-node-resolve": "^15",
     "@rollup/plugin-replace": "^5",

package/tools/constructor-test-helper.js

@@ -1,4 +1,4 @@
-import { expect } from '@open-wc/testing';
+import { expect } from '@brightspace-ui/testing';
 
 export function runConstructor(nodeName) {
 	const ctor = customElements.get(nodeName);