elm-ui-widgets icon indicating copy to clipboard operation
elm-ui-widgets copied to clipboard

Trouble with custom pallette

Open ncc-swann opened this issue 1 year ago • 5 comments

Hi

I am using elm-ui-widgets and generally find it awesome and very useful, so thank you!

I am currently trying to change the default pallette to something custom, but colors do not seem to be represented correctly on screen. I am only changing the primary color of the default pallette for now, leaving the rest alone, but I am experiencing jade represented as light brownish, a shade of blue being represented as green etc.

Do you have any insight into what might be going on?

Best regards Nicolai

ncc-swann avatar Jun 15 '23 11:06 ncc-swann

Hi. Could you please create an ellie showcasing the wierd behavior?

ncc-swann @.***> schrieb am Do., 15. Juni 2023, 13:10:

Hi

I am using elm-ui-widgets and generally find it awesome and very useful, so thank you!

I am currently trying to change the default pallette to something custom, but colors do not seem to be represented correctly on screen. I am only changing the primary color of the default pallette for now, leaving the rest alone, but I am experiencing jade represented as light brownish, a shade of blue being represented as green etc.

Do you have any insight into what might be going on?

Best regards Nicolai

— Reply to this email directly, view it on GitHub https://github.com/Orasund/elm-ui-widgets/issues/93, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABYRMLHWG2CBEPLC6JCS3LDXLLUSNANCNFSM6AAAAAAZHVZRF4 . You are receiving this because you are subscribed to this thread.Message ID: @.***>

Orasund avatar Jun 15 '23 11:06 Orasund

Of course. !! Now with updated link to the Ellie !!

Here is a link to the Ellie: https://ellie-app.com/n62jNwkctCHa1 And here is a link to how the CIELCH color used as primary color should look like: https://convertingcolors.com/cielch-color-42_62.639_291.077.html?search=CIELCH(42,%2062.639,%20291.077)#00ABAB

ncc-swann avatar Jun 15 '23 11:06 ncc-swann

I'm not sure why this happens. I'm actually relying on noahzgordon/elm-color-extra to convert from LAB to RGB. The conversion from LCH to LAB should be correct (it's just one line, that I double-checked just now).

Also, I never had any problems with that function – I have actually been using it a lot. However, I never tried to directly create a CIELCH color, like you are doing. So my guess is that some factor is wrong somewhere. Maybe the range of my implementation is different to the range used by convertingcolors.com?

I am considering this a niche-problem, and I currently don't have time to fix this. So I will leave it as is.

If you need a fix, please try to find a solution and open a PR.

Orasund avatar Jun 21 '23 05:06 Orasund

Thank you for looking into it. People at my company don't like the default purple look, so I will have to come up with a solution eventually.

ncc-swann avatar Jun 21 '23 06:06 ncc-swann

Use avh4/elm-color to define colors. You can use any rgb value.

Orasund avatar Jun 21 '23 08:06 Orasund