Repository navigation
docs: keep the rules for a UI icon and leave the icons to the site - #9
Merged
Merged
Conversation
The Copy and Copied icons, with their geometry, belong to the site that uses them. The Iconography chapter keeps only the rules every UI icon follows and the check before one ships. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01An6o9bF4mjtTFxdLfPtsqa
🦋 Changeset detectedLatest commit: db67580 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Merged
This was referenced Oct 17, 2026
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.
Summary
The guideline states rules, not one site's specifics. The Copy and Copied icons are www's, so
iconography.mdkeeps the rules for drawing a UI icon and drops the table that listed them.No other chapter names these icons. The geometry moves to www's
docs/design.md, where the Copy button is specified (withvariablelab/www#3).One patch changeset.
Evidence
grep -rn -i "copied\|5\.75" skills/→ two rows iniconography.md.After: no match.
pnpm checkexits 0.Merge Danger
Door: two-way
Blast Radius: docs
Agents install the skill from
main, so after merging they no longer find the Copy and Copied geometry in the skill. www keeps it in its own docs and code; no other Project uses these icons.🤖 Generated with Claude Code
https://claude.ai/code/session_01An6o9bF4mjtTFxdLfPtsqa