Chris Simmons

Results 477 comments of Chris Simmons

@Mahmoud-zino I'll review and provide a more in-depth response when I do my full review. But I did want to clarify on this point: > I noticed there's no binding...

@Mahmoud-zino this wasn't marked as "ready for review" so feel free to ignore if things are still in progress. But thought I'd chime in with a quick overview. I'll approach...

@Mahmoud-zino I've gone ahead and closed all feedback comments above so we can start "fresh". You were right, there's a number of places where the ARIA spec diverges from my...

@Mahmoud-zino here's the second wave of updates: - Further refinement of the local SvelteKit preview page - Updated the TW config to enable all themes for either preview app -...

@Mahmoud-zino I've go ahead and merged this PR. The reason being is this had some linting changes that spilled over into the documentation section. We need the doc section as...

@coldino unfortunately after researching this I think it's going to be a heavier lift than expected. The reason being is that we now implement our components on top of Zag....

@coldino understood - thanks for detailing the use case. Now that you mentioned it I do seem to recall that being a feature of Svelte snippets. I just realized I...

This will be solved in the new Skeleton v4 release. We launched the Release Candidate today if you would like to try it out right away: https://github.com/skeletonlabs/skeleton/discussions/3844 Docs: https://next.skeleton.dev/docs/components/slider/svelte

Thanks @eagerestwolf, two things of note. First off, make sure you've read through this post. This outlines my availability, especially this week: - https://github.com/skeletonlabs/skeleton/discussions/2729 Second, per this > Third, it's...

Quick update per the audit items above - we've merged the following PR: - https://github.com/skeletonlabs/skeleton/pull/2734 This does two thigns: 1. Sets the proper `$lib` alias path in the `skeleton-react` project...