@brightspace-ui/core 3.148.3 → 3.148.5

package/components/list/demo/demo-list-nav.js

@@ -130,7 +130,7 @@ class ListDemoNav extends LitElement {
 				if (!this.shadowRoot) return;
 				const newItem = this.shadowRoot.querySelector('d2l-list').getListItemByKey(sourceListItems[0].key);
 				newItem.activateDragHandle();
-			}, 10);
+			}, 50);
 		}
 	}
 

package/components/list/demo/demo-list-nested.js

@@ -141,7 +141,7 @@ class ListDemoNested extends LitElement {
 				if (!this.shadowRoot) return;
 				const newItem = this.shadowRoot.querySelector('d2l-list').getListItemByKey(sourceListItems[0].key);
 				newItem.activateDragHandle();
-			}, 10);
+			}, 50);
 		}
 
 	}

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

@@ -1,6 +1,7 @@
 import { css, html, LitElement } from 'lit';
 import { findComposedAncestor, isComposedAncestor } from '../../helpers/dom.js';
 import { getComposedActiveElement, getFirstFocusableDescendant, getFocusableDescendants, getLastFocusableDescendant, getNextFocusable, getPreviousFocusable } from '../../helpers/focus.js';
+import { getFlag } from '../../helpers/flags.js';
 import { isInteractiveDescendant } from '../../mixins/interactive/interactive-mixin.js';
 import { RtlMixin } from '../../mixins/rtl/rtl-mixin.js';
 
@@ -17,6 +18,8 @@ const keyCodes = {
 	UP: 38
 };
 
+const listItemUpButtonFixFlag = getFlag('GAUD-8229-list-up-button-fix', true);
+
 /**
  * A component for generating a list item's layout with forced focus ordering and grid support.
  * Focusable items placed in the "content" slot will have their focus removed; use the content-action
@@ -422,13 +425,27 @@ class ListItemGenericLayout extends RtlMixin(LitElement) {
 				do {
 					// this check needs to account for standard list-items as well as custom
 					nestedList = previousElement.querySelector('[slot="nested"]') || previousElement.shadowRoot.querySelector('d2l-list');
-					if (nestedList) {
-						const nestedListItems = [...nestedList.children].filter(node => node.role === 'row');
-						if (nestedListItems.length) {
-							previousElement = nestedListItems[nestedListItems.length - 1];
+					if (listItemUpButtonFixFlag) {
+						// if there is nested list and nested list content is accessible
+						if (nestedList && (!previousElement.expandable || (previousElement.expandable && previousElement.expanded))) {
+							const nestedListItems = [...nestedList.children].filter(node => node.role === 'row');
+							if (nestedListItems.length) {
+								previousElement = nestedListItems[nestedListItems.length - 1];
+							} else {
+								break;
+							}
 						} else {
 							break;
 						}
+					} else {
+						if (nestedList) {
+							const nestedListItems = [...nestedList.children].filter(node => node.role === 'row');
+							if (nestedListItems.length) {
+								previousElement = nestedListItems[nestedListItems.length - 1];
+							} else {
+								break;
+							}
+						}
 					}
 				}	while (nestedList);
 				return previousElement;

package/package.json

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

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

@@ -977,6 +977,7 @@ class TemplatePrimarySecondary extends RtlMixin(LocalizeCoreElement(LitElement))
 
 				.d2l-template-primary-secondary-container > footer {
 					box-shadow: none;
+					padding: 0;
 				}
 
 				.d2l-template-primary-secondary-container,