gristlabs/grist-core

chart series limit notification

Open

#851 geöffnet am 13. Feb. 2024

Auf GitHub ansehen
 (1 Kommentar) (0 Reaktionen) (0 zugewiesene Personen)TypeScript (3.613 Stars) (168 Forks)batch import
good first issue

Beschreibung

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.

Contributor Guide

Tech Stack
typescriptreact
Domain
frontend
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
half day
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
fresh
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
basic knowledge of Reactfamiliarity with chart components
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65
Research-Richtung
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.