gristlabs/grist-core

chart series limit notification

Open

#851 ouverte le 13 févr. 2024

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)TypeScript (3 613 stars) (168 forks)batch import
good first issue

Description

Hi, when trying to setup a line graph with splitted series, looks like we hit a 50 series limit on the chart and legend.

Would be nice to have a notification so we know that the chart only show a part of the data and we may miss some.

Ability to set higher series limit would be nice too.

Guide contributeur

Stack technique
typescriptreact
Domaine
frontend
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
half day
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
fresh
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
basic knowledge of Reactfamiliarity with chart components
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
65
Direction de recherche
The series limit of 50 is likely enforced in the client side chart rendering code, possibly in files like `app/client/components/chart.ts` or similar. To implement this feature, first locate where the series data is processed and the limit applied, then add a warning notification (e.g., toast or banner) when the limit is reached. Optionally, expose a configuration setting to allow users to increase the limit. Review the UI component used for notifications elsewhere in the app for consistency.