Close icon is ovrlapping the title text in modal in claro

Added the css so that close icon does not overlap the title.

MR appears to have issues.

But not sure we want to add that css to the global body selector.

The MR issue has been resolved . I have added the css in body selector because in the olivero theme also this css added in body selector.Please review .

Hello @Sandeep, I've successfully reproduced the issue you were talking about, and the patch provided by you also got applied successfully. After applying the patch the issue is resolved now and there is a breathing space between the section title and the close button. So moving it to RTBC++.
Attaching screenshots before and after applying the patch for reference!

I have a question? Why're we using Claro as a frontend theme?
Claro is an admin theme. Do we support using Claro as frontend theme?
FYI: This is working fine with Olivero frontend theme.

Hi, @Gauravvvv
As far as I know there is no hard and fast rules regarding the usage of theme. Its completely subjective. Though I've also seen Claro being used as admin theme predominantly, but one might choose to opt it for their frontend theme based on their needs. So I think it'll be better to address the issue and fix it, no matter if someone is using it as frontend theme or admin theme, we should fix the flaws in the respective theme!

We have the same rule in olivero, What's missing here would be to remove the hyphen rules in other places it is used in claro like the pcss files: media-library, vertical-tabs, form--managed-file, details,

comment in mr

Feedback addressed.

we can still remove the hyphens: auto from the managed file css, we just need to keep break-word since it's different from the default

Hello @here, Commit is successfully removing the hyphens: auto from css & .pcss.css files. Looks good to me., So moving it to RTBC++. Thanks!.

Committed 98b2153 and pushed to 11.x. Thanks!

nod_ → closed merge request !7493

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

"body {hyphens: auto}" is in 10.3.x/11.x and git blame is pointing to this issue.
Can someone check if the commits were merged correctly?

Echoing Lars' comment #21.

We have had feedback from our editors regarding hyphens appearing in editor textareas. This is causing confusion between legit hyphen usage and those inserted by the CSS.

I am here to second kevinb623, and also, why do we not want WYSIWYG anymore? Let's keep the WYG as close to the WYS as we can.