Skip to content

docs: remove external design-system attributions - #7

Merged
sriramveeraghanta merged 2 commits into
mainfrom
docs/remove-external-attributions
Aug 25, 2026
Merged

docs: remove external design-system attributions#7
sriramveeraghanta merged 2 commits into
mainfrom
docs/remove-external-attributions

Conversation

@anmolsinghbhatia

Copy link
Copy Markdown
Contributor

Description

Removes citations of other design systems from propel's internal guidance docs, so the conventions stand on their own reasoning.

  • packages/propel/AGENTS.md — dropped kumo-ui.com as the cited source of the variant/size vocabulary (rules 6b, 6c, "Why these names").
  • apps/docs/CLAUDE.md, apps/docs/PRODUCT.md — dropped the Radix/Stripe/shadcn/Tailwind comparisons, describing the intended qualities directly instead.

Item 2 of "Why these names" had no justification beyond the Kumo precedent, so it was given one rather than left bare. The rule is unchanged; only its rationale is new — worth a read.

Type of Change

  • Documentation update

Rules 6b, 6c and the "Why these names" list cited kumo-ui.com as the source of
propel's `variant`/`size` vocabulary. The rules stand on their own reasoning, so
drop the attributions.

Item 2 of "Why these names" had no justification beyond the Kumo precedent, so
it gets one: a caller picks exactly one look, and a separate `color` prop would
only make contradictory pairs representable -- the same failure item 1 already
describes for the `prominence`/`tone` split.
CLAUDE.md and PRODUCT.md positioned the docs site by naming Radix, Stripe,
shadcn and Tailwind. Describe the qualities directly instead, so the guidance
does not lean on another project's identity.
@anmolsinghbhatia
anmolsinghbhatia requested a review from vihar August 25, 2026 10:06
@anmolsinghbhatia anmolsinghbhatia self-assigned this Aug 25, 2026

@bhaveshraja bhaveshraja left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only comments level changes are a addressed.

@sriramveeraghanta
sriramveeraghanta merged commit 8786a80 into main Aug 25, 2026
1 of 2 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.

3 participants