WordPress / twentytwentyfive

152 stars 105 forks source link

Remove custom focus style #412

Closed richtabor closed 1 month ago

richtabor commented 1 month ago

If this is necessary it should be done from the core perspective:

/* Focus styles */
:where(.wp-site-blocks *:focus) {
    outline-width: 2px;
    outline-style: solid;
}
carolinan commented 1 month ago

This has been discussed among contributors since January and as separate issue in this repository. https://github.com/WordPress/twentytwentyfive/issues/6

richtabor commented 1 month ago

So it's ok to add a focus style to every block? How do we know this doesn't conflict with any block? Why isn't this pushed into core?

These questions remain.

richtabor commented 1 month ago

This suggests it is needed for relevant elements, not every block. https://github.com/WordPress/twentytwentyfive/issues/6#issuecomment-2259484213

richtabor commented 1 month ago

My goal is to prevent and reduce conflicts (like was introduced with Twenty Twenty-Four).