Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
carolinan opened this issue Jan 26, 2021 · 4 comments · Fixed by #229
Closed

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

carolinan opened this issue Jan 26, 2021 · 4 comments · Fixed by #229
Assignees

Comments

@carolinan
Copy link
Collaborator

carolinan commented Jan 26, 2021

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
Copy link
Collaborator

kjellr commented Jan 28, 2021

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
Copy link
Collaborator Author

carolinan commented Jan 28, 2021

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
Copy link
Collaborator

kjellr commented Jan 28, 2021

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
Copy link
Member

mtias commented Feb 9, 2021

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants