-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Send contentInfo information when updating HTML. #23115
Conversation
Size Change: 0 B Total Size: 1.12 MB ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM @SergioEstevao ! I've only left one comment but I don't consider it blocking for the PR.
@@ -252,7 +252,7 @@ export default compose( [ | |||
title: getEditedPostAttribute( 'title' ), | |||
getEditedPostContent, | |||
selectedBlockIndex: getBlockIndex( selectedBlockClientId ), | |||
blockCount: getBlockCount( rootClientId ), | |||
blockCount: getGlobalBlockCount(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like getBlockCount
is not needed anymore and we can remove it from the extraction list above, right?
Description
Send content structure information to the native apps for them to be able to display it.
How has this been tested?
This can be tested using the GB-mobile PR here: wordpress-mobile/gutenberg-mobile#2380
Screenshots
Types of changes
Checklist: