Skip to content

add function that set color about heatmap#25

Open
mizuirorivi wants to merge 4 commits intomainfrom
rivi/feature/change_color_on_heatmap
Open

add function that set color about heatmap#25
mizuirorivi wants to merge 4 commits intomainfrom
rivi/feature/change_color_on_heatmap

Conversation

@mizuirorivi
Copy link
Contributor

@mizuirorivi mizuirorivi commented Dec 15, 2024

ユーザーが定義した色でコンポーネントを変えられるようにする基礎を作成した
とりあえず今の所heatmapだけ
この機能がいいかどうか感じたこと教えて
また、コードのレビューお願いしまっす

@mizuirorivi
Copy link
Contributor Author

"color": "^4.2.3",
"firebase": "^11.0.2",
"framer-motion": "^11.14.4",
"js-cookie": "^3.0.5",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

いらないかも

/>
</FormControl>
<FormControl>
<FormLabel>color map(heatmap)</FormLabel>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

heatmapというのは明示的に書く必要があるか


const level3Color = darkenHexColor(userColor, 40);

const getColor = (count: number | undefined): string => {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

darkenHexColorはコンポーネントの外に定義されるのに getColorはコンポーネント内に定義される

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants