Merge pull request #2125 from apuliasoft/fix/kup-automplete-combobox

smeup · Oct 18, 2024 · ce526ff · ce526ff
2 parents c521a9f + 3277450
commit ce526ff
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/packages/ketchup/src/components/kup-autocomplete/kup-autocomplete.tsx b/packages/ketchup/src/components/kup-autocomplete/kup-autocomplete.tsx
@@ -444,8 +444,8 @@ export class KupAutocomplete {
             this.#closeList();
             return false;
         }
-        const hasError = this.error.trim().length > 0;
-        const hasAlert = this.alert.trim().length > 0;
+        const hasError = this.error?.trim().length > 0;
+        const hasAlert = this.alert?.trim().length > 0;
         const topOffset = hasError || hasAlert ? -20 : 0;
         this.#textfieldWrapper.classList.add('toggled');
         this.#listEl.menuVisible = true;

diff --git a/packages/ketchup/src/components/kup-combobox/kup-combobox.tsx b/packages/ketchup/src/components/kup-combobox/kup-combobox.tsx
@@ -422,8 +422,8 @@ export class KupCombobox {
             return;
         }
         // Manage list open while helperline is displayed
-        const hasError = this.error.trim().length > 0;
-        const hasAlert = this.alert.trim().length > 0;
+        const hasError = this.error?.trim().length > 0;
+        const hasAlert = this.alert?.trim().length > 0;
         const topOffset = hasError || hasAlert ? -20 : 0;
         this.#textfieldWrapper.classList.add('toggled');
         this.#listEl.menuVisible = true;