Skip to content

feat(files): fullscreen preview, relative markdown links, and preinstalled Chromium - #369

Merged
chriswritescode-dev merged 4 commits into
mainfrom
fix/file-preview-follow-ups
Sep 29, 2026
Merged

chriswritescode-dev merged 4 commits into
mainfrom
fix/file-preview-follow-ups

Conversation

@chriswritescode-dev

Copy link
Copy Markdown
Owner

Problem

Three commits were pushed to fix/file-browser-paths-and-preview after #367 had already been squash-merged, so they never reached main: the fullscreen file preview, relative markdown link handling, and the Chromium preinstall in the Docker image.

Changes

  • Add a fullscreen file preview with a minimal close control.
  • Open relative markdown links from rendered markdown in the file browser and in chat messages, through a shared markdownLinks helper and MarkdownLink component.
  • Preinstall the pinned Playwright Chromium binary and its system dependencies into a shared browsers path in the Docker image.
  • Update the Docker config test to cover the new Chromium install step.

Testing

  • Frontend: 1744 tests passed; tsc clean.
  • Backend: 2475 tests passed; tsc clean.
  • pnpm lint -> 0 errors (pre-existing no-explicit-any warnings only).

Add a fullscreen toggle to the desktop file preview header. Fullscreen fills the viewport, hides the header, and shows only a small on-top X to exit.

Keep the mobile preview fullscreen by default with its close button layered above the content, and give the headerless mobile preview the same small on-top X.
Resolve relative and absolute markdown links against the previewed file
and open them in the file browser; external links open in a new tab.
Install Playwright and Chromium into the Manager image with NODE_PATH
and PLAYWRIGHT_BROWSERS_PATH so agents can run browser automation
without a project-local install.
@chriswritescode-dev
chriswritescode-dev merged commit 7fbb755 into main Sep 29, 2026
1 of 2 checks passed
@coderabbitai

coderabbitai Bot commented Sep 29, 2026

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Advanced

Run ID: 1496dc75-a761-4fcc-b9fd-64d69e8207ba

📥 Commits

Reviewing files that changed from the base of the PR and between 304f636 and d328423.

📒 Files selected for processing (16)
  • Dockerfile
  • backend/src/constants.ts
  • backend/test/scripts/docker-config.test.ts
  • docs/configuration/docker.md
  • docs/features/sandboxing.md
  • frontend/src/components/file-browser/FileBrowser.tsx
  • frontend/src/components/file-browser/FilePreview.test.tsx
  • frontend/src/components/file-browser/FilePreview.tsx
  • frontend/src/components/file-browser/MarkdownRenderer.tsx
  • frontend/src/components/file-browser/MobileFilePreviewModal.tsx
  • frontend/src/components/message/MessagePart.test.tsx
  • frontend/src/components/message/MessagePart.tsx
  • frontend/src/components/message/TextPart.tsx
  • frontend/src/components/ui/markdown-link.tsx
  • frontend/src/lib/markdownLinks.test.ts
  • frontend/src/lib/markdownLinks.ts
 _______________________________________
< `NaN` is not a valid user experience. >
 ---------------------------------------
  \
   \   (\__/)
       (•ㅅ•)
       /   づ
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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