Merge pull request #363 from IBM/fixes

Fixes

src/ComboBox/ComboBox.svelte

@@ -285,7 +285,7 @@
     </ListBoxField>
     {#if open}
       <ListBoxMenu aria-label="{ariaLabel}" id="{id}">
-        {#each filteredItems as item, i (item.id || i)}
+        {#each filteredItems as item, i (item.id)}
           <ListBoxMenuItem
             id="{item.id}"
             active="{selectedIndex === i || selectedId === item.id}"

src/Dropdown/Dropdown.svelte

@@ -235,7 +235,7 @@
     </button>
     {#if open}
       <ListBoxMenu aria-labelledby="{id}" id="{id}">
-        {#each items as item, i (item.id || i)}
+        {#each items as item, i (item.id)}
           <ListBoxMenuItem
             id="{item.id}"
             active="{selectedIndex === i || selectedId === item.id}"

src/MultiSelect/MultiSelect.svelte

@@ -420,7 +420,7 @@
     </ListBoxField>
     {#if open}
       <ListBoxMenu aria-label="{ariaLabel}" id="{id}">
-        {#each filterable ? filteredItems : sortedItems as item, i (item.id || i)}
+        {#each filterable ? filteredItems : sortedItems as item, i (item.id)}
           <ListBoxMenuItem
             id="{item.id}"
             active="{item.checked}"

src/UIShell/GlobalHeader/Header.svelte

@@ -32,9 +32,9 @@
   /**
    * Specify the platform name
    * Alternatively, use the named slot "platform" (e.g. <span slot="platform">...</span>)
-   * @type {string} [platformName]
+   * @type {string} [platformName=""]
    */
-  export let platformName = undefined;
+  export let platformName = "";
 
   /**
    * Obtain a reference to the HTML anchor element

src/UIShell/HeaderGlobalAction.svelte

@@ -27,8 +27,6 @@
   on:click
 >
   <slot>
-    {#if icon}
-      <svelte:component this="{icon}" />
-    {/if}
+    <svelte:component this="{icon}" />
   </slot>
 </button>