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

Rebuild #23

Merged
merged 8 commits into from
Mar 1, 2024
Merged

Rebuild #23

merged 8 commits into from
Mar 1, 2024

Conversation

Elliot-Alexander
Copy link
Collaborator

Restructured a huge chunk of the controls and Drawer Root to make things more readable and more 'Vue' flavoured.

Currently running into a big issue with the controlled (and the types of controlled) vs uncontrolled. The test case to compare between this and the original package is the controlled test in the playwright tests through playground.

Would definitely be open to any help on this one.

@zernonia
Copy link
Member

Thanks for the PR @Elliot-Alexander ! Looking at this PR today! 😁

Copy link

cloudflare-workers-and-pages bot commented Feb 27, 2024

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: a058285
Status: ✅  Deploy successful!
Preview URL: https://84811f97.vaul-vue.pages.dev
Branch Preview URL: https://rebuild.vaul-vue.pages.dev

View logs

refactor: use antfu/eslint
@zernonia
Copy link
Member

@Elliot-Alexander I've made 'some' changes 😂 and seems like all the tests are passing now. (temporary hide webkit as there's some weird behavior with 1 of the test).

Let me know if the above fixes the

big issue with the controlled (and the types of controlled) vs uncontrolled.

you mentioned earlier.

@zernonia zernonia marked this pull request as ready for review February 27, 2024 18:10
@zernonia
Copy link
Member

zernonia commented Mar 1, 2024

@Elliot-Alexander Nicely done on making it closer to Vue DX. I will merge this in for now ya!

@zernonia zernonia merged commit 389c825 into main Mar 1, 2024
2 checks passed
@zernonia zernonia deleted the rebuild branch March 1, 2024 15:04
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.

2 participants