NEXT Svelte Popover snippet has inaccurate description

[endigo9740]

Reported here: #2919 (comment)

I've found documentation inconsistency, doc states that the Popover, Modal and Tooltip components could have custom trigger element such as button

But actually there's already hard-coded button provided with no way to override it:

skeleton/packages/skeleton-svelte/src/lib/components/Modal/Modal.svelte

Lines 69 to 72 in 67e6918

	{#if trigger}
	<button {...api.getTriggerProps()} class="{triggerBase} {triggerBackground} {triggerClasses}">
	{@render trigger()}
	</button>

skeleton/packages/skeleton-svelte/src/lib/components/Popover/Popover.svelte

Lines 60 to 63 in 67e6918

	<!-- Snippet: Trigger -->
	<button {...api.getTriggerProps()} class="{triggerBase} {triggerBackground} {triggerClasses}">
	{@render trigger?.()}
	</button>

skeleton/packages/skeleton-svelte/src/lib/components/Tooltip/Tooltip.svelte

Lines 55 to 58 in 67e6918

	<!-- Snippet: Trigger -->
	<button {...api.getTriggerProps()} class="{triggerBase} {triggerBackground} {triggerClasses}">
	{@render trigger?.()}
	</button>

Originally posted by @codercms in #2919 (comment)

[phamduylong]

I'll throw a docs update, changing the description to something like: Provide the template contents of the trigger button.

[endigo9740]

That would be great @phamduylong

Just FYI I'm working through this backlog of smaller issues this week, starting with bugfixes. So you'll likely notice those thinning out. If you could please keep calling out the issues you want so we don't double up.

[phamduylong]

Thanks for the notice Chris. I'll try to update along the way. And don't hesitate to ping me if the issues were to pile up and you need help 😄