@brightspace-ui/core 3.257.2 → 3.258.0

package/components/calendar/calendar.js

@@ -737,20 +737,10 @@ class Calendar extends LocalizeCoreElement(LitElement) {
 				break;
 			case keyCodes.HOME: {
 				preventDefault = true;
-				let numDaysChange;
 				const dayOfTheWeek = this._focusDate.getDay();
-				if (isRtl) {
-					numDaysChange = 6 - dayOfTheWeek + calendarData.firstDayOfWeek;
-					if (numDaysChange > 6) {
-						numDaysChange -= daysInWeek;
-					}
-				} else {
-					numDaysChange = dayOfTheWeek - calendarData.firstDayOfWeek;
-					if (numDaysChange < 0) {
-						numDaysChange += daysInWeek;
-					}
-					numDaysChange *= -1;
-				}
+				let numDaysChange = dayOfTheWeek - calendarData.firstDayOfWeek;
+				if (numDaysChange < 0) numDaysChange += daysInWeek;
+				numDaysChange *= -1;
 				const possibleFocusDate = new Date(
 					this._focusDate.getFullYear(),
 					this._focusDate.getMonth(),
@@ -761,20 +751,9 @@ class Calendar extends LocalizeCoreElement(LitElement) {
 				break;
 			} case keyCodes.END: {
 				preventDefault = true;
-				let numDaysChange;
 				const dayOfTheWeek = this._focusDate.getDay();
-				if (isRtl) {
-					numDaysChange = dayOfTheWeek - calendarData.firstDayOfWeek;
-					if (numDaysChange < 0) {
-						numDaysChange += daysInWeek;
-					}
-					numDaysChange *= -1;
-				} else {
-					numDaysChange = 6 - dayOfTheWeek + calendarData.firstDayOfWeek;
-					if (numDaysChange > 6) {
-						numDaysChange -= daysInWeek;
-					}
-				}
+				let numDaysChange = 6 - dayOfTheWeek + calendarData.firstDayOfWeek;
+				if (numDaysChange > 6) numDaysChange -= daysInWeek;
 				const possibleFocusDate = new Date(
 					this._focusDate.getFullYear(),
 					this._focusDate.getMonth(),

package/components/dialog/dialog-mixin.js

@@ -5,6 +5,7 @@ import { clearDismissible, setDismissible } from '../../helpers/dismissible.js';
 import { findComposedAncestor, getComposedChildren, isComposedAncestor } from '../../helpers/dom.js';
 import { getComposedActiveElement, getFirstFocusableDescendant, getFirstFocusableRelative, getNextFocusable, isFocusable } from '../../helpers/focus.js';
 import { classMap } from 'lit/directives/class-map.js';
+import { getFlag } from '../../helpers/flags.js';
 import { getUniqueId } from '../../helpers/uniqueId.js';
 import { html } from 'lit';
 import { ifDefined } from 'lit/directives/if-defined.js';
@@ -25,6 +26,8 @@ const reduceMotion = matchMedia('(prefers-reduced-motion: reduce)').matches;
 const abortAction = 'abort';
 const defaultMargin = { top: 75, right: 30, bottom: 30, left: 30 };
 
+const closeDialogWhenDisconnectedFlag = getFlag('GAUD-10113-close-dialog-when-disconnected', true);
+
 export const DialogMixin = superclass => class extends superclass {
 
 	static get properties() {
@@ -107,6 +110,11 @@ export const DialogMixin = superclass => class extends superclass {
 	disconnectedCallback() {
 		super.disconnectedCallback();
 		window.removeEventListener('d2l-mvc-dialog-open', this._handleMvcDialogOpen);
+
+		// If the dialog is disconnected before the close animation finishes
+		if (this.opened && closeDialogWhenDisconnectedFlag) {
+			this._handleClose();
+		}
 	}
 
 	async updated(changedProperties) {
@@ -477,7 +485,7 @@ export const DialogMixin = superclass => class extends superclass {
 	_removeHandlers() {
 		window.removeEventListener('resize', this._updateSize);
 		this.removeEventListener('touchstart', this._handleTouchStart);
-		if (this.shadowRoot) this.shadowRoot.querySelector('.d2l-dialog-content').removeEventListener('scroll', this._updateOverflow);
+		this.shadowRoot?.querySelector('.d2l-dialog-content')?.removeEventListener('scroll', this._updateOverflow);
 	}
 
 	_render(inner, info, iframeTopOverride) {

package/package.json

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