gajus / gajus/d3js-practical-test
Use d3.scale.category10 to abstract colours
- Dominant language
- JavaScript
- Stars
- 0
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
Now colours are [hardcoded in the CSS](https://github.com/gajus/d3js-practical-test/blob/7a247751d101c27dcaba6d8de6e57bbd87ea0de1/client/src/app/components/TradeChart/styles.scss#L31-L41). This it makes easy for testing (exchanges keep their colour between data iterations), but would not work in production.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start with the linked client/src/app/components/TradeChart/styles.scss lines 31-41, then trace where TradeChart assigns exchange colours. Replace the hardcoded colour handling with d3.scale.category10 while keeping colours stable across data iterations, and verify the chart and relevant tests.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- d3, javascript
- Domain
- data-visualization, frontend
- Issue type
- Refactor
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100