Skip to content

fix(multiselect): corrige emissão do valor 'on' ao clicar no checkbox - #1120

Merged
lucasn4s merged 2 commits into
mainfrom
hotfix/multiselect-selected-value-events
Jul 31, 2026
Merged

fix(multiselect): corrige emissão do valor 'on' ao clicar no checkbox#1120
lucasn4s merged 2 commits into
mainfrom
hotfix/multiselect-selected-value-events

Conversation

@ezequias21

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

  • Corrige a seleção de opções no Multiselect ao clicar no checkbox visual.
    • Antes, ao clicar no checkbox de uma opção, o componente emitia o valor "on" em vez do objeto da opção selecionada.
    • Agora, o checkbox emite corretamente o objeto da opção, tanto na seleção quanto na deseleção.
    • O comportamento do clique no texto da opção permanece inalterado.

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

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

  • Abra um Multiselect com opções disponíveis.
  • Clique no checkbox visual de uma opção e verifique que o objeto da opção é emitido (não o valor "on").
  • Clique no checkbox novamente e verifique que a opção é removida.
  • Clique no texto da opção e confirme que a seleção continua funcionando como antes.
  • Verifique que o checkmark visual acompanha o estado selecionado/deselecionado.

5 - Imagem ou exemplo de uso:

6 - Esse pull request adiciona breaking changes?

  • Sim
  • Não

O preventDefault no clique do input nativo impede que o toggle do
checkbox dispare eventos input/change com target.value 'on', que
borbulhavam até handlers do consumidor via .
@github-actions github-actions Bot added the 🐛 Bug Algo não está funcionando label Jul 31, 2026
@greptile-apps

greptile-apps Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Corrige o comportamento do checkbox visual do Multiselect.

  • Substitui o v-model por uma vinculação unidirecional ao estado isSelected.
  • Impede que o clique sintético no input propague o valor nativo "on".
  • Remove o tratamento manual de seleção para reutilizar o fluxo normal da opção.

Confidence Score: 5/5

O pull request parece seguro para merge, sem falhas funcionais ou de segurança acionáveis identificadas.

O label continua propagando o clique para o mecanismo de seleção da opção, enquanto o clique sintético subsequente no input é interrompido para evitar a emissão indevida do valor "on".

Important Files Changed

Filename Overview
src/components/Multiselect.vue A alteração evita a emissão do valor nativo do checkbox e mantém a seleção pelo clique propagado no label, sem falhas acionáveis identificadas.

Reviews (1): Last reviewed commit: "fix(multiselect): corrige emissão do val..." | Re-trigger Greptile

@lucasn4s
lucasn4s merged commit c1b8eee into main Jul 31, 2026
8 checks passed
@lucasn4s
lucasn4s deleted the hotfix/multiselect-selected-value-events branch July 31, 2026 19:38
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