Skip to content

Commit 7ee47b7

Browse files
committed
Add TODO to handle sign colors with the theme
1 parent c89f42c commit 7ee47b7

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

projects/js-packages/charts/src/components/leaderboard-chart/leaderboard-chart.tsx

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -140,6 +140,10 @@ export const LeaderboardChart: FC< LeaderboardChartProps > = ( {
140140
className,
141141
style,
142142
} ) => {
143+
// TODO: Integrate with ThemeProvider:
144+
// 1. Use theme.colors for primaryColor/secondaryColor defaults
145+
// 2. Get delta sign colors from theme instead of hardcoding
146+
// 3. Add useChartTheme() hook like other chart components
143147
const signColors = [ '#D63638', '#757575', '#008A20' ];
144148

145149
const chartStyle = {

0 commit comments

Comments
 (0)