Wrong syntax for hsl color values

Very good find - looks like an easy fix as well. I'll queue it up in my workload to knock out.

Ah, let me make sure I understand, because I believe the IS is missing a key context.

Right now, hsl is looking for hsl(0,0,0)|Color, but you need hsl(0,0%,0%)|Color (the inclusion of percentages for second and third parameters), correct?

In Drupal 10 > CKEditor 5 > Font Colors :

In the Font Colors box you have to enter e.g. the following color values to be correct:

hsl(0,0%,0%)|Black
hsl(0,0%,30%)|Dim grey
hsl(0,0%,60%)|Grey
hsl(0,0%,90%)|Light grey
hsl(0,0%,100%)|White
hsl(0,75%,60%)|Red
hsl(30,75%,60%)|Orange
hsl(60,75%,60%)|Yellow
hsl(90,75%,60%)|Light green
hsl(120,75%,60%)|Green
hsl(150,75%,60%)|Aquamarine
hsl(180,75%,60%)|Turquoise
hsl(210,75%,60%)|Light blue
hsl(240,75%,60%)|Blue
hsl(270,75%,60%)|Purple

So, indeed in your guidelines below the box you need hsl(0,0%,0%)|Color. Inclusion of percentages for second and third parameters is correct.

Webbeh → closed merge request !11

Webbeh → closed merge request !10

Webbeh → closed merge request !12

Don't judge - I'm awful with GitLab UI for pushing to a branch. Fix in field description in place. Thanks.

@Webbeh

Thank you! Unfortunately I cannot test the 2.0.0-dev because I'm only working with a production site.

However if this translation issue 🐛 Buttons are not translatable Fixed could be fixed, then I assume that a new stable release instead of 2.0.0-beta 1 could be put out.

redseujac wrote in #20

I will temporary remove the 2.0.0-beta 1 and replace it with the newest 2.0.0-dev so I can check the fix.

Well, I have done so and actually the issue is fixed in 2.0.0-dev.

Congrats!

Now only try to fix this translation issue 🐛 Buttons are not translatable Fixed and it's done.

Automatically closed - issue fixed for 2 weeks with no activity.