@brightspace-ui/core 3.219.4 → 3.219.6

package/components/list/list-item-mixin.js

@@ -197,8 +197,8 @@ export const ListItemMixin = superclass => class extends composeMixins(
 			:host([_hovering-selection]) [slot="control-container"]::after,
 			:host([_hovering-primary-action]) [slot="control-container"]::before,
 			:host([_hovering-primary-action]) [slot="control-container"]::after,
-			:host([selectable][_focusing]) [slot="control-container"]::before,
-			:host([selectable][_focusing]) [slot="control-container"]::after,
+			:host([selectable][_focusing]:not([selection-disabled])) [slot="control-container"]::before,
+			:host([selectable][_focusing]:not([selection-disabled])) [slot="control-container"]::after,
 			:host([_focusing-primary-action]) [slot="control-container"]::before,
 			:host([_focusing-primary-action]) [slot="control-container"]::after,
 			:host([selected]:not([selection-disabled]):not([skeleton])) [slot="control-container"]::before,

package/components/table/table-wrapper.js

@@ -4,7 +4,6 @@ import '../backdrop/backdrop-loading.js';
 import { css, html, LitElement, nothing } from 'lit';
 import { cssSizes } from '../inputs/input-checkbox.js';
 import { getComposedParent } from '../../helpers/dom.js';
-import { getFlag } from '../../helpers/flags.js';
 import { isPopoverSupported } from '../popover/popover-mixin.js';
 import { PageableMixin } from '../paging/pageable-mixin.js';
 import ResizeObserver from 'resize-observer-polyfill/dist/ResizeObserver.es.js';
@@ -14,8 +13,6 @@ import { usePopoverMixin as useTooltipPopover } from '../tooltip/tooltip.js';
 
 export const isUsingNativePopover = isPopoverSupported && useDropdownPopover && useTooltipPopover;
 
-const colSyncFix = getFlag('GAUD-8228-8186-improved-table-col-sync', true);
-
 export const tableStyles = css`
 	.d2l-table {
 		border-collapse: separate; /* needed to override reset stylesheets */
@@ -228,22 +225,17 @@ export const tableStyles = css`
 	}
 
 	/* sticky + scroll-wrapper */
-	${colSyncFix ? css`
-		d2l-table-wrapper[sticky-headers][sticky-headers-scroll-wrapper]:not([_no-scroll-width]) .d2l-table {
-			display: flex;
-			flex-direction: column;
-		}
+	d2l-table-wrapper[sticky-headers][sticky-headers-scroll-wrapper]:not([_no-scroll-width]) .d2l-table {
+		display: flex;
+		flex-direction: column;
+	}
 
-		d2l-table-wrapper[sticky-headers][sticky-headers-scroll-wrapper][_no-scroll-width] .d2l-table > thead {
-			display: table-header-group;
-		}
+	d2l-table-wrapper[sticky-headers][sticky-headers-scroll-wrapper][_no-scroll-width] .d2l-table > thead {
+		display: table-header-group;
+	}
 
-		d2l-table-wrapper[sticky-headers][sticky-headers-scroll-wrapper][_no-scroll-width] .d2l-table > tbody {
-			display: table-row-group;
-		}` : css`
-		d2l-table-wrapper[sticky-headers][sticky-headers-scroll-wrapper] .d2l-table {
-			display: block;
-		}`
+	d2l-table-wrapper[sticky-headers][sticky-headers-scroll-wrapper][_no-scroll-width] .d2l-table > tbody {
+		display: table-row-group;
 	}
 
 	d2l-table-wrapper[sticky-headers][sticky-headers-scroll-wrapper] .d2l-table > thead {
@@ -392,7 +384,7 @@ export class TableWrapper extends PageableMixin(SelectionMixin(LitElement)) {
 		this._controlsMutationObserver = null;
 		this._controlsScrolled = false;
 		this._controlsScrolledMutationObserver = null;
-		this._noScrollWidth = colSyncFix;
+		this._noScrollWidth = true;
 		this._table = null;
 		this._tableIntersectionObserver = null;
 		this._tableMutationObserver = null;
@@ -598,7 +590,7 @@ export class TableWrapper extends PageableMixin(SelectionMixin(LitElement)) {
 
 		if (!this._tableResizeObserver) this._tableResizeObserver = new ResizeObserver(entries => this._syncColumnWidths(entries));
 		this._tableResizeObserver.observe(this._table);
-		colSyncFix && this.querySelectorAll('tr:first-child *').forEach(el => this._tableResizeObserver.observe(el));
+		this.querySelectorAll('tr:first-child *').forEach(el => this._tableResizeObserver.observe(el));
 
 		this._handleTableChange();
 	}
@@ -633,7 +625,7 @@ export class TableWrapper extends PageableMixin(SelectionMixin(LitElement)) {
 
 		if (updates.count) this._updateItemShowingCount();
 		if (updates.classNames) this._applyClassNames();
-		colSyncFix && await Promise.all([...updateList, ...this.querySelectorAll('d2l-table-col-sort-button')].map(n => n.updateComplete));
+		await Promise.all([...updateList, ...this.querySelectorAll('d2l-table-col-sort-button')].map(n => n.updateComplete));
 		if (updates.syncWidths) this._syncColumnWidths();
 		if (updates.sticky) this._updateStickyTops();
 	}
@@ -652,12 +644,10 @@ export class TableWrapper extends PageableMixin(SelectionMixin(LitElement)) {
 		const head = this._table.querySelector('thead');
 		const body = this._table.querySelector('tbody');
 
-		if (colSyncFix) {
-			const maxScrollWidth = Math.max(head?.scrollWidth, body?.scrollWidth);
-			setTimeout(() => {
-				this._noScrollWidth = this.clientWidth === maxScrollWidth;
-			});
-		}
+		const maxScrollWidth = Math.max(head?.scrollWidth, body?.scrollWidth);
+		setTimeout(() => {
+			this._noScrollWidth = this.clientWidth === maxScrollWidth;
+		});
 		if (!head || !body || !this._table || !this.stickyHeaders || !this.stickyHeadersScrollWrapper || this._noScrollWidth) return;
 
 		const candidateRowHeadCells = [];

package/package.json

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