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

EditorSkeleton: Add Changelog entries, drop backwards-compat class names #20132

Merged
merged 4 commits into from
Feb 10, 2020

Conversation

ockham
Copy link
Contributor

@ockham ockham commented Feb 10, 2020

Description

Follow-up to #20050, where the component was moved and renamed.
Dropping backwards-compat classnames as discussed here.

How has this been tested?

Rebuild Gutenberg (npm run build), and start it (npx wp-env start). Verify that the editor styling isn't broken (specifically regions like header or sidebar).

Types of changes

Add Changelog entries, drop backwards-compat class names.

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows the accessibility standards.
  • My code has proper inline documentation.

@ockham ockham added the [Package] Block editor /packages/block-editor label Feb 10, 2020
@ockham ockham requested a review from youknowriad February 10, 2020 07:28
@ockham ockham requested a review from ellatrix as a code owner February 10, 2020 07:28
@ockham ockham self-assigned this Feb 10, 2020
@@ -1,3 +1,7 @@
## Master

- Add new `__experimentalEditorSkeleton` component. This has been moved over from the `@wordpress/edit-post` package, where it was an internal component called `EditorRegions`. Its class names have thus been renamed from `edit-post-editor-regions` to `block-editor-editor-skeleton`.
Copy link
Contributor

Choose a reason for hiding this comment

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

In general we add "sections" to changelog entries. I believe this one could be "### New Features" and the other one could be ### Internal

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah right, thanks. Done in 67061c6

Copy link
Contributor

@youknowriad youknowriad left a comment

Choose a reason for hiding this comment

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

Thanks 👍

@ockham ockham merged commit 2c518a4 into master Feb 10, 2020
@ockham ockham deleted the add/editor-skeleton-changelog branch February 10, 2020 09:09
@github-actions github-actions bot added this to the Gutenberg 7.5 milestone Feb 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Package] Block editor /packages/block-editor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants