Skip to content

docs(contributing): note when items.color is required#37

Merged
IgorShevchik merged 1 commit into
mainfrom
docs/items-color-avatar-prop
May 13, 2026
Merged

docs(contributing): note when items.color is required#37
IgorShevchik merged 1 commit into
mainfrom
docs/items-color-avatar-prop

Conversation

@IgorShevchik
Copy link
Copy Markdown
Collaborator

Summary

Documents the edge case discovered while landing #36 (Error) and applied to #27 (User): when a component prop is typed via another component (e.g. color?: AvatarProps['color']), the docs generator cannot resolve the palette from the host theme's variants.color, and the dropdown ends up empty. The fix is to declare items.color explicitly in the ::component-code block.

Adds a ::note block right after the existing ### Dropdown Items example in .github/contributing/documentation.md, with pointers to the canonical fixes already in docs/content/docs/2.components/user.md and docs/content/docs/2.components/error.md.

Test plan

  • Skim the rendered contributing page — the note appears under ### Dropdown Items
  • No code / theme / test changes; lint / typecheck / test are no-ops

🤖 Generated with Claude Code

When a prop is typed via another component (e.g.
`color?: AvatarProps['color']`), the docs generator cannot resolve the
palette from the host theme's `variants.color`. The dropdown ends up
empty unless the component-code block declares `items.color` explicitly.
Document this in the `### Dropdown Items` section, with pointers to the
canonical fixes in `user.md` (`275c8b0b`-style fix on `User`) and
`error.md` (`275c8b0b docs(Error): update color list`).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@IgorShevchik IgorShevchik merged commit 574a835 into main May 13, 2026
@IgorShevchik IgorShevchik deleted the docs/items-color-avatar-prop branch May 13, 2026 07:04
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