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

feat: Astro View Transitions #44

Merged
merged 3 commits into from
Aug 1, 2023
Merged

Conversation

JoshuaKGoldberg
Copy link
Owner

@JoshuaKGoldberg JoshuaKGoldberg commented Aug 1, 2023

PR Checklist

Overview

Uses the most minimal usage of https://docs.astro.build/en/guides/view-transitions to get all pages to fade between each other.

I'm very impressed with how straightforward this was. The diff count is really more like +7 🟩 -0, with two of those additions just being blank lines. I also happened to bump the astro package version in this PR.

Screen recording of my personal site. The user is navigating repeatedly between the homepage, blog, projects, and speaking. There are half-second fades between pages (implied to be from the view transitions API).

I'd initially wanted to only fade a few elements (header, hero images) at first - but pending withastro/roadmap#633 I don't see a good way to do that.

@vercel
Copy link

vercel bot commented Aug 1, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
joshuakgoldberg-dot-com ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 1, 2023 10:07pm

@JoshuaKGoldberg JoshuaKGoldberg changed the title wip: astro view transitions feat: Astro View Transitions Aug 1, 2023
@JoshuaKGoldberg JoshuaKGoldberg marked this pull request as ready for review August 1, 2023 22:06
@JoshuaKGoldberg JoshuaKGoldberg merged commit e79604a into main Aug 1, 2023
@JoshuaKGoldberg JoshuaKGoldberg deleted the astro-view-transitions branch August 1, 2023 22:09
@@ -29,7 +29,7 @@
"@kobalte/core": "^0.9.6",
"@typescript-eslint/eslint-plugin": "^5.59.9",
"@typescript-eslint/parser": "^5.59.9",
"astro": "^2.6.2",
"astro": "^2.9.7",
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that the pnpm-lock.yaml file in this PR contains a bunch of unrelated changes from me running pnpm dedupe and updating Astro to the latest version.

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.

🚀 Feature: Add page transitions, such as with view transitions API
1 participant