Skip to content

Add in-app browser launch for Tweakcn settings#418

Merged
BunsDev merged 1 commit intomainfrom
okcode/open-tweakcn-settings-browser
Apr 13, 2026
Merged

Add in-app browser launch for Tweakcn settings#418
BunsDev merged 1 commit intomainfrom
okcode/open-tweakcn-settings-browser

Conversation

@BunsDev
Copy link
Copy Markdown
Member

@BunsDev BunsDev commented Apr 12, 2026

Summary

  • Add a shared openUrlInAppBrowser helper to open URLs in the desktop preview when thread context is available, pop the preview out when requested, or fall back to the external browser.
  • Update GitHub URL handling to route through the new helper and keep preview/external behavior centralized.
  • Add a Tweakcn button in Settings that opens https://tweakcn.com in the in-app browser with a fallback to the system browser.
  • Cover the new browser helper with unit tests for preview, pop-out, and external fallback paths.

Testing

  • Not run (not requested).
  • New tests added in apps/web/src/lib/openUrlInAppBrowser.test.ts for preview, pop-out, and external open behavior.
  • Existing GitHub URL opening flow updated to use the shared helper.

- Share URL-opening logic between GitHub links and the settings browser button
- Support preview pop-out with external fallback when in-app context is unavailable
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 12, 2026

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

Project Deployment Actions Updated (UTC)
okcode-marketing Ready Ready Preview Apr 12, 2026 10:32pm
v0-compute-the-platform-to-build Ready Ready Preview, Open in v0 Apr 12, 2026 10:32pm

@github-actions github-actions Bot added the vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. label Apr 12, 2026
@BunsDev BunsDev merged commit 95528d8 into main Apr 13, 2026
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:L 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