WordPress / theme-experiments

Experimenting with themes made out of blocks.
GNU General Public License v2.0
546 stars 180 forks source link

TT1 Blocks: Remove the Dark gray social link block style #176

Closed carolinan closed 3 years ago

carolinan commented 3 years ago

With the added color options to the social links block, the block style is no longer needed, it is subpar to the new color settings.

And when the block style is selected in the block setting sidebar, the color options has no effect in the editor, only on the front, this may cause confusion.

A question arises on how one can "migrate" existing block styles, for users who has already selected this style. Especially if we consider making the same change to Twenty Twenty-One, once this option is added to WordPress core.

kjellr commented 3 years ago

A question arises on how one can "migrate" existing block styles, for users who has already selected this style. Especially if we consider making the same change to Twenty Twenty-One, once this option is added to WordPress core.

Good question — I was about to suggest that we remove the block style registration, but keep the styles in place for existing blocks... but then I realized that in order to remove these styles from existing blocks, folks would need to find the "CSS Classes" field and remove the classname. That seems cumbersome.

Maybe it's better to just keep this block style in place, even though it can be reproduced by the new settings? It'll still work, so I don't think it does any harm to include it.

carolinan commented 3 years ago

It doesn't work fully: And when the block style is selected in the block setting sidebar, the color options has no effect in the editor, only on the front, this may cause confusion.

kjellr commented 3 years ago

Oh ok I see what you mean. Yeah, this is a tough one. 🤔 Maybe removing this block style but keeping the CSS in place is the best overall move then.

mtias commented 3 years ago

That's how we have dealt with a few of the style variations removals before.