Skip to content

Remove screenshot tool and footer controls#509

Merged
BunsDev merged 1 commit into
mainfrom
feature/remove-screenshot-footer-and-space
May 11, 2026
Merged

Remove screenshot tool and footer controls#509
BunsDev merged 1 commit into
mainfrom
feature/remove-screenshot-footer-and-space

Conversation

@BunsDev
Copy link
Copy Markdown
Member

@BunsDev BunsDev commented May 11, 2026

Summary

  • Remove the screenshot tool UI, floating footer controls, and screenshot keyboard shortcut from the chat shell and widget shell.
  • Delete screenshot capture state/helpers and remove the unused html-to-image dependency.
  • Remove the desktop captureWindow bridge/IPC surface that only supported the screenshot tool.
  • Add a regression check that the chat shell no longer exposes screenshot controls or shortcuts.

Testing

  • bun run test src/routes/-_chat.test.ts
  • bun fmt
  • bun lint (0 errors; reports existing unused-symbol warnings outside this change)
  • bun typecheck
  • bun run build
  • Rendered production preview at http://127.0.0.1:4177/ and verified no screenshot button, overlay, screenshot text, or fixed bottom-right footer on desktop/mobile.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 11, 2026

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

Project Deployment Actions Updated (UTC)
okcode-marketing Ready Ready Preview May 11, 2026 9:02am
v0-compute-the-platform-to-build Ready Ready Preview, Open in v0 May 11, 2026 9:02am

@BunsDev BunsDev force-pushed the feature/remove-screenshot-footer-and-space branch from bef8f78 to 0577e6d Compare May 11, 2026 09:01
@BunsDev BunsDev changed the title Remove OpenClaw screenshot and integration code Remove screenshot tool and footer controls May 11, 2026
@BunsDev BunsDev requested a review from Copilot May 11, 2026 09:02
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the (previously integrated) screenshot capture feature across the web chat shells and the desktop bridge surface area that supported it, along with the now-unused html-to-image dependency.

Changes:

  • Removes screenshot UI elements (tool/overlay/button), footer controls, and the chat shell keyboard shortcut.
  • Deletes screenshot-related state/helpers/tests and drops the html-to-image dependency.
  • Removes the desktop captureWindow bridge/IPC handler and updates contracts accordingly; adds a regression test ensuring screenshot affordances aren’t present in the chat shell/widget shell sources.

Reviewed changes

Copilot reviewed 12 out of 13 changed files in this pull request and generated no comments.

Show a summary per file
File Description
packages/contracts/src/ipc.ts Removes captureWindow from the DesktopBridge contract.
bun.lock Removes html-to-image from the lockfile.
apps/web/src/screenshotStore.ts Deletes the Zustand store backing screenshot selection mode.
apps/web/src/routes/-_chat.test.ts Adds a regression test asserting screenshot UI/shortcuts aren’t present in chat/widget shells.
apps/web/src/routes/_chat.tsx Removes screenshot imports, shortcut handling, and UI render points (tool + floating footer button).
apps/web/src/lib/screenshotCapture.ts Deletes DOM/native capture helper logic (now unused).
apps/web/src/lib/screenshotCapture.test.ts Deletes tests for the removed capture helpers.
apps/web/src/components/widget/ChatWidgetShell.tsx Removes screenshot tool/button from the widget shell.
apps/web/src/components/settings/SettingsUi.tsx Minor import formatting change (no functional impact).
apps/web/src/components/ScreenshotTool.tsx Deletes screenshot overlay/tool/button component implementation.
apps/web/package.json Removes html-to-image dependency from the web app.
apps/desktop/src/preload.ts Removes captureWindow from the exposed desktop bridge implementation.
apps/desktop/src/main.ts Removes the desktop:capture-window IPC channel/handler.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@BunsDev BunsDev merged commit dd37002 into main May 11, 2026
14 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XL 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.

2 participants