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

refactor: migrate context menu to zeego/context-menu #2698

Merged
merged 1 commit into from
Feb 7, 2025

Conversation

lawvs
Copy link
Member

@lawvs lawvs commented Feb 6, 2025

Refactor the context menu implementation to use the zeego/context-menu package, updating all relevant usages throughout the application.

This change enhances maintainability and aligns with the latest library standards.

Copy link

vercel bot commented Feb 6, 2025

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

Name Status Preview Comments Updated (UTC)
follow ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 6, 2025 4:54pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
follow-external-ssr ⬜️ Ignored (Inspect) Visit Preview Feb 6, 2025 4:54pm

@follow-reviewer-bot
Copy link

Suggested PR Title:

refactor: replace context-menu with zeego/context-menu

Change Summary:
Refactor context menus to use new zeego/context-menu package, improving maintainability and performance. Removed deprecated context menus, cleaned up action handling, and optimized component structures. This change significantly reduces code complexity in context-menu related components.

Code Review:
No change requests necessary.

@lawvs lawvs merged commit 900a347 into dev Feb 7, 2025
10 checks passed
@lawvs lawvs deleted the refactor/context-menu branch February 7, 2025 05:47
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