po-ui / po-angular

Biblioteca de componentes Angular.
https://po-ui.io
MIT License
909 stars 211 forks source link

po-chart: gráfico com mais de 12 itens desaparece da tela #986

Closed camposmichel closed 2 years ago

camposmichel commented 3 years ago

Reprodução

Evidência:

Captura de Tela 2021-08-25 às 11 18 47

StackBlitz: https://stackblitz.com/edit/angular-ivy-cyffd8?file=src%2Fapp%2Fapp.component.ts

Passos para reproduzir:

  1. Utilizar o po-chart
  2. O array de dados deve conter mais de 13 itens

Qual o comportamento atual?

O gráfico está desaparecendo.

Que comportamento você esperava ver?

Algo semelhante ao chart.js

Captura de Tela 2021-08-25 às 12 23 35

Ambiente

jhonyeduardo commented 3 years ago

Bom dia @camposmichel

Os gráficos atualmente tem suporte para 12 series, vamos ajustar para permitir, porém não terá definição de cores. Para utilizar corretamente será preciso definir a cor via PoChatSerie.color.

Esta issue será adicionado no backlog.

rogelho commented 3 years ago

Bom dia @camposmichel

Os gráficos atualmente tem suporte para 12 series, vamos ajustar para permitir, porém não terá definição de cores. Para utilizar corretamente será preciso definir a cor via PoChatSerie.color.

Esta issue será adicionado no backlog.

A limitação hoje ocorre devido ao "PoDefaultColors" ter apenas 12 cores, não? https://github.com/po-ui/po-angular/blob/master/projects/ui/src/lib/services/po-color/po-colors.constant.ts

Nesse caso o mais correto ao meu ver é permitir ao usuário passar uma colorPalette e usar essa paleta ao invés do PoDefaultColors

jhonyeduardo commented 3 years ago

Olá @rogelho

É uma opção também, vamos avaliar a implementação e quando possível liberar na biblioteca, obrigado!

rogelho commented 3 years ago

Olá @rogelho

É uma opção também, vamos avaliar a implementação e quando possível liberar na biblioteca, obrigado!

Beleza Jhony, eu fiz essa correção localmente, vi a que a issue DTHFUI-5314 ainda está no backlog lá no Jira, se for o caso avise que faço o PR aqui, valeu

jhonyeduardo commented 3 years ago

Fique a vontade @rogelho, contribuições são muito bem vindas :)

wsteixeira commented 2 years ago

Olá @alinelariguet,

Esta issue já foi resolvida na PR fix(chart): exibe grafico após a 12 série #1232

alinelariguet commented 2 years ago

Essa correção foi feita e entregue a na versão 6.7.0