From 8b39ccae89fac340810f0134d8c8757e029ebe4a Mon Sep 17 00:00:00 2001 From: metonym Date: Wed, 9 Feb 2022 19:53:07 -0800 Subject: [PATCH] fix(multi-select): MultiSelect should correctly lose focus (#1077) Currently, opening a MultiSelect and then clicking an interactive target will cause the MultiSelect to retain focus. This removes the logic that interferes with native blurring behavior. --- src/MultiSelect/MultiSelect.svelte | 22 ---------------------- 1 file changed, 22 deletions(-) diff --git a/src/MultiSelect/MultiSelect.svelte b/src/MultiSelect/MultiSelect.svelte index d3c00ef1..c90fdeda 100644 --- a/src/MultiSelect/MultiSelect.svelte +++ b/src/MultiSelect/MultiSelect.svelte @@ -337,16 +337,6 @@ if (inputRef) inputRef.focus(); } }}" - on:blur="{({ relatedTarget }) => { - if ( - relatedTarget && - !['INPUT', 'SELECT', 'TEXTAREA'].includes(relatedTarget.tagName) && - relatedTarget.getAttribute('role') !== 'button' && - relatedTarget.getAttribute('role') !== 'searchbox' - ) { - fieldRef.focus(); - } - }}" id="{id}" disabled="{disabled}" translateWithId="{translateWithId}" @@ -421,18 +411,6 @@ on:keyup on:focus on:blur - on:blur="{({ relatedTarget }) => { - if ( - relatedTarget && - !['INPUT', 'SELECT', 'TEXTAREA'].includes( - relatedTarget.tagName - ) && - relatedTarget.getAttribute('role') !== 'button' && - relatedTarget.getAttribute('role') !== 'searchbox' - ) { - inputRef.focus(); - } - }}" disabled="{disabled}" placeholder="{placeholder}" id="{id}"