Only mount portal on open

This commit is contained in:
Conor_Mack 2020-06-29 15:32:37 +01:00
parent fab2a4d23b
commit ffa0dcf601
1 changed files with 12 additions and 11 deletions

View File

@ -27,13 +27,10 @@
onMount(() => {
if (select) {
select.addEventListener("keydown", handleEnter)
selectMenu.addEventListener("keydown", handleEscape)
}
return () => {
select.removeEventListener("keydown", handleEnter)
selectMenu.removeEventListener("keydown", handleEscape)
}
})
@ -73,7 +70,6 @@
getDimensions()
if (isOpen) {
icon.style.transform = "rotate(180deg)"
selectMenu.focus()
} else {
icon.style.transform = "rotate(0deg)"
}
@ -99,6 +95,10 @@
? options.find(o => o.value === value)
: {}
$: if(open && selectMenu) {
selectMenu.focus()
}
$: displayLabel =
selectedOption && selectedOption.label ? selectedOption.label : value || ""
</script>
@ -112,13 +112,15 @@
<span>{displayLabel}</span>
<i bind:this={icon} class="ri-arrow-down-s-fill" />
</div>
{#if open}
<Portal>
<div
tabindex="0"
class:open
bind:this={selectMenu}
style={menuStyle}
class="bb-select-menu"
class:open>
on:keydown={handleEscape}
class="bb-select-menu">
<ul>
{#if isOptionsObject}
{#each options as { value: v, label }}
@ -141,11 +143,10 @@
{/if}
</ul>
</div>
<div on:click|self={() => toggleSelect(false)} class="overlay" />
</Portal>
{/if}
</div>
{#if open}
<div on:click|self={() => toggleSelect(false)} class="overlay" />
{/if}
<style>
.overlay {