Skip to content

Support using custom color names#66

Open
sehilyi wants to merge 3 commits intomasterfrom
sehilyi/color-fn
Open

Support using custom color names#66
sehilyi wants to merge 3 commits intomasterfrom
sehilyi/color-fn

Conversation

@sehilyi
Copy link
Copy Markdown
Member

@sehilyi sehilyi commented Mar 12, 2025

Fix #60 (review)

import { GetColorName } from 'color-hex-to-color-name';
// ...
<AltGoslingComponent
  spec={spec}
  // user can define their own color names
  customColorNames={(hex: string) => GetColorName(hex)}
/>

@thomcsmits Do you think this is something that we can add to the AltGosling?

@sehilyi sehilyi changed the title [draft] Support including custom color names Support using custom color names Mar 13, 2025
@sehilyi sehilyi requested review from thomcsmits and removed request for thomcsmits March 13, 2025 20:50
@sehilyi
Copy link
Copy Markdown
Member Author

sehilyi commented Mar 14, 2025

I think this is a lower priority since we figured out the issue of color names.

@thomcsmits
Copy link
Copy Markdown
Collaborator

Yes, but still great support to have! I'll take a look soon!

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