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

Replace gutenberg in classNames with block-editor #10783

Merged
merged 1 commit into from
Oct 19, 2018

Conversation

youknowriad
Copy link
Contributor

closes #10777

Testing instructions

  • Just make sure the replaced classNames are not referenced anywhere anymore.

@youknowriad youknowriad added this to the 4.1 - UI freeze milestone Oct 19, 2018
@youknowriad youknowriad self-assigned this Oct 19, 2018
@youknowriad youknowriad requested review from pento and a team October 19, 2018 08:50
@youknowriad youknowriad force-pushed the replace/gutenberg-classnames branch from df24788 to 28d13a8 Compare October 19, 2018 08:51
@@ -21,14 +21,16 @@
* The main entry point for the Gutenberg editor. Renders the editor on the
* wp-admin page for the plugin.
*
* The gutenberg and gutenberg__editor classNames are left for backward compatibility.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we really need it? It's going to be moved to WordPress core as far as I understand?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yah, all of this is going to be committed to Core.

Copy link
Member

@pento pento left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not too fussed whether the plugin keeps the old class names for a little bit, the main thing is that all the CSS is changed, so I can drop them from core. 🙂

@youknowriad youknowriad merged commit 7936eea into master Oct 19, 2018
@youknowriad youknowriad deleted the replace/gutenberg-classnames branch October 19, 2018 09:25
@mtias mtias added the [Type] Code Quality Issues or PRs that relate to code quality label Oct 19, 2018
antpb pushed a commit to antpb/gutenberg that referenced this pull request Oct 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Code Quality Issues or PRs that relate to code quality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rename "gutenberg" Class Names
4 participants