Skip to content

docs: keep the rules for a UI icon and leave the icons to the site - #9

Merged
rqbazan merged 1 commit into
mainfrom
docs/ui-icon-rules
Oct 17, 2026
Merged

rqbazan merged 1 commit into
mainfrom
docs/ui-icon-rules

Conversation

@rqbazan

@rqbazan rqbazan commented Oct 17, 2026 •

Copy link
Copy Markdown
Member

Summary

The guideline states rules, not one site's specifics. The Copy and Copied icons are www's, so iconography.md keeps the rules for drawing a UI icon and drops the table that listed them.

 ## UI icons

+This guideline holds the rules for drawing a UI icon, not the icons: a site draws the icons it needs and keeps their geometry in its own specs. Every UI icon follows these rules:
+
 - `viewBox 0 0 16 16`, shown at 14px. Stroke 1.5, no fill, …
 - `currentColor`: `dim` at rest, `ink` on hover.
 - Decorative, so `aria-hidden="true"`. …
 - Outlined, so a UI icon never reads as a Sign, which is filled.

-| Icon | Use | Geometry |
-| Copy | Copies a command. | `<rect …/>` and `<path …/>` |
-| Copied | Replaces Copy for a moment after a copy. | `<path …/>` |
-
-A new icon follows the stroke rules. Before it ships, set it at 14px …
+Before an icon ships, set it at 14px beside every Project's `brand/sign.svg` and both Class stamps: …

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

  • Before: grep -rn -i "copied\|5\.75" skills/ → two rows in iconography.md.
    After: no match. pnpm check exits 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

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-bot

changeset-bot Bot commented Oct 17, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: db67580

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@variablelab/brand Patch

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

@rqbazan
rqbazan merged commit 908b3bb into main Oct 17, 2026
2 checks passed
@rqbazan
rqbazan deleted the docs/ui-icon-rules branch October 17, 2026 20:22
@vland-bot vland-bot Bot mentioned this pull request Oct 17, 2026
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