Skip to content

Remove Molty attachment affordance#51

Merged
vincentkoc merged 1 commit into
openclaw:mainfrom
vyctorbrzezowski:brzezowski/remove-molty-attach
Jun 17, 2026
Merged

Remove Molty attachment affordance#51
vincentkoc merged 1 commit into
openclaw:mainfrom
vyctorbrzezowski:brzezowski/remove-molty-attach

Conversation

@vyctorbrzezowski

Copy link
Copy Markdown
Contributor

Summary

Removes the paperclip attachment affordance from the Molty docs chat composer.

This PR keeps the send button and composer behavior intact while avoiding an upload affordance for a capability that openclaw/ask-molty does not process yet.

Validation

  • npm run docs:build:preview
  • node --check scripts/docs-site/build.mjs
  • node --check scripts/docs-site/assets.mjs
  • node --check scripts/docs-site/smoke.mjs
  • git diff --check
  • generated HTML/CSS assertion that .docs-chat-attach is absent and [data-chat-submit] remains present
  • in-app browser DOM check on http://127.0.0.1:4176/: attachNodes: 0, paperclipIcons: 0, sendButtons: 1

@vyctorbrzezowski vyctorbrzezowski marked this pull request as ready for review June 16, 2026 21:09
@vyctorbrzezowski vyctorbrzezowski requested a review from a team as a code owner June 16, 2026 21:09
@vincentkoc vincentkoc force-pushed the brzezowski/remove-molty-attach branch from 5d4663d to d248d0a Compare June 17, 2026 00:11
@vincentkoc vincentkoc force-pushed the brzezowski/remove-molty-attach branch from d248d0a to e0b8b54 Compare June 17, 2026 00:15
@vincentkoc

Copy link
Copy Markdown
Member

Land-ready after maintainer review.

Summary:

  • removes the unused Molty attachment affordance while retaining the message composer.

Verification:

  • npm run docs:build
  • npm run docs:smoke
  • Chrome check: no attachment affordance or paperclip; one Send control remains
  • autoreview: clean, no accepted/actionable findings

@vincentkoc vincentkoc merged commit 44944b9 into openclaw:main Jun 17, 2026
4 checks 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