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

Migrate to Chakra V3 #701

Open
Amnish04 opened this issue Oct 28, 2024 · 10 comments
Open

Migrate to Chakra V3 #701

Amnish04 opened this issue Oct 28, 2024 · 10 comments
Assignees

Comments

@Amnish04
Copy link
Collaborator

Amnish04 commented Oct 28, 2024

Recently, Chakra rolled out a new major version with lots of new components and updates styles.
https://www.chakra-ui.com/blog/00-announcing-v3

It would be awesome to upgrade and give ChatCraft a much deserved modern look :D

Would also be interesting to explore the new components and discover places where they could be used for a better UX.

Maybe we should wait for a few patch releases before upgrading though.

@humphd
Copy link
Collaborator

humphd commented Nov 10, 2024

https://www.chakra-ui.com/docs/get-started/migration is the migration guide, for someone who wants to take this.

@fadingNA
Copy link
Collaborator

@humphd , @Amnish04

We can easily migrate to v3.0, and change Chakra component of application wide to match new version, what do you think

@humphd
Copy link
Collaborator

humphd commented Nov 20, 2024

If it's "easy" go for it. I have no idea what it will break in the UI until we try.

@fadingNA
Copy link
Collaborator

@humphd yep, it break entirely our UI application

  • the problem is on framer-motion

Screenshot 2024-11-20 at 1 50 33 PM

@humphd
Copy link
Collaborator

humphd commented Nov 20, 2024

@tarasglek can you please send an invite to @fadingNA? I want to see preview URLs of this change as it happens.

@fadingNA when you get the invite, create your branch as fadingNA/issue-701 and push to this repo vs. your fork.

For framer-motion, do we still need it? I thought they removed it.

@fadingNA
Copy link
Collaborator

@tarasglek can you please send an invite to @fadingNA? I want to see preview URLs of this change as it happens.

  • Thank you so much once change done I will provide URl.

@fadingNA when you get the invite, create your branch as fadingNA/issue-701 and push to this repo vs. your fork.

  • Thank you so much.

For framer-motion, do we still need it? I thought they removed it.

  • Yep v3.0 no need framer anymore.

@humphd
Copy link
Collaborator

humphd commented Nov 20, 2024

if we still need it, it's fine to include. If we can drop the dep, that's even better.

@fadingNA
Copy link
Collaborator

if we still need it, it's fine to include. If we can drop the dep, that's even better.

Got it let me work on this.

@tarasglek
Copy link
Owner

tarasglek commented Nov 22, 2024

@fadingNA sent invite, sorry for delay

@fadingNA
Copy link
Collaborator

@tarasglek no worry at all, appreciate that.

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

No branches or pull requests

4 participants