CKEditor 5 has its own border color and is not using Claro's colors

pere orga → changed the visibility of the branch 3478281-ckeditor-5-has to hidden.

pere orga → changed the visibility of the branch 11.x to hidden.

pere orga → changed the visibility of the branch 3478281-ckeditor-5-has to active.

Pushed these changes:

/* Make border color consistent. */
.ck {
  --ck-color-base-border: var(--input-border-color);
}

Don't see an MR opened.

UI changes should have before/after screenshots which is part of the standard template, so missing sections should be added back

pere orga → changed the visibility of the branch 3478281-ckeditor-5-has to hidden.

pere orga → changed the visibility of the branch 3478281-opcache-typos to hidden.

pere orga → changed the visibility of the branch 3478281-ckeditor-5-has to active.

@smustgrave, MR created, sorry: https://git.drupalcode.org/project/drupal/-/merge_requests/9728

Adding before/after screenshot changes. The field names are different in the screenshot, but the color issue applies.

pere orga → closed merge request !9729

Thanks. Recommend going forward to leave all sections of the issue template. Even if it doesn’t apply can use “N/A” or none.

But screenshots were added so removed the tag.

Did very fix is addressing issue described.

The css variabled used is defined in claro but this is the main module CSS file. If a claro variable is used the CSS file should live in Claro.

That makes sense.

I see there is no CKEditor-dedicated file in Claro theme. I'm wondering if the right file to put this should be css/theme/field-ui.admin.pcss.css, or maybe a new file css/theme/ckeditor.admin.pcss.css should be created instead?

nod_ → closed merge request !9728

all good thx

Committed and pushed 9c0bd3775ca to 11.x and 562fa1c9b6b to 11.0.x and 9dd570757ea to 10.4.x and 229a850b786 to 10.3.x. Thanks!

Very nice! 👏

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