@brightspace-ui/core 3.62.4 → 3.63.0

package/components/dialog/dialog-mixin.js

@@ -19,8 +19,7 @@ window.D2L.DialogMixin = window.D2L.DialogMixin || {};
 // https://bugs.webkit.org/show_bug.cgi?id=233320
 // starting in Chrome 102, all elements inside <dialog>s that are inside shadow roots have null offsetParent
 // https://bugs.chromium.org/p/chromium/issues/detail?id=1331803
-window.D2L.DialogMixin.hasNative = false && (window.HTMLDialogElement !== undefined)
-	&& (navigator.vendor && navigator.vendor.toLowerCase().indexOf('apple') === -1);
+window.D2L.DialogMixin.hasNative = false;
 if (window.D2L.DialogMixin.preferNative === undefined) {
 	window.D2L.DialogMixin.preferNative = true;
 }

package/components/filter/demo/filter-tags.html

@@ -77,14 +77,12 @@
 				coreFilter.addEventListener('d2l-filter-change', (e) => {
 					logEvent(e, 'Filter value(s) changed!');
 				});
-				/* eslint-disable no-console */
 				function logEvent(e, text) {
 					console.group(text);
 					console.log('event', e);
 					if (e.detail) console.log('detail', e.detail);
 					console.groupEnd();
 				}
-				/* eslint-enable no-console */
 			</script>
 
 			<h2>Registered to both filters</h2>

package/components/filter/demo/filter.html

@@ -191,14 +191,14 @@
 	<script type="module">
 		document.addEventListener('d2l-filter-change', e => {
 			if (e.detail.dimensions.length === 1) {
-				console.log(`Filter selection(s) changed for dimension "${e.detail.dimensions[0].dimensionKey}":`, e.detail.dimensions[0].changes); // eslint-disable-line no-console
-				if (e.detail.dimensions[0].cleared) console.log(`(Dimension "${e.detail.dimensions[0].dimensionKey}" cleared)`); // eslint-disable-line no-console
+				console.log(`Filter selection(s) changed for dimension "${e.detail.dimensions[0].dimensionKey}":`, e.detail.dimensions[0].changes);
+				if (e.detail.dimensions[0].cleared) console.log(`(Dimension "${e.detail.dimensions[0].dimensionKey}" cleared)`);
 			} else {
-				console.log('Multiple dimension selections changed:', e.detail.dimensions); // eslint-disable-line no-console
+				console.log('Multiple dimension selections changed:', e.detail.dimensions);
 			}
 
 			if (e.detail.allCleared) {
-				console.log('(All dimensions cleared)'); // eslint-disable-line no-console
+				console.log('(All dimensions cleared)');
 			}
 		});
 

package/components/inputs/demo/input-search.html

@@ -66,7 +66,6 @@
 
 			<script>
 				document.body.addEventListener('d2l-input-search-searched', (e) => {
-					// eslint-disable-next-line no-console
 					console.log(e);
 				});
 			</script>

package/components/inputs/input-date-time.js

@@ -143,7 +143,7 @@ class InputDateTime extends FocusMixin(LabelledMixin(SkeletonMixin(FormElementMi
 				const dateObj = parseISODateTime(val);
 				const localDateTime = convertUTCToLocalDateTime(dateObj);
 				this._maxValueLocalized = formatDateInISO(localDateTime);
-			} catch (e) {
+			} catch {
 				this._maxValueLocalized = undefined;
 			}
 		}
@@ -160,7 +160,7 @@ class InputDateTime extends FocusMixin(LabelledMixin(SkeletonMixin(FormElementMi
 				const dateObj = parseISODateTime(val);
 				const localDateTime = convertUTCToLocalDateTime(dateObj);
 				this._minValueLocalized = formatDateInISO(localDateTime);
-			} catch (e) {
+			} catch {
 				this._minValueLocalized = undefined;
 			}
 		}
@@ -176,7 +176,7 @@ class InputDateTime extends FocusMixin(LabelledMixin(SkeletonMixin(FormElementMi
 			try {
 				getLocalDateTimeFromUTCDateTime(val);
 				this._value = val;
-			} catch (e) {
+			} catch {
 				this._value = '';
 			}
 		}

package/components/inputs/input-date.js

@@ -371,7 +371,7 @@ class InputDate extends FocusMixin(LabelledMixin(SkeletonMixin(FormElementMixin(
 		try {
 			const date = parseDate(value);
 			await this._updateValueDispatchEvent(formatDateInISO({ year: date.getFullYear(), month: (parseInt(date.getMonth()) + 1), date: date.getDate() }));
-		} catch (err) {
+		} catch {
 			// leave value the same when invalid input
 		}
 		this._setFormattedValue(); // keep out here in case parseDate is same date, e.g., user adds invalid text to end of parseable date

package/components/list/list-item-generic-layout.js

@@ -205,7 +205,6 @@ class ListItemGenericLayout extends RtlMixin(LitElement) {
 				const slot = (event.path || event.composedPath()).find((node) =>
 					node.nodeName === 'SLOT' && ['content'].includes(node.name)
 				);
-				// eslint-disable-next-line no-console
 				console.warn(`${slot.name} area should not have focusable items in it. Consider using href or creating a custom list-item.`);
 			},
 			capture: true

package/helpers/focus.js

@@ -202,7 +202,7 @@ export function isFocusVisibleSupported() {
 			document.head.appendChild(style);
 			style.sheet.insertRule(':focus-visible { color: inherit; }');
 			_isFocusVisibleSupported = true;
-		} catch (error) {
+		} catch {
 			_isFocusVisibleSupported = false;
 		} finally {
 			style.remove();

package/helpers/framed.js

@@ -13,7 +13,7 @@ export async function isFramed() {
 			framed = false;
 			return framed;
 		}
-	} catch (e) {
+	} catch {
 		framed = false;
 		return framed;
 	}

package/mixins/async-container/async-container-mixin.js

@@ -59,7 +59,7 @@ export const AsyncContainerMixin = superclass => class extends superclass {
 		try {
 			await promise;
 			if (this._asyncPromises.indexOf(promise) !== -1) this._asyncCounts.fulfilled++;
-		} catch (error) {
+		} catch {
 			if (this._asyncPromises.indexOf(promise) !== -1) this._asyncCounts.rejected++;
 		} finally {
 			if (this._asyncPromises.indexOf(promise) !== -1) {

package/mixins/loading-complete/loading-complete-mixin.js

@@ -1,7 +1,6 @@
 import { dedupeMixin } from '@open-wc/dedupe-mixin';
 
 export const LoadingCompleteMixin = dedupeMixin((superclass) => class extends superclass {
-
 	get loadingComplete() {
 		return this.getLoadingComplete();
 	}
@@ -22,7 +21,6 @@ export const LoadingCompleteMixin = dedupeMixin((superclass) => class extends su
 
 	#loadingCompleteResolve;
 
-	// eslint-disable-next-line sort-class-members/sort-class-members
 	#loadingCompletePromise = !Object.prototype.hasOwnProperty.call(this.constructor.prototype, 'getLoadingComplete')
 		? new Promise(resolve => this.#loadingCompleteResolve = resolve)
 		: Promise.resolve();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@brightspace-ui/core",
-  "version": "3.62.4",
+  "version": "3.63.0",
   "description": "A collection of accessible, free, open-source web components for building Brightspace applications",
   "type": "module",
   "repository": "https://github.com/BrightspaceUI/core.git",
@@ -16,7 +16,7 @@
     "build": "npm run build:clean && npm run build:icons && npm run build:illustrations && npm run build:sass && npm run build:wca",
     "build-static": "rollup -c ./rollup/rollup.config.js",
     "lint": "npm run lint:eslint && npm run lint:style",
-    "lint:eslint": "eslint . --ext .js,.html --ignore-path .gitignore",
+    "lint:eslint": "eslint .",
     "lint:style": "stylelint \"**/*.{js,html}\" --ignore-path .gitignore",
     "start": "web-dev-server --node-resolve --watch --open",
     "test": "npm run lint && npm run test:translations && npm run test:unit && npm run test:axe",
@@ -47,13 +47,16 @@
   "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": "^8",
-    "eslint-config-brightspace": "^1",
+    "eslint": "^9",
+    "eslint-config-brightspace": "^1.2.1",
     "glob-all": "^3",
     "messageformat-validator": "^2",
     "node-sass": "^9",

package/templates/primary-secondary/primary-secondary.js

@@ -1066,7 +1066,7 @@ class TemplatePrimarySecondary extends RtlMixin(LocalizeCoreElement(LitElement))
 				const key = computeSizeKey(this.storageKey);
 				try {
 					localStorage.setItem(key, this._size);
-				} catch (ex) {
+				} catch {
 					// throws if storage is full or in private mode in mobile Safari
 				}
 			}
@@ -1146,7 +1146,7 @@ class TemplatePrimarySecondary extends RtlMixin(LocalizeCoreElement(LitElement))
 				const key = computeSizeKey(this.storageKey);
 				try {
 					size = parseFloat(localStorage.getItem(key));
-				} catch (ex) {
+				} catch {
 					// may throw SecurityError if localStorage isn't allowed to be accessed
 				}
 			}