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

Make BlockNavigationList component experimental #17446

Merged
merged 2 commits into from
Sep 16, 2019

Conversation

talldan
Copy link
Contributor

@talldan talldan commented Sep 15, 2019

Description

Updates the new BlockNavigationList component to be experimental while it's in development.

Types of changes

Refactor

@talldan talldan added the [Type] Task Issues or PRs that have been broken down into an individual action to take label Sep 15, 2019
@talldan talldan self-assigned this Sep 15, 2019
@talldan talldan mentioned this pull request Sep 15, 2019
5 tasks
@talldan talldan added this to the Gutenberg 6.5 milestone Sep 15, 2019
@@ -16,7 +16,7 @@ import { __ } from '@wordpress/i18n';
*/
import BlockIcon from '../block-icon';

export default function BlockNavigationList( {
export default function __experimentalBlockNavigationList( {
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you think we can keep the name as is internally and just rename the one exported in the public API?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@youknowriad Yep, that's now been changed 👍

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 @talldan LGTM 👍

@talldan talldan merged commit 4202ab3 into master Sep 16, 2019
@talldan talldan deleted the update/experimental-block-nav-list branch September 16, 2019 03:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Task Issues or PRs that have been broken down into an individual action to take
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants