CaptainCodeman / svelte-headlessui

HeadlessUI components for Svelte
https://captaincodeman.github.io/svelte-headlessui/
MIT License
554 stars 25 forks source link

Do not require function onSelect for Menu #13

Closed saturnonearth closed 1 year ago

saturnonearth commented 1 year ago

If we are just using anchor links or forms and we don't need to track the events in JS, we should not need to have function onSelect(e: Event). As of now, if you omit this function, it will cause an error.

ReferenceError: onSelect is not defined

CaptainCodeman commented 1 year ago

That should just be in the consumer, sounds like you may still have on:select={onSelect} on the menu button (?)

saturnonearth commented 1 year ago

You're correct! My bad.