Skip to content

docs: refine useCursor implementation#1501

Merged
ulivz merged 2 commits intomainfrom
fix/replace-usecursor-hook
Sep 13, 2025
Merged

docs: refine useCursor implementation#1501
ulivz merged 2 commits intomainfrom
fix/replace-usecursor-hook

Conversation

@ulivz
Copy link
Member

@ulivz ulivz commented Sep 13, 2025

Summary

Completely remove CursorContext.tsx and replace with local state implementation to resolve SSG build issues.

This issue was encountered during the rspress upgrade in #1495, where the context-based cursor implementation caused SSR hydration problems. The original Context design was over-engineered - cursor functionality works perfectly with local state since CustomCursor manages its own mouse events.

Checklist

  • Added or updated necessary tests (Optional).
  • Updated documentation to align with changes (Optional).
  • Verified no breaking changes, or prepared solutions for any occurring breaking changes (Optional).
  • My change does not involve the above items.

@netlify
Copy link

netlify bot commented Sep 13, 2025

Deploy Preview for agent-tars-docs ready!

Name Link
🔨 Latest commit 5d9290d
🔍 Latest deploy log https://app.netlify.com/projects/agent-tars-docs/deploys/68c5cc0b43b1930008cf2d64
😎 Deploy Preview https://deploy-preview-1501--agent-tars-docs.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.

Delete CursorContext.tsx and replace with local state in CustomCursor.
Remove all context dependencies from Link, HomePage, and theme files.
@ulivz ulivz force-pushed the fix/replace-usecursor-hook branch from 528a78d to ac77b56 Compare September 13, 2025 19:47
@ulivz ulivz changed the title fix(docs): replace useCursor hook implementation docs: refine useCursor implementation Sep 13, 2025
@ulivz ulivz merged commit 760072b into main Sep 13, 2025
8 of 9 checks passed
@ulivz ulivz deleted the fix/replace-usecursor-hook branch September 13, 2025 19:55
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.

1 participant

Comments