Accordion caret classesControlCaret is repeated on SVG element about skeleton HOT 5 CLOSED

@Keagel @salisshuaibu11 after reviewing the source directly I see the issue here. It looks like the SVG was abstracted to a template literal to cleanup the template a bit.

The class={classesControlCaret} carried through, even though that should have been removed. You can see the new home for those styles is now applied to a wrapping div elements here:

If for some reason you need access to to style the SVG element itself, this is already possible using this Tailwind syntax:
https://www.skeleton.dev/docs/styling#tailwinds-arbitrary-variants

For example:

<AccordionItem classesControlCaret="[&>svg]:fill-red-500">...</AccordionItem>

Alternatively, the caret region is actually a Svelte slot, which means you can replace the open and close icons with custom SVGs. Then adjust the styles as desired.
https://www.skeleton.dev/components/accordions#custom-icons

That said, I'll go ahead and submit a quick change to remove the duplicate class on the SVG template literal now to close this out though. This should help avoid further confusion in the future. Expect this to go out as part of the next release of Skeleton v2.

from skeleton.

@salisshuaibu11 thanks for diving into this. I'd say leave this one to me. I need to dive in an review the full context before I can provide an answer to how we should approach this. And by that time I can probably implement the adjustment directly just as quickly. I'll take all information above into consideration when I get to that point though!

from skeleton.

@endigo9740 i would like to take this

from skeleton.

@endigo9740, I've reviewed the svgCaretIcon. Despite correcting the interpolation, the class remains undefined even though "classesControlCaret" is present. The solution was to embed the SVG directly into the HTML without passing it as a variable and utilizing the @html special tags.

Considering that we're utilizing the "svgCaretIcon" constant in three different locations, is there a more optimal solution available?

from skeleton.

@endigo9740, incorporating the dollar sign label resolved the issue, although I'm uncertain if this is the most efficient approach.

$: svgCaretIcon = svg

from skeleton.