Skip to content

Upgrade astro version v4 #220

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

Merged
merged 7 commits into from
Apr 15, 2024
Merged

Upgrade astro version v4 #220

merged 7 commits into from
Apr 15, 2024

Conversation

zzq0826
Copy link
Member

@zzq0826 zzq0826 commented Mar 22, 2024

Copy link

netlify bot commented Mar 22, 2024

Deploy Preview for scroll-documentation ready!

Name Link
🔨 Latest commit dc1dc92
🔍 Latest deploy log https://app.netlify.com/sites/scroll-documentation/deploys/661d8fa681f9b50008d00fd5
😎 Deploy Preview https://deploy-preview-220.docs.scroll.xyz
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@zzq0826 zzq0826 force-pushed the upgrade-astro-version-v4 branch from 588ba1a to 60839b0 Compare March 22, 2024 12:35
@zzq0826 zzq0826 marked this pull request as ready for review March 22, 2024 12:48
@zzq0826 zzq0826 requested a review from dghelm March 22, 2024 12:53
@dghelm
Copy link
Contributor

dghelm commented Mar 27, 2024

Images are no longer allowing for "zoom" as in this page: https://deploy-preview-220.docs.scroll.xyz/en/technology/

@zzq0826
Copy link
Member Author

zzq0826 commented Mar 28, 2024

Images are no longer allowing for "zoom" as in this page: https://deploy-preview-220.docs.scroll.xyz/en/technology/

image Are you referring to this feature? It works for me, it might require the page to be fully loaded?

@dghelm
Copy link
Contributor

dghelm commented Apr 15, 2024

Two bugs fixed:

  • flash of unformatted header (darkmode icon?) caused search bar and tools item to "jump" on page transition
  • page transitions default to fade (despite being marked as none) if initial load is on homepage, not an article page

Also, moved ViewTransition invocation logic from Header (UI element) to HeadCommon (lives in <Head>)

For follow-up: Can we move to a shared tag between different pages? Would this let us use a transition with expected behavior?

@dghelm
Copy link
Contributor

dghelm commented Apr 15, 2024

For follow-up (low priority): Can we maintain page transition across translation shifts when language selector is used?

@dghelm dghelm merged commit a30b9bb into develop Apr 15, 2024
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