Conversation
| * | ||
| * @param {String} id Unique identifier for editor instance | ||
| * @param {Object} post API entity for post to edit | ||
| */ |
There was a problem hiding this comment.
Should the documentation match the one changed above?
Bootstrapped post object
There was a problem hiding this comment.
Argh, I totally overlooked this comment. Yes, it probably should.
| */ | ||
| function gutenberg_scripts_and_styles( $hook ) { | ||
| if ( 'toplevel_page_gutenberg' !== $hook ) { | ||
| if ( ! preg_match( '/(toplevel|gutenberg)_page_gutenberg(-demo)?/', $hook, $page_match ) ) { |
There was a problem hiding this comment.
Not 100% sure but somehow my browser tried loading all of the Gutenberg scripts when not on Gutenberg, when logging into WP-Admin. I haven't been able to reproduce yet, so I will check this out again in a bit.
Update: Can't reproduce, should be fine.
Check, check, and check. 👍 |
youknowriad
left a comment
There was a problem hiding this comment.
I think this is good to go 👍
Longer term, maybe we should drop the "Gutenberg Menu" and move these links under "Posts":
Posts
-> All (highlighted when editing on guteberg too)
-> Add New
-> Add New Gutenberg Post
-> Add New Gutenberg Demo Post
(Same for other Post Types)
|
Thanks both! |
Closes #1000
This pull request seeks to enable creating new posts, separating Demo to a submenu item of the top-level Gutenberg menu item.
Testing instructions:
Verify that...