svelte-multiselect 8.4.0 → 8.5.0

package/dist/CmdPalette.svelte

@@ -1,12 +1,18 @@
-<script>import { tick } from 'svelte/internal';
+<script>/* eslint-disable no-undef */ // TODO: remove when fixed
+// https://github.com/sveltejs/eslint-plugin-svelte3/issues/201
+import { tick } from 'svelte';
 import { fade } from 'svelte/transition';
 import Select from '.';
 export let actions;
 export let trigger = `k`;
 export let fade_duration = 200; // in ms
-let open = false;
-let dialog;
-let input;
+export let style = ``; // for dialog
+// for span in option slot, has no effect when passing slot="option"
+export let span_style = ``;
+export let open = false;
+export let dialog;
+export let input;
+export let placeholder = `Filter actions...`;
 async function toggle(event) {
     if (event.key === trigger && event.metaKey && !open) {
         // open on cmd+trigger
@@ -24,7 +30,7 @@ function close_if_outside(event) {
         open = false;
     }
 }
-function move(event) {
+function run_and_close(event) {
     event.detail.option.action();
     open = false;
 }
@@ -33,14 +39,25 @@ function move(event) {
 <svelte:window on:keydown={toggle} on:click={close_if_outside} />
 
 {#if open}
-  <dialog class:open bind:this={dialog} transition:fade={{ duration: fade_duration }}>
+  <dialog
+    class:open
+    bind:this={dialog}
+    transition:fade={{ duration: fade_duration }}
+    {style}
+  >
     <Select
       options={actions}
       bind:input
-      placeholder="Go to..."
-      on:add={move}
+      {placeholder}
+      on:add={run_and_close}
       on:keydown={toggle}
-    />
+      {...$$props}
+    >
+      <!-- wait for https://github.com/sveltejs/svelte/pull/8304 -->
+      <slot slot="option" name="option" let:option>
+        <span style={span_style}>{option.label}</span>
+      </slot>
+    </Select>
   </dialog>
 {/if}
 

package/dist/CmdPalette.svelte.d.ts

@@ -1,17 +1,28 @@
 import { SvelteComponentTyped } from "svelte";
 declare const __propDef: {
     props: {
+        [x: string]: any;
         actions: {
             label: string;
             action: () => void;
         }[];
         trigger?: string | undefined;
         fade_duration?: number | undefined;
+        style?: string | undefined;
+        span_style?: string | undefined;
+        open?: boolean | undefined;
+        dialog: HTMLDialogElement;
+        input: HTMLInputElement;
+        placeholder?: string | undefined;
     };
     events: {
         [evt: string]: CustomEvent<any>;
     };
-    slots: {};
+    slots: {
+        option: {
+            slot: string;
+        };
+    };
 };
 export type CmdPaletteProps = typeof __propDef.props;
 export type CmdPaletteEvents = typeof __propDef.events;

package/dist/MultiSelect.svelte

@@ -287,10 +287,10 @@ async function handle_keydown(event) {
     }
 }
 function remove_all() {
-    dispatch(`removeAll`, { options: selected });
-    dispatch(`change`, { options: selected, type: `removeAll` });
     selected = [];
     searchText = ``;
+    dispatch(`removeAll`, { options: selected });
+    dispatch(`change`, { options: selected, type: `removeAll` });
 }
 $: is_selected = (label) => selected.map(get_label).includes(label);
 const if_enter_or_space = (handler) => (event) => {
@@ -344,13 +344,10 @@ const dragstart = (idx) => (event) => {
   class:disabled
   class:single={maxSelect === 1}
   class:open
-  aria-expanded={open}
-  aria-multiselectable={maxSelect === null || maxSelect > 1}
   class:invalid
   class="multiselect {outerDivClass}"
   on:mouseup|stopPropagation={open_dropdown}
   title={disabled ? disabledInputTitle : null}
-  aria-disabled={disabled ? `true` : null}
   data-id={id}
 >
   <!-- bind:value={selected} prevents form submission if required prop is true and no options are selected -->
@@ -379,10 +376,16 @@ const dragstart = (idx) => (event) => {
   <slot name="expand-icon" {open}>
     <ExpandIcon width="15px" style="min-width: 1em; padding: 0 1pt; cursor: pointer;" />
   </slot>
-  <ul class="selected {ulSelectedClass}">
+  <ul
+    class="selected {ulSelectedClass}"
+    role="listbox"
+    aria-multiselectable={maxSelect === null || maxSelect > 1}
+    aria-label="selected options"
+  >
     {#each selected as option, idx (get_label(option))}
       <li
         class={liSelectedClass}
+        role="option"
         aria-selected="true"
         animate:flip={{ duration: 100 }}
         draggable={selectedOptionsDraggable && !disabled && selected.length > 1}
@@ -397,8 +400,7 @@ const dragstart = (idx) => (event) => {
           {#if parseLabelsAsHtml}
             {@html get_label(option)}
           {:else}
-            {get_label(option)}
-          {/if}
+            {get_label(option)}{/if}
         </slot>
         {#if !disabled && (minSelect === null || selected.length > minSelect)}
           <button
@@ -415,38 +417,36 @@ const dragstart = (idx) => (event) => {
         {/if}
       </li>
     {/each}
-    <li style="display: contents;">
-      <input
-        class={inputClass}
-        bind:this={input}
-        bind:value={searchText}
-        on:mouseup|self|stopPropagation={open_dropdown}
-        on:keydown|stopPropagation={handle_keydown}
-        on:focus
-        on:focus={open_dropdown}
-        {id}
-        {disabled}
-        {autocomplete}
-        {inputmode}
-        {pattern}
-        placeholder={selected.length == 0 ? placeholder : null}
-        aria-invalid={invalid ? `true` : null}
-        ondrop="return false"
-        on:blur
-        on:change
-        on:click
-        on:keydown
-        on:keyup
-        on:mousedown
-        on:mouseenter
-        on:mouseleave
-        on:touchcancel
-        on:touchend
-        on:touchmove
-        on:touchstart
-      />
-      <!-- the above on:* lines forward potentially useful DOM events -->
-    </li>
+    <input
+      class={inputClass}
+      bind:this={input}
+      bind:value={searchText}
+      on:mouseup|self|stopPropagation={open_dropdown}
+      on:keydown|stopPropagation={handle_keydown}
+      on:focus
+      on:focus={open_dropdown}
+      {id}
+      {disabled}
+      {autocomplete}
+      {inputmode}
+      {pattern}
+      placeholder={selected.length == 0 ? placeholder : null}
+      aria-invalid={invalid ? `true` : null}
+      ondrop="return false"
+      on:blur
+      on:change
+      on:click
+      on:keydown
+      on:keyup
+      on:mousedown
+      on:mouseenter
+      on:mouseleave
+      on:touchcancel
+      on:touchend
+      on:touchmove
+      on:touchstart
+    />
+    <!-- the above on:* lines forward potentially useful DOM events -->
   </ul>
   {#if loading}
     <slot name="spinner">
@@ -482,7 +482,14 @@ const dragstart = (idx) => (event) => {
 
   <!-- only render options dropdown if options or searchText is not empty needed to avoid briefly flashing empty dropdown -->
   {#if (searchText && noMatchingOptionsMsg) || options?.length > 0}
-    <ul class:hidden={!open} class="options {ulOptionsClass}">
+    <ul
+      class:hidden={!open}
+      class="options {ulOptionsClass}"
+      role="listbox"
+      aria-multiselectable={maxSelect === null || maxSelect > 1}
+      aria-expanded={open}
+      aria-disabled={disabled ? `true` : null}
+    >
       {#each matchingOptions as option, idx}
         {@const {
           label,
@@ -512,6 +519,7 @@ const dragstart = (idx) => (event) => {
           }}
           on:mouseout={() => (activeIndex = null)}
           on:blur={() => (activeIndex = null)}
+          role="option"
           aria-selected="false"
         >
           <slot name="option" {option} {idx}>
@@ -627,7 +635,7 @@ const dragstart = (idx) => (event) => {
     margin: auto 0; /* CSS reset */
     padding: 0; /* CSS reset */
   }
-  :where(div.multiselect > ul.selected > li > input) {
+  :where(div.multiselect > ul.selected > input) {
     border: none;
     outline: none;
     background: none;
@@ -640,7 +648,7 @@ const dragstart = (idx) => (event) => {
     border-radius: 0; /* reset ul.selected > li */
   }
   /* don't wrap ::placeholder rules in :where() as it seems to be overpowered by browser defaults i.t.o. specificity */
-  div.multiselect > ul.selected > li > input::placeholder {
+  div.multiselect > ul.selected > input::placeholder {
     padding-left: 5pt;
     color: var(--sms-placeholder-color);
     opacity: var(--sms-placeholder-opacity);

package/package.json

@@ -5,7 +5,7 @@
   "homepage": "https://janosh.github.io/svelte-multiselect",
   "repository": "https://github.com/janosh/svelte-multiselect",
   "license": "MIT",
-  "version": "8.4.0",
+  "version": "8.5.0",
   "type": "module",
   "svelte": "./dist/index.js",
   "bugs": "https://github.com/janosh/svelte-multiselect/issues",
@@ -23,17 +23,17 @@
     "update-coverage": "vitest tests/unit --run --coverage && npx istanbul-badges-readme"
   },
   "dependencies": {
-    "svelte": "^3.55.1"
+    "svelte": "^3.56.0"
   },
   "devDependencies": {
     "@iconify/svelte": "^3.1.0",
-    "@playwright/test": "^1.31.1",
+    "@playwright/test": "^1.31.2",
     "@sveltejs/adapter-static": "^2.0.1",
-    "@sveltejs/kit": "^1.9.2",
+    "@sveltejs/kit": "^1.11.0",
     "@sveltejs/package": "2.0.2",
     "@sveltejs/vite-plugin-svelte": "^2.0.3",
-    "@typescript-eslint/eslint-plugin": "^5.54.0",
-    "@typescript-eslint/parser": "^5.54.0",
+    "@typescript-eslint/eslint-plugin": "^5.54.1",
+    "@typescript-eslint/parser": "^5.54.1",
     "@vitest/coverage-c8": "^0.29.2",
     "eslint": "^8.35.0",
     "eslint-plugin-svelte3": "^4.0.0",
@@ -46,11 +46,11 @@
     "prettier-plugin-svelte": "^2.9.0",
     "rehype-autolink-headings": "^6.1.1",
     "rehype-slug": "^5.1.0",
-    "svelte-check": "^3.0.4",
+    "svelte-check": "^3.1.0",
     "svelte-preprocess": "^5.0.1",
     "svelte-toc": "^0.5.2",
     "svelte-zoo": "^0.3.4",
-    "svelte2tsx": "^0.6.2",
+    "svelte2tsx": "^0.6.3",
     "typescript": "^4.9.5",
     "vite": "^4.1.4",
     "vitest": "^0.29.2"

package/readme.md

@@ -8,7 +8,7 @@
 [![Tests](https://github.com/janosh/svelte-multiselect/actions/workflows/test.yml/badge.svg)](https://github.com/janosh/svelte-multiselect/actions/workflows/test.yml)
 [![GitHub Pages](https://github.com/janosh/svelte-multiselect/actions/workflows/gh-pages.yml/badge.svg)](https://github.com/janosh/svelte-multiselect/actions/workflows/gh-pages.yml)
 [![NPM version](https://img.shields.io/npm/v/svelte-multiselect?logo=NPM&color=purple)](https://npmjs.com/package/svelte-multiselect)
-[![Needs Svelte version](https://img.shields.io/npm/dependency-version/svelte-multiselect/dev/svelte?color=teal&logo=Svelte&label=Svelte)](https://github.com/sveltejs/svelte/blob/master/CHANGELOG.md)
+[![Needs Svelte version](https://img.shields.io/npm/dependency-version/svelte-multiselect/svelte?color=teal&logo=Svelte&label=Svelte)](https://github.com/sveltejs/svelte/blob/master/CHANGELOG.md)
 [![REPL](https://img.shields.io/badge/Svelte-REPL-blue?label=Try%20it!)](https://svelte.dev/repl/a5a14b8f15d64cb083b567292480db05)
 [![Open in StackBlitz](https://img.shields.io/badge/Open%20in-StackBlitz-darkblue?logo=stackblitz)](https://stackblitz.com/github/janosh/svelte-multiselect)
 
@@ -611,7 +611,7 @@ Odd as it may seem, you get the most fine-grained control over the styling of ev
 :global(div.multiselect > ul.selected > li button, button.remove-all) {
   /* buttons to remove a single or all selected options at once */
 }
-:global(div.multiselect > ul.selected > li > input) {
+:global(div.multiselect > input[autocomplete]) {
   /* input inside the top-level wrapper div */
 }
 :global(div.multiselect > ul.options) {