On focus small icon part is visible in forced colour mode

Pushed the code kindly review it.
Thanks

@shweta__sharma, I've tested the shared MR- MR !6238 mergeable I was able to reproduce the issue but while applying the patch, I was getting an error & patch didn't apply. Error attached.

Seems like a small change and confirmed the issue.

Let's not resort to !important unless it's absolutely necessary.

A better fix would be to override the exact same selector (

  .claro-details[open]>.claro-details__summary:focus::before,

) within the forced-colors media query. You could add this on line 292.

Note that the details.pcss.css file needs quite a bit of refactoring at some point. It doesn't make use of modern CSS, or nesting like many other Claro stylesheets do. But in the meantime, let's keep the stylesheet consistent and keep the same patterns.

Changes are updated as per #7

Thanks

Mithun S → made their first commit to this issue’s fork.

Removed the unnecessary trailing comma in the css files and updated the PR with a commit.

Hi
I've tested the shared MR- MR !6238 mergeable I was able to reproduce the issue and the issue is fixed with the Patch.

Attaching Screenshot

RTBC+1

Thanks @mhercel

Appears suggestions was implemented.

nod_ → closed merge request !6238

Committed 331141c and pushed to 11.x. Thanks!

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