Repository navigation
Conversation
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The focused styling fix matches the reported behavior and has comprehensive regression coverage.
Review effort: Balanced
Findings: None
What changed in this PR
Exposes copy controls for grouped code examples while hiding duplicate labels.
Changes:
- Adds targeted grouped-code caption styling.
- Adds smoke coverage across themes, viewports, tabs, keyboard activation, and clipboard feedback.
| File | Description |
|---|---|
scripts/docs-site/docs-layout.css |
Shows grouped copy controls and hides duplicate labels. |
scripts/docs-site/smoke.mjs |
Verifies the generated CSS contract. |
scripts/docs-site/visual-smoke.mjs |
Tests grouped copying across responsive layouts and themes. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Related: openclaw/openclaw#165115
What Problem This Solves
Fixes missing copy buttons when readers select a tabbed code example, including the Slack app manifests.
User Impact
Readers can copy the selected example with a visible, keyboard-accessible control and receive the existing copy feedback.
Why This Change Was Made
Expose grouped captions in the docs layout while hiding only their duplicate label. Keep the existing clipboard runtime and tab behavior.
Evidence
Before
After
Screenshots use a local 667px dark preview. Before restores the original caption-hiding CSS; JSON indentation is intentionally unchanged in this PR and tracked separately in #165114. Screenshot assets live on a separate evidence branch and are excluded from this PR's diff.