Skip to content

IDE polish; image viewer, preview toolbar, editor themes - #33

Merged
rijulshrestha merged 17 commits into
mainfrom
feat/ide-polish
Sep 8, 2026
Merged

IDE polish; image viewer, preview toolbar, editor themes#33
rijulshrestha merged 17 commits into
mainfrom
feat/ide-polish

Conversation

@rijulshrestha

Copy link
Copy Markdown
Contributor

Four IDE improvements plus two clone-boot fixes.

  • Image viewer: image files open in a viewer instead of dumping raw bytes into Monaco.
  • Preview toolbar: reworked with hide and reload controls.
  • Editor themes: a settings menu for switching TextMate themes.
  • Cloned repos: Next.js clones run on webpack instead of Turbopack; the rolldown wasm binding now applies only from Vite 8.2.

@rijulshrestha rijulshrestha self-assigned this Aug 25, 2026
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 1, 2026

Copy link
Copy Markdown

Deploying browsercode with  Cloudflare Pages  Cloudflare Pages

Latest commit: 3b61052
Status: ✅  Deploy successful!
Preview URL: https://0e4f7d2f.browsercode.pages.dev
Branch Preview URL: https://feat-ide-polish.browsercode.pages.dev

View logs

@rijulshrestha

Copy link
Copy Markdown
Contributor Author

Refactors

  1. Portal takes the PortalState controller instead of 18 forwarded props.
  2. Agent boots gate on a CredentialSpec in cliConfigs instead of tool === 'codex' branches
  3. Pane-drag lifecycle moves to utils/drag.ts, shared by IdeShell and the agents route.

No logic changes has been made. Visibly the terminal drags to 90% instead of stopping at 600px.

@GabrielaReyna GabrielaReyna left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excellent work as usual, just added some minor comments. After addressing these it should be ready for merging.

One question, since you added "Cloned repos: Next.js clones run on webpack instead of Turbopack; the rolldown wasm binding now applies only from Vite 8.2.", these will start working only after merging the changes in the other PR, correct?

Comment thread src/lib/ide/session.svelte.ts
Comment thread src/lib/ide/media.ts
Comment thread src/lib/components/ide/ImageViewer.svelte
Comment thread src/lib/components/ide/SettingsMenu.svelte Outdated
@rijulshrestha

Copy link
Copy Markdown
Contributor Author

The first two commits overlap in both PRs. The Vite and Next changes are present in this PR, so they'll work even without merging the other PR. These two commits are meant to be compatible with browserpod v3.0.1, which is why they're present here.

The Vite modification is required for the latest Vite to work, but the Next fix isn't present in browserpod v3.0.1. There's no harm in adding it here already, though. The other changes in the other PR are based on the updated browserpod version, which isn't publicly available at the moment.

@rijulshrestha
rijulshrestha merged commit 59992eb into main Sep 8, 2026
1 check passed
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