Skip to content

feat: adiciona slot append em checkbox e checkboxgroup#1097

Merged
ezequias21 merged 3 commits into
mainfrom
feature/checkbox-append-slot
Jul 9, 2026
Merged

feat: adiciona slot append em checkbox e checkboxgroup#1097
ezequias21 merged 3 commits into
mainfrom
feature/checkbox-append-slot

Conversation

@jvictordev1

@jvictordev1 jvictordev1 commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Por favor, verifique se o seu pull request está de acordo com o checklist abaixo:

  • A implementação feita possui testes (Caso haja um motivo para não haver testes/haver apenas testes de snapshot, descrever abaixo)
  • A documentação no mdx foi feita ou atualizada, caso necessário
  • O eslint passou localmente

1 - Resumo

Adiciona slot append no Checkbox e CheckboxGroup para ser possível adicionar informações no lado direito do checkbox.

2 - Tipo de pull request

  • 🧱 Novo componente
  • ✨ Nova feature ou melhoria
  • 🐛 Fix
  • 👨‍💻 Refatoração
  • 📝 Documentação
  • 🎨 Estilo
  • 🤖 Build ou CI/CD

3 - Esse PR fecha alguma issue? Favor referenciá-la

Não

4 - Quais são os passos para avaliar o pull request?

  • Execute o comando npm run test e garanta que todos os testes passaram com sucesso
  • Valide o slot append no checkbox individual enviando algum elemento para o slot #append e verifique se ele aparece no componente
  • Adicione um conteúdo ao slot #append desestruturando o objeto da opção ( v-slot="{ option }" ). Verifique se o conteúdo é inserido em todos os itens do grupo de forma consistente com os dados passados em cada option .
  • Declare um slot nomeado com base no valor de uma das opções (ex: #append-valorDaOpcao). Garanta que o conteúdo desse slot seja renderizado exclusivamente no checkbox correspondente.

5 - Imagem ou exemplo de uso:

image

6 - Esse pull request adiciona breaking changes?

  • Sim
  • Não

@github-actions github-actions Bot added the 🐛 Bug Algo não está funcionando label Jul 8, 2026
@greptile-apps

greptile-apps Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Adiciona um slot append ao Checkbox e ao CheckboxGroup, permitindo renderizar conteúdo adicional à direita de cada opção (ex.: badges, ícones). O CheckboxGroup encaminha o slot para cada filho passando o objeto option como dado escopado.

  • Checkbox.vue: novo <div class=\"cds-checkbox__append\"> com margin-left: auto envolve o slot; o wrapper é renderizado incondicionalmente, mesmo quando o slot está vazio.
  • CheckboxGroup.vue: encaminha o slot append com v-if=\"$slots.append\", mas inclui {{ option.append }} como fallback que nunca é alcançado devido ao próprio guard.
  • Documentação, testes e snapshots foram atualizados de forma consistente com as mudanças.

Confidence Score: 4/5

A feature é aditiva e não quebra comportamentos existentes; as observações são pequenos pontos de qualidade nos componentes principais.

O slot append funciona corretamente e os testes cobrem os casos esperados. O wrapper div vazio sempre presente no Checkbox.vue e o fallback de slot inalcançável no CheckboxGroup.vue são oportunidades de melhoria, mas não introduzem comportamentos errados.

Os dois arquivos de componentes (src/components/Checkbox.vue e src/components/CheckboxGroup.vue) merecem uma segunda olhada nos pontos levantados antes do merge.

Important Files Changed

Filename Overview
src/components/Checkbox.vue Adiciona slot append com wrapper div e estilo margin-left: auto; o div é sempre renderizado mesmo quando o slot está vazio.
src/components/CheckboxGroup.vue Encaminha o slot append para cada CdsCheckbox filho com option como dado escopado; contém fallback {{ option.append }} que é código morto por causa do guard v-if="$slots.append".
src/tests/Checkbox.spec.ts Adiciona teste de renderização do slot append com verificação de presença e conteúdo de texto.
src/tests/CheckboxGroup.spec.ts Adiciona teste de slot escopado verificando que option é corretamente passado para cada item do grupo.
docs/components/forms/checkbox-group.md Adiciona exemplo de uso do slot append com CdsBadge e seção de Slots na documentação do CheckboxGroup.
package.json Bump de versão de 3.158.4 para 3.159.0 (minor), coerente com nova feature.

Reviews (1): Last reviewed commit: "feat: adiciona slot append em checkbox e..." | Re-trigger Greptile

Comment thread src/components/Checkbox.vue
Comment thread src/components/CheckboxGroup.vue Outdated
Comment thread docs/components/forms/checkbox-group.md Outdated
Comment thread docs/components/forms/checkbox-group.md
Comment thread src/components/CheckboxGroup.vue
@ezequias21
ezequias21 merged commit 6c756dc into main Jul 9, 2026
8 checks passed
@ezequias21
ezequias21 deleted the feature/checkbox-append-slot branch July 9, 2026 12:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 Bug Algo não está funcionando

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants