Site Editor: Navigation templates #25739
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,68 @@ | ||
| export const ITEM_CONTENTS = { | ||
david-szabo97 marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| // General | ||
| 'front-page': { | ||
| title: 'Front page', | ||
| description: '', | ||
|
There was a problem hiding this comment. We could've added "Display the contents of your Front Page" here, or something along the similar lines. There was a problem hiding this comment. @david-szabo97 @vindl — Let's not forget i18n. Constants files like this should be i18n-ready from the start, otherwise getting everything ready for a testable Site Editor around WP 5.6 will be much more difficult. There was a problem hiding this comment. @mcsf My fault, I should have caught this in review! |
||
| }, | ||
| archive: { | ||
| title: 'Archive', | ||
| description: | ||
| 'Displays the content lists when no other template is found', | ||
| }, | ||
| single: { | ||
| title: 'Single', | ||
| description: 'Displays the content of a single post', | ||
| }, | ||
| singular: { | ||
| title: 'Singular', | ||
| description: 'Displays the content of a single page', | ||
| }, | ||
| index: { | ||
| title: 'Default (index)', | ||
| description: 'Displays the content of a single page', | ||
| }, | ||
| search: { | ||
| title: 'Search results', | ||
| description: '', | ||
| }, | ||
| '404': { | ||
| title: '404', | ||
| description: 'Displayed when a non-existing page requested', | ||
| }, | ||
|
|
||
| // Pages | ||
| page: { | ||
| title: 'Default (Page)', | ||
| description: 'Displays the content of a single page', | ||
| }, | ||
|
|
||
| // Posts | ||
| home: { | ||
| title: 'Posts (home)', | ||
| description: 'Displayed on your homepage', | ||
| }, | ||
| 'archive-post': { | ||
| title: 'Default (Post archive)', | ||
| description: 'Displays a list of posts', | ||
| }, | ||
| 'single-post': { | ||
| title: 'Default (Single post)', | ||
| description: 'Displays the content of a single post', | ||
| }, | ||
| }; | ||
|
|
||
| export const GENERAL_TEMPLATES = [ | ||
| 'front-page', | ||
| 'archive', | ||
| 'single', | ||
| 'singular', | ||
| 'index', | ||
| 'search', | ||
| '404', | ||
| ]; | ||
|
|
||
| export const POSTS_GENERAL_TEMPLATES = [ | ||
| 'single-post', | ||
| 'archive-post', | ||
| 'home', | ||
| ]; | ||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,46 @@ | ||
| /** | ||
| * WordPress dependencies | ||
| */ | ||
| import { __ } from '@wordpress/i18n'; | ||
| import { useSelect } from '@wordpress/data'; | ||
| import { | ||
| __experimentalNavigationItem as NavigationItem, | ||
| __experimentalNavigationMenu as NavigationMenu, | ||
| } from '@wordpress/components'; | ||
|
|
||
| /** | ||
| * Internal dependencies | ||
| */ | ||
| import TemplateNavigationItems from '../template-navigation-items'; | ||
|
|
||
| export default function TemplatePartsMenu( { onActiveTemplatePartIdChange } ) { | ||
| const templateParts = useSelect( ( select ) => { | ||
| const currentTheme = select( 'core' ).getCurrentTheme()?.textdomain; | ||
|
|
||
| return select( 'core' ).getEntityRecords( | ||
| 'postType', | ||
| 'wp_template_part', | ||
| { | ||
| theme: currentTheme, | ||
| status: [ 'publish', 'auto-draft' ], | ||
| per_page: -1, | ||
| } | ||
| ); | ||
| }, [] ); | ||
|
|
||
| return ( | ||
| <NavigationMenu | ||
| menu="template-parts" | ||
| title="Template Parts" | ||
| parentMenu="root" | ||
| > | ||
| <TemplateNavigationItems | ||
| entityType="wp_template_part" | ||
| templates={ templateParts } | ||
| onActivate={ onActiveTemplatePartIdChange } | ||
| /> | ||
|
|
||
| { ! templateParts && <NavigationItem title={ __( 'Loading…' ) } /> } | ||
| </NavigationMenu> | ||
| ); | ||
| } |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,43 @@ | ||
| /** | ||
| * WordPress dependencies | ||
| */ | ||
| import { | ||
| __experimentalNavigationGroup as NavigationGroup, | ||
| __experimentalNavigationMenu as NavigationMenu, | ||
| } from '@wordpress/components'; | ||
|
|
||
| /** | ||
| * Internal dependencies | ||
| */ | ||
| import TemplateNavigationItems from '../template-navigation-items'; | ||
|
|
||
| export default function TemplatesPagesMenu( { templates, onActiveIdChange } ) { | ||
| const defaultTemplate = templates?.find( ( { slug } ) => slug === 'page' ); | ||
| const specificPageTemplates = templates?.filter( ( { slug } ) => | ||
| slug.startsWith( 'page-' ) | ||
| ); | ||
|
|
||
| return ( | ||
| <NavigationMenu | ||
| menu="templates-pages" | ||
| title="Pages" | ||
| parentMenu="templates" | ||
| > | ||
| <NavigationGroup title="Specific"> | ||
| <TemplateNavigationItems | ||
| templates={ specificPageTemplates } | ||
| onActivate={ onActiveIdChange } | ||
| /> | ||
| </NavigationGroup> | ||
|
|
||
| { defaultTemplate && ( | ||
| <NavigationGroup title="General"> | ||
| <TemplateNavigationItems | ||
| templates={ defaultTemplate } | ||
| onActivate={ onActiveIdChange } | ||
| /> | ||
| </NavigationGroup> | ||
| ) } | ||
| </NavigationMenu> | ||
| ); | ||
| } |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,44 @@ | ||
| /** | ||
| * WordPress dependencies | ||
| */ | ||
| import { | ||
| __experimentalNavigationGroup as NavigationGroup, | ||
| __experimentalNavigationMenu as NavigationMenu, | ||
| } from '@wordpress/components'; | ||
|
|
||
| /** | ||
| * Internal dependencies | ||
| */ | ||
| import TemplateNavigationItems from '../template-navigation-items'; | ||
| import { POSTS_GENERAL_TEMPLATES } from '../constants'; | ||
|
|
||
| export default function TemplatePostsMenu( { templates, onActiveIdChange } ) { | ||
| const generalTemplates = templates?.find( ( { slug } ) => | ||
| POSTS_GENERAL_TEMPLATES.includes( slug ) | ||
| ); | ||
| const specificTemplates = templates?.filter( ( { slug } ) => | ||
| slug.startsWith( 'post-' ) | ||
| ); | ||
|
|
||
| return ( | ||
| <NavigationMenu | ||
|
There was a problem hiding this comment. We could've delayed adding this until we had actual content for it. |
||
| menu="templates-posts" | ||
| title="Posts" | ||
| parentMenu="templates" | ||
| > | ||
| <NavigationGroup title="Specific"> | ||
| <TemplateNavigationItems | ||
| templates={ specificTemplates } | ||
| onActivate={ onActiveIdChange } | ||
| /> | ||
| </NavigationGroup> | ||
|
|
||
| <NavigationGroup title="General"> | ||
| <TemplateNavigationItems | ||
| templates={ generalTemplates } | ||
| onActivate={ onActiveIdChange } | ||
| /> | ||
| </NavigationGroup> | ||
| </NavigationMenu> | ||
| ); | ||
| } | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
As already mentioned, accessing these files on each request is going to have a negative impact on performance. We can probably store this info once on theme activation and we'll also need it after theme updates.