Skip to content

feat(docs): add missing component variants examples #2123

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
beatrizmilhomem wants to merge 1 commit into
base: main
Choose a base branch
from
Open

feat(docs): add missing component variants examples #2123

beatrizmilhomem wants to merge 1 commit into from

Conversation

@beatrizmilhomem
Copy link
Contributor

@beatrizmilhomem beatrizmilhomem commented Nov 25, 2025

📝 Descrição

Este PR adiciona arquivos de variantes faltantes para diversos componentes do Shoreline, além de corrigir e completar arquivos existentes.

✨ Alterações

Novos arquivos criados (11):

  • checkboxgroup_variants.tsx - 5 variantes (default, horizontal, with error, with description, horizontal with error)
  • collection_variants.tsx - 3 variantes (default, with content, with aligned row)
  • confirmationmodal_variants.tsx - 2 variantes (default, with custom text)
  • contextualhelp_variants.tsx - 5 variantes (default, top-start, bottom-start, top-end, bottom-end)
  • daterangepicker_variants.tsx - 5 variantes (default, disabled, readonly, with error, controlled)
  • page_variants.tsx - 4 variantes (default, with button, narrow, wide)
  • radiogroup_variants.tsx - 5 variantes (default, horizontal, with error, with description, horizontal with error)
  • spinner_variants.tsx - 2 variantes (default, with custom description)
  • tablist_variants.tsx - 3 variantes (default, with three tabs, with multiple panels)
  • table_variants.tsx - 3 variantes (default, with tags, with custom widths)

Arquivos atualizados:

  • emptystate_variants.tsx - Adicionadas 3 variantes (with illustration, with actions, complete)
  • daterangepicker_variants.tsx - Corrigidos erros de props (disabled→isDisabled, readOnly→isReadOnly)
  • datepicker_variants.tsx - Adicionadas 2 variantes (with error, controlled)
  • README.md - Documentação completa de todas as novas variantes

📊 Impacto

  • Total de componentes: 35
  • Total de variantes: 139+
  • Todos os arquivos seguem o padrão estabelecido
  • Nenhum erro de linting
  • Exemplos prontos para serem usados na documentação

✅ Checklist

  • Código segue o padrão do projeto
  • Sem erros de linting
  • README.md atualizado
  • Commits seguem o conventional commits
  • Todas as props principais dos componentes estão representadas

@beatrizmilhomem
feat(docs): add missing component variants examples
67a2587 
- Add 11 new variant files: checkboxgroup, collection, confirmationmodal,
  contextualhelp, daterangepicker, page, radiogroup, spinner, tablist, table
- Update emptystate_variants.tsx with illustration and action variants
- Fix daterangepicker_variants.tsx props and add error/controlled variants
- Update datepicker_variants.tsx with error and controlled variants
- Update README.md with all new variants documentation
- Total: 139+ variants across 35 components
@beatrizmilhomem beatrizmilhomem requested a review from a team as a code owner November 25, 2025 16:50
@vercel
Copy link
Contributor

vercel bot commented Nov 25, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
shoreline-docs Error Error Nov 25, 2025 4:53pm

@beatrizmilhomem beatrizmilhomem self-assigned this Nov 25, 2025
@beatrizmilhomem beatrizmilhomem moved this to Active in Shoreline Nov 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lucasaarcoverde lucasaarcoverde Awaiting requested review from lucasaarcoverde

At least 1 approving review is required to merge this pull request.

Assignees

@beatrizmilhomem beatrizmilhomem

Labels

None yet

Projects

Shoreline
Status: Active

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@beatrizmilhomem