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

Add view transitions for card on search page #118

Merged

Conversation

tanishqmanuja
Copy link
Contributor

No description provided.

@satnaing
Copy link
Owner

@tanishqmanuja
Thanks for this PR.

Just one thing. I created Card.astro only because I didn't realize we could add ViewTransitions in jsx.
If we can achieve that in Card.tsx, I assume we don't need a separate Card.astro file anymore.

Additionally, we can also put related styles back inside Card.tsx.
What do you think?

@tanishqmanuja
Copy link
Contributor Author

Sounds good to me.

@satnaing
Copy link
Owner

So, do you want to make changes on your own? Or do you want me to co-author the changes?

@tanishqmanuja
Copy link
Contributor Author

I would be happy with both of the options. I have to do that for my fork anyways, so I will push here, if it needs any refinement you can co-author :)

@satnaing
Copy link
Owner

So far so good. I just made a small change.
Thanks for your contribution.

@satnaing satnaing merged commit b873ed5 into satnaing:feature/version-3 Sep 18, 2023
satnaing added a commit that referenced this pull request Sep 25, 2023
* feat: use predictable slug for card transition

* feat: enable view transitions in search card

* refactor: use Card.tsx instead of Card.astro

* refactor: handle card styling in component

* refactor: remove unused class name

---------

Co-authored-by: Sat Naing <satnaingdev@gmail.com>
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