fix(list-item): Always display open toggle cell. #8667

Esri · Jan 30, 2024 · ef27d6c · ef27d6c
1 parent 80dec4e
commit ef27d6c
Showing 1 changed file with 14 additions and 10 deletions.
diff --git a/packages/calcite-components/src/components/list-item/list-item.tsx b/packages/calcite-components/src/components/list-item/list-item.tsx
@@ -473,19 +473,23 @@ export class ListItem
   renderOpen(): VNode {
     const { el, open, openable, messages } = this;
     const dir = getElementDir(el);
-    const icon = open ? ICONS.open : dir === "rtl" ? ICONS.closedRTL : ICONS.closedLTR;
-    const tooltip = open ? messages.collapse : messages.expand;
+    const icon = openable
+      ? open
+        ? ICONS.open
+        : dir === "rtl"
+          ? ICONS.closedRTL
+          : ICONS.closedLTR
+      : ICONS.blank;
 
-    return openable ? (
-      <td
-        class={CSS.openContainer}
-        key="open-container"
-        onClick={this.handleToggleClick}
-        title={tooltip}
-      >
+    const tooltip = openable ? (open ? messages.collapse : messages.expand) : undefined;
+
+    const openClickHandler = openable ? this.handleToggleClick : undefined;
+
+    return (
+      <td class={CSS.openContainer} key="open-container" onClick={openClickHandler} title={tooltip}>
         <calcite-icon icon={icon} key={icon} scale="s" />
       </td>
-    ) : null;
+    );
   }
 
   renderActionsStart(): VNode {