@brightspace-ui/core 2.184.4 → 2.184.6

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

@@ -434,6 +434,7 @@ export const ListItemMixin = superclass => class extends composeMixins(
 		this.first = false;
 		this.noPrimaryAction = false;
 		this.paddingType = 'normal';
+		this._addButtonTopId = getUniqueId();
 		this._contentId = getUniqueId();
 		this._displayKeyboardTooltip = false;
 		this._hasColorSlot = false;
@@ -667,7 +668,14 @@ export const ListItemMixin = superclass => class extends composeMixins(
 
 		const alignNested = ((this.draggable && this.selectable) || (this.expandable && this.selectable && this.color)) ? 'control' : undefined;
 		const primaryAction = ((!this.noPrimaryAction && this._renderPrimaryAction) ? this._renderPrimaryAction(this._contentId) : null);
-		const tooltipForId = (primaryAction ? this._primaryActionId : (this.selectable ? this._checkboxId : null));
+		let tooltipForId = null;
+		if (this._showAddButton) {
+			tooltipForId = this._addButtonTopId;
+		} else if (primaryAction) {
+			tooltipForId = this._primaryActionId;
+		} else if (this.selectable) {
+			tooltipForId = this._checkboxId;
+		}
 		const addButtonText = this._addButtonText || this.localize('components.list-item.addItem');
 		const innerView = html`
 			<d2l-list-item-generic-layout
@@ -680,7 +688,13 @@ export const ListItemMixin = superclass => class extends composeMixins(
 				?no-primary-action="${this.noPrimaryAction}">
 				${this._showAddButton && this.first ? html`
 				<div slot="add-top">
-					<d2l-button-add text="${addButtonText}" mode="icon-when-interacted" @click="${this._handleButtonAddClick}" data-is-first></d2l-button-add>
+					<d2l-button-add
+						text="${addButtonText}"
+						mode="icon-when-interacted"
+						@click="${this._handleButtonAddClick}"
+						data-is-first
+						id="${this._addButtonTopId}">
+					</d2l-button-add>
 				</div>
 				` : nothing}
 				<div slot="outside-control-container" class="${classMap(bottomBorderClasses)}"></div>

package/components/tag-list/demo/tag-list.html

@@ -42,8 +42,8 @@
 			</script>
 			<d2l-demo-snippet full-width>
 				<d2l-tag-list description="A bunch of example tags">
-					<d2l-tag-list-item text="Example 1"></d2l-tag-list-item>
-					<d2l-tag-list-item text="Example 2 - much much much much much much much much longer"></d2l-tag-list-item>
+					<d2l-tag-list-item text="Example 1 - much much much much much much much much longer"></d2l-tag-list-item>
+					<d2l-tag-list-item text="Example 2"></d2l-tag-list-item>
 					<d2l-tag-list-item text="Example 3"></d2l-tag-list-item>
 					<d2l-tag-list-item-mixin-consumer name="Custom Tag List Item"></d2l-tag-list-item-mixin-consumer>
 					<d2l-tag-list-item text="Example 5"></d2l-tag-list-item>

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

@@ -191,11 +191,11 @@ export const TagListItemMixin = superclass => class extends LocalizeCoreElement(
 
 				announce(message);
 			}
+			this._onFocusIn();
 
 			await this.updateComplete;
 			// delay the focus to allow focusin to fire
 			setTimeout(() => {
-				this._onFocusIn();
 				container.focus();
 			});
 		});

package/package.json

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