Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: add qwik example #48

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

chore: add qwik example #48

wants to merge 2 commits into from

Conversation

TBroz15
Copy link

@TBroz15 TBroz15 commented Sep 1, 2024

Added a Qwik example of swapy based on /example/react/

@TBroz15 TBroz15 deleted the branch TahaSh:main September 15, 2024 02:38
@TBroz15 TBroz15 closed this Sep 15, 2024
@TBroz15 TBroz15 deleted the main branch September 15, 2024 02:38
@TBroz15 TBroz15 restored the main branch September 15, 2024 02:39
@TBroz15 TBroz15 reopened this Sep 15, 2024
@TahaSh
Copy link
Owner

TahaSh commented Dec 15, 2024

Thank you so much for this PR! But can you please update it to follow the new examples in v1.0? For consistency, it's better if it follows the same layout and styles; so it's only different in how the framework is used.

@JuicyBenjamin
Copy link

It should probably use a ref instead of grabbing element by class. And then a side note, does it make sense to use local storage in the example? To me it's cool to have an example, but it's also a preference to use localStorage to save it, so potentially it bloats the example making it harder to read.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants