Skip to content

Remove project-specific preview link handling#25

Merged
BunsDev merged 1 commit intomainfrom
okcode/fix-remove-project-placement
Mar 27, 2026
Merged

Remove project-specific preview link handling#25
BunsDev merged 1 commit intomainfrom
okcode/fix-remove-project-placement

Conversation

@BunsDev
Copy link
Copy Markdown
Member

@BunsDev BunsDev commented Mar 27, 2026

Summary

  • Stop forwarding terminal link coordinates to the desktop bridge and let Electron position the native menu itself.
  • Remove the embedded preview link-routing path and simplify preview handling to localhost-only URLs.
  • Delete the openLinksExternally setting and its settings UI since link opening now always uses the native browser fallback.
  • Tighten preview validation and update copy/tests to reflect the localhost-only preview flow.

Testing

  • bun fmt
  • bun lint
  • bun typecheck
  • bun run test

- Call the Electron desktop bridge with menu items only
- Keep fallback menu positioning unchanged for web
@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
okcode-marketing Ready Ready Preview Mar 27, 2026 10:45pm

@github-actions github-actions Bot added size:XS vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels Mar 27, 2026
@BunsDev BunsDev merged commit 20ee1f4 into main Mar 27, 2026
7 of 8 checks passed
@BunsDev BunsDev deleted the okcode/fix-remove-project-placement branch March 28, 2026 05:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant