Skip to content

fix(docs-site): restore the Copy page split button - #126

Merged
vyctorbrzezowski merged 1 commit into
openclaw:mainfrom
vyctorbrzezowski:brzezowski/fix-copy-page-split-button
Aug 8, 2026
Merged

fix(docs-site): restore the Copy page split button#126
vyctorbrzezowski merged 1 commit into
openclaw:mainfrom
vyctorbrzezowski:brzezowski/fix-copy-page-split-button

Conversation

@vyctorbrzezowski

Copy link
Copy Markdown
Member

Summary

Restore the page-header actions as one split control with two independent targets: the left segment copies the page and the right segment opens the contextual menu.

Cause

The design-token migration applied var(--oc-radius-control) to all four corners of both segments. Combined with the existing 1 px overlap, that rendered two rounded pills pressed together instead of one composed control.

Fix

  • keep var(--oc-radius-control) only on the primary segment outer-left corners
  • keep the same token only on the menu segment outer-right corners
  • retain the existing 1 px overlap so the two borders resolve into one continuous hairline
  • add computed-style and geometry coverage for radii, height, seam width, wrapping, distinct hit targets, menu opening, alignment, and Escape handling across desktop, narrow, light, and dark views

No DOM, interaction behavior, menu content, or shared button styling changed.

Before / After

Current on main Proposed in this draft PR
Current on main Proposed in this draft PR

Theme and narrow-view proof

Proposed in this draft PR — dark Proposed in this draft PR — narrow dark
Proposed in this draft PR in dark mode Proposed in this draft PR at a narrow viewport

Validation

  • npm test — 4 passed
  • DOCS_SITE_PREVIEW_INCLUDE_FIXTURE=1 npm run docs:build:preview — 31 pages built
  • npm run docs:visual — passed against the full renderer build
  • npm run docs:smoke — passed against the full renderer build
  • node scripts/docs-site/r2-prepare.mjs — 84,806 objects prepared
  • node --check scripts/docs-site/assets.mjs
  • node --check scripts/docs-site/smoke.mjs
  • node --check scripts/docs-site/visual-smoke.mjs
  • git diff --check

The regression test fails on the previous styling because both segments compute four rounded corners instead of external corners only.

Scope

Only the docs renderer CSS and its smoke coverage changed. Generated docs/** content and dist/docs-site/** output are not included.

Proof head: 11a2f06c912b41de1b9a985ae618f5346aa0fdbe

@vyctorbrzezowski
vyctorbrzezowski marked this pull request as ready for review August 8, 2026 20:11
@vyctorbrzezowski
vyctorbrzezowski requested a review from a team as a code owner August 8, 2026 20:11
@vyctorbrzezowski
vyctorbrzezowski marked this pull request as draft August 8, 2026 20:15
@vyctorbrzezowski
vyctorbrzezowski marked this pull request as ready for review August 8, 2026 20:16
@vyctorbrzezowski
vyctorbrzezowski merged commit 945fd58 into openclaw:main Aug 8, 2026
5 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.

1 participant