Northeastern-Electric-Racing / Northeastern-Electric-Racing/FinishLine
[Gantt Chart Improvements] - Reduce Rerendering
Nobody has claimed this yet.
- Dominant language
- TypeScript
- Stars
- 36
- Forks
- 9
- Avg merge
- 4d 21h
- Merged PRs (30d)
- 11
Description
Description
We need to significantly reduce the amount of rerendering going on with the current frontend. A good way to do this would be by removing showWorkPackagesMap from the ProjectGanttChartPage level, and ideally moving it down to around the GanttTaskBarDisplay file (much lower in the tree structure for components). If you notice other ways to reduce rerendering along the way, do let me know!
Acceptance Criteria
Page rerenders significantly less.
Proposed Solution
As mentioned, look into moving showWorkPackagesMap down into the lowest componenet possible.
Mocks
No response
Contributor guide
No contributing guide indexed for this repository
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.
Research direction
Start at ProjectGanttChartPage and trace how showWorkPackagesMap reaches GanttTaskBarDisplay and the lower component tree. Measure the current rerendering behavior, then verify that moving the state or prop closer to its use reduces page rerenders without changing the chart behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- frontend, performance
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100