Skip to content

fix(site): improve trajectory iframe loading state and reduce flicker#30

Merged
wsxiaoys merged 1 commit intomainfrom
fix/site-trajectory-loading-flicker
Mar 20, 2026
Merged

fix(site): improve trajectory iframe loading state and reduce flicker#30
wsxiaoys merged 1 commit intomainfrom
fix/site-trajectory-loading-flicker

Conversation

@liangfung
Copy link
Copy Markdown
Contributor

Summary

  • Prevents initial skeleton flicker by setting iframeLoading to true by default.
  • Implements a smoother transition after the iframe loads by delaying the removal of the skeleton.
  • Properly handles timeout cleanup on component unmount to prevent memory leaks or state updates on unmounted components.

Test plan

  • Navigate to the trajectory page.
  • Verify that the skeleton appears immediately and transitions smoothly to the iframe content once it's ready.
  • Check that there are no console errors or unexpected behavior when switching tabs or navigating away.

🤖 Generated with Pochi | Task

- Set initial loading state to true to prevent skeleton flash
- Use a timeout to delay hiding the skeleton after iframe loads for smoother transition
- Clean up timeout on component unmount
- Ensure opacity is set correctly after loading

🤖 Generated with [Pochi](https://getpochi.com) | [Task](https://app.getpochi.com/share/p-4022753ad5ac4e3b8473f718f9c07026)

Co-Authored-By: Pochi <noreply@getpochi.com>
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 20, 2026

PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-03-20 09:52 UTC

@wsxiaoys wsxiaoys merged commit 1285fac into main Mar 20, 2026
1 check passed
zwpaper pushed a commit that referenced this pull request Mar 31, 2026
…#30)

- Set initial loading state to true to prevent skeleton flash
- Use a timeout to delay hiding the skeleton after iframe loads for smoother transition
- Clean up timeout on component unmount
- Ensure opacity is set correctly after loading

🤖 Generated with [Pochi](https://getpochi.com) | [Task](https://app.getpochi.com/share/p-4022753ad5ac4e3b8473f718f9c07026)

Co-authored-by: Pochi <noreply@getpochi.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