gristlabs/grist-core

chart series limit notification

Open

#851 opened on Feb 13, 2024

View on GitHub
 (1 comment) (0 reactions) (0 assignees)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.

Contributor guide

Tech stack
typescriptreact
Domain
frontend
Issue type
feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
clear
Prerequisites
basic knowledge of Reactfamiliarity with chart components
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
65
Research direction
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.