Skip to content

Feature/bar chart dataset variant - #1121

Merged
ezequias21 merged 4 commits into
mainfrom
feature/bar-chart-dataset-variant
Jul 31, 2026
Merged

Feature/bar chart dataset variant#1121
ezequias21 merged 4 commits into
mainfrom
feature/bar-chart-dataset-variant

Conversation

@lucasn4s

Copy link
Copy Markdown
Collaborator

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 possibilidade de informar variante por dataset no BarChart.

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?

  • Acesse o BarChart e verifique que agora, ao passar um variant em um dataset com alguma variante válida do cuida, as barras correspondentes àquele dataset ficam com a variante enviada, independentemente da variante do próprio componente;
  • Deixe seu like e se inscreva no canal.

5 - Imagem ou exemplo de uso:

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 31, 2026
@greptile-apps

greptile-apps Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

O PR adiciona suporte à definição de uma variante de cor específica por dataset no BarChart.

  • Propaga variant dos dados de entrada para os datasets internos.
  • Usa a cor 400 da paleta solicitada, mantendo a paleta geral como fallback.
  • Atualiza a documentação com um exemplo usando a variante amber.
  • Atualiza a versão do pacote para 3.161.0.

Confidence Score: 4/5

A atualização reativa da variante por dataset precisa ser corrigida antes do merge, pois alterações aninhadas deixam as barras com cores obsoletas.

A variante é copiada e aplicada somente quando o watcher de data recompõe os datasets; como esse watcher não observa alterações aninhadas, mutações no novo campo não atualizam backgroundColor, e os testes atuais não cobrem esse comportamento.

Files Needing Attention: src/components/BarChart.vue

Important Files Changed

Filename Overview
src/components/BarChart.vue Implementa a seleção de cor por variante do dataset, mas alterações aninhadas dessa propriedade não acionam a recomposição das cores e o novo caminho não possui cobertura específica.
docs/components/charts/bar-chart.md Adiciona ao exemplo um dataset com a variante amber.
package.json Atualiza a versão do pacote de 3.160.1 para 3.161.0.
package-lock.json Mantém a versão do lockfile sincronizada com o manifesto.

Reviews (1): Last reviewed commit: "Sobe versão do projeto" | Re-trigger Greptile

Comment thread src/components/BarChart.vue
Comment thread src/components/BarChart.vue
@ezequias21
ezequias21 merged commit da070ac into main Jul 31, 2026
8 checks passed
@ezequias21
ezequias21 deleted the feature/bar-chart-dataset-variant branch July 31, 2026 19:57
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