Skip to content

feat(chart): adiciona possibilidade de formatação do eixo de valor#2359

Merged
felipepetuco merged 1 commit intopo-ui:masterfrom
leonardorocio:po-chart/2358
Feb 3, 2025
Merged

feat(chart): adiciona possibilidade de formatação do eixo de valor#2359
felipepetuco merged 1 commit intopo-ui:masterfrom
leonardorocio:po-chart/2358

Conversation

@leonardorocio
Copy link

@leonardorocio leonardorocio commented Jan 17, 2025

po-chart

#2358


PR Checklist [Revisor]

  • Padrão de Commit (Coeso, de acordo com o que está sendo realizado)
  • Código (Boas práticas, nome de variavéis/métodos, etc.)
  • Testes unitários (Cobre a situação implementada e coverage está mantido)
  • Documentação (Clara, objetiva e com exemplos caso necessário)
  • Samples (A implementação possui exemplo no Labs/Caso de uso)
  • Rodado em navegadores suportados (Chrome, FireFox, Edge)

Qual o comportamento atual?
O po-chart não permite formatação numérica ou monetária do eixo de valor, dificultando a visualização de valores muito altos.

Qual o novo comportamento?
Foram adicionadas novas propriedades nas opções do gráfico que possibilitam informar o tipo dos valores do eixo e outra propriedade para especificar a formatação dos valores.
Agora os valores do gráfico podem ser formatados de forma numérica ou monetária.

Simulação
Esta correção pode ser validada na seção Coffee Ranking da documentação do po-chart. Alguns gráficos foram alterados para ilustrar o uso desta propriedade, facilitando a compreensão do usuário.
Esta correção pode ser validada utilizando o sample labs no portal.

@leonardorocio leonardorocio force-pushed the po-chart/2358 branch 5 times, most recently from f07fdc4 to 49a59e6 Compare January 27, 2025 21:35
O componente po-chart não realiza a formatação do milhares eixo de valor
do gráfico, dificultando a visualização de grandes valores. Adiciona a
propriedade labelType, que indica o tipo de formatação que deve ser
aplicada ao eixo de valor do gráfico.

Fixes po-ui#2358
@felipepetuco felipepetuco merged commit 8709bc6 into po-ui:master Feb 3, 2025
7 of 8 checks passed
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.

4 participants