Skip to content

chore: remove unused view transition code#2898

Merged
SoonIter merged 1 commit intomainfrom
syt/remove-view-transition
Dec 17, 2025
Merged

chore: remove unused view transition code#2898
SoonIter merged 1 commit intomainfrom
syt/remove-view-transition

Conversation

@SoonIter
Copy link
Copy Markdown
Member

@SoonIter SoonIter commented Dec 17, 2025

Summary

Currently, enableContentAnimation is not implemented, so this configuration is currently invalid.

First, delete the code that is not currently used, and it can be added back later.

Related Issue

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

Copilot AI review requested due to automatic review settings December 17, 2025 03:26
@netlify
Copy link
Copy Markdown

netlify bot commented Dec 17, 2025

Deploy Preview for rspress-v2 ready!

Name Link
🔨 Latest commit d575fde
🔍 Latest deploy log https://app.netlify.com/projects/rspress-v2/deploys/694222fc8029bb00086e7fad
😎 Deploy Preview https://deploy-preview-2898--rspress-v2.netlify.app
📱 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 project configuration.

@github-actions
Copy link
Copy Markdown
Contributor

Rsdoctor Bundle Diff Analysis

📊 Quick Summary (Click to expand)
Project Total Size Change
web 14.6 MB ➡️ -522.0 B (-0.0%)

📁 web

Path: website/doc_build/web/rsdoctor-data.json

📌 Baseline Commit: 324386d2a8 | PR: #2896

Metric Current Baseline Change
📊 Total Size 14.6 MB 14.6 MB -522.0 B (-0.0%)
📄 JavaScript 14.0 MB 14.0 MB -522.0 B (-0.0%)
🎨 CSS 113.7 KB 113.7 KB 0 B (0.0%)
🌐 HTML 0 B 0 B N/A
📁 Other Assets 480.4 KB 480.4 KB 0 B (0.0%)

📦 Download Diff Report: web Bundle Diff

Generated by Rsdoctor GitHub Action

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@SoonIter SoonIter merged commit 2de9231 into main Dec 17, 2025
16 checks passed
@SoonIter SoonIter deleted the syt/remove-view-transition branch December 17, 2025 06:45
@SoonIter SoonIter changed the title chore: remove view transition chore: remove unused view transition code Dec 17, 2025
@SoonIter
Copy link
Copy Markdown
Member Author

some new APIs can be considered

#1552

@naico-wang
Copy link
Copy Markdown

image Always get this in my dev environment, maybe some issue caused by useViewTransition hook

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.

4 participants