McMaster-Baja-Racing / McMaster-Baja-Racing/Better-Data-Viewer
Bring back multiple graphs!
Open
enhancement
HIGH_PRIORITY
- Dominant language
- TypeScript
- Stars
- 3
- Forks
- 2
- PR merge metrics
- No merged PRs in 30d
Description
Feature Description
Multiple graphs is close to being very easy to add back. There are two main issues in the way
- State management: Currently we have a provider that stores chartOptions globally, but we will need this for each chart. It might be as easy as assigning each chart an ID and storing in a list sort of thing, then have a chartOptionsContext for each. The same will need to be done for chartQuery, but they can be separate a bit I think?
- Placing charts: Not sure if you want to add a drag and drop, but that would be the most intuitive and easy to work around, tresting it like widgets. To start on the first part you can skip it and come back, but would be very nice.
Additional Context
Keep in mind, when clicking the edit for each graph, that this will make other graphs disappear and the only graph visible will be the one youre editing, basiclly just making it nice and large. Other than that should be super easy.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.