| CARVIEW |
Select Language
HTTP/2 200
date: Sat, 11 Oct 2025 10:33:55 GMT
content-type: text/html; charset=utf-8
cache-control: no-cache
content-security-policy: default-src 'none'; base-uri 'self'; child-src github.githubassets.com github.com/assets-cdn/worker/ github.com/assets/ gist.github.com/assets-cdn/worker/; connect-src 'self' uploads.github.com www.githubstatus.com collector.github.com raw.githubusercontent.com api.github.com github-cloud.s3.amazonaws.com github-production-repository-file-5c1aeb.s3.amazonaws.com github-production-upload-manifest-file-7fdce7.s3.amazonaws.com github-production-user-asset-6210df.s3.amazonaws.com *.rel.tunnels.api.visualstudio.com wss://*.rel.tunnels.api.visualstudio.com github.githubassets.com objects-origin.githubusercontent.com copilot-proxy.githubusercontent.com proxy.individual.githubcopilot.com proxy.business.githubcopilot.com proxy.enterprise.githubcopilot.com *.actions.githubusercontent.com wss://*.actions.githubusercontent.com productionresultssa0.blob.core.windows.net/ productionresultssa1.blob.core.windows.net/ productionresultssa2.blob.core.windows.net/ productionresultssa3.blob.core.windows.net/ productionresultssa4.blob.core.windows.net/ productionresultssa5.blob.core.windows.net/ productionresultssa6.blob.core.windows.net/ productionresultssa7.blob.core.windows.net/ productionresultssa8.blob.core.windows.net/ productionresultssa9.blob.core.windows.net/ productionresultssa10.blob.core.windows.net/ productionresultssa11.blob.core.windows.net/ productionresultssa12.blob.core.windows.net/ productionresultssa13.blob.core.windows.net/ productionresultssa14.blob.core.windows.net/ productionresultssa15.blob.core.windows.net/ productionresultssa16.blob.core.windows.net/ productionresultssa17.blob.core.windows.net/ productionresultssa18.blob.core.windows.net/ productionresultssa19.blob.core.windows.net/ github-production-repository-image-32fea6.s3.amazonaws.com github-production-release-asset-2e65be.s3.amazonaws.com insights.github.com wss://alive.github.com wss://alive-staging.github.com api.githubcopilot.com api.individual.githubcopilot.com api.business.githubcopilot.com api.enterprise.githubcopilot.com; font-src github.githubassets.com; form-action 'self' github.com gist.github.com copilot-workspace.githubnext.com objects-origin.githubusercontent.com; frame-ancestors 'none'; frame-src viewscreen.githubusercontent.com notebooks.githubusercontent.com; img-src 'self' data: blob: github.githubassets.com media.githubusercontent.com camo.githubusercontent.com identicons.github.com avatars.githubusercontent.com private-avatars.githubusercontent.com github-cloud.s3.amazonaws.com objects.githubusercontent.com release-assets.githubusercontent.com secured-user-images.githubusercontent.com/ user-images.githubusercontent.com/ private-user-images.githubusercontent.com opengraph.githubassets.com marketplace-screenshots.githubusercontent.com/ copilotprodattachments.blob.core.windows.net/github-production-copilot-attachments/ github-production-user-asset-6210df.s3.amazonaws.com customer-stories-feed.github.com spotlights-feed.github.com objects-origin.githubusercontent.com *.githubusercontent.com; manifest-src 'self'; media-src github.com user-images.githubusercontent.com/ secured-user-images.githubusercontent.com/ private-user-images.githubusercontent.com github-production-user-asset-6210df.s3.amazonaws.com gist.github.com; script-src github.githubassets.com; style-src 'unsafe-inline' github.githubassets.com; upgrade-insecure-requests; worker-src github.githubassets.com github.com/assets-cdn/worker/ github.com/assets/ gist.github.com/assets-cdn/worker/
referrer-policy: no-referrer-when-downgrade
server-timing: pull_request_layout-fragment;desc="pull_request_layout fragment";dur=318.138689,conversation_content-fragment;desc="conversation_content fragment";dur=1322.297262,conversation_sidebar-fragment;desc="conversation_sidebar fragment";dur=314.04857,nginx;desc="NGINX";dur=0.482285,glb;desc="GLB";dur=139.237372
strict-transport-security: max-age=31536000; includeSubdomains; preload
vary: X-PJAX, X-PJAX-Container, Turbo-Visit, Turbo-Frame, X-Requested-With,Accept-Encoding, Accept, X-Requested-With
x-content-type-options: nosniff
x-frame-options: deny
x-voltron-version: aab62e3
x-xss-protection: 0
server: github.com
content-encoding: gzip
accept-ranges: bytes
set-cookie: _gh_sess=Usx4narMRYpW6fD8RQWxoAxElYxjo%2BkiqWPsc3ONmKKADe70JPgST3AvMozFBapZihT%2Bx9Txmia5DpEzOTlV7WGMOlFdmaPEmxNEX6czXFKA7UXvIKG%2F2o2llEeamk2Iq39nBkxpJ5wbUBjX0%2FTzlwSAHykwxaz3NgTUddeOgtOpdZAkUPW8PZNGBPhXQHYvO%2FGCdGtPu0hAA3nM%2BW2tkD%2B3efpB3oCqe4Z9jgs8JexcbNj8M1q6PK2owbXiG%2BLYa4W5fdqBnHBNG%2FogWZaxAg%3D%3D--Zp4snkcyOdPnZxPo--Njn2Q5PPSGJ17Bf5ifg8Pw%3D%3D; Path=/; HttpOnly; Secure; SameSite=Lax
set-cookie: _octo=GH1.1.1003236032.1760178834; Path=/; Domain=github.com; Expires=Sun, 11 Oct 2026 10:33:54 GMT; Secure; SameSite=Lax
set-cookie: logged_in=no; Path=/; Domain=github.com; Expires=Sun, 11 Oct 2026 10:33:54 GMT; HttpOnly; Secure; SameSite=Lax
x-github-request-id: D78E:E3EA6:5B1270:7A250F:68EA3292
Template activation: reduce templates listed as options for post/page by ellatrix · Pull Request #72141 · WordPress/gutenberg · GitHub
ellatrix
added
the
Backport to Gutenberg RC
Pull request that needs to be backported to a Gutenberg release candidate (RC)
label
Oct 7, 2025
ellatrix
force-pushed
the
fix/post-template-modal
branch
from
October 7, 2025 19:06
cbravobernal
removed
the
Backport to Gutenberg RC
Pull request that needs to be backported to a Gutenberg release candidate (RC)
label
Oct 8, 2025
Skip to content
Navigation Menu
{{ message }}
-
Notifications
You must be signed in to change notification settings - Fork 4.6k
Template activation: reduce templates listed as options for post/page #72141
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
Merged
Conversation
This file contains hidden or 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
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
|
Size Change: +52 B (0%) Total Size: 1.96 MB
ℹ️ View Unchanged
|
ellatrix
added
the
Backport to Gutenberg RC
ellatrix
force-pushed
the
fix/post-template-modal
branch
from
07d31b4 to
4f23323
Compare
|
Flaky tests detected in 4f23323. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/18323374394
|
Mamaduka
reviewed
Oct 8, 2025
Comment on lines
76
to
89
| return [ | ||
| ...registeredTemplates, | ||
| ...userTemplates.filter( | ||
| ( template ) => | ||
| // Only give "custom" templates as an option, which | ||
| // means the is_wp_suggestion meta field is not set and | ||
| // the slug is not found in the default template types. | ||
| // https://github.com/WordPress/wordpress-develop/blob/97382397b2bd7c85aef6d4cd1c10bafd397957fc/src/wp-includes/block-template-utils.php#L858-L867 | ||
| ! template.meta.is_wp_suggestion && | ||
| ! defaultTemplateTypes.find( | ||
| ( type ) => type.slug === template.slug | ||
| ) | ||
| ), | ||
| }; | ||
| ]; |
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.
This is a practice we're trying to discourage and use rarely. It's probably better to do this merging and filtering in useMemo, no?
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.
Oh that's news to me, why is it discouraged?
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.
I know it's not the case here, but it's easy to "break" shallow equality checks if you're not careful and manipulate objects/arrays inside the useSelect callback, IMO.
We even introduced a dev warning to catch those early and help third-party developers.
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.
Hm, but the same can be said of returning an object breaking shallow equality if you nest them. I get your argument, but I think there are valid cases like being able avoid passing values to the render function and avoid re-renders due to value changes. Not super applicable here, but I like avoiding the useMemo call if it's possible like this.
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.
Restored in 532f4b1
|
Merging because this is blocking GB stable. Let me know if there's concerns and I'll follow up. |
cbravobernal
pushed a commit
that referenced
this pull request
Oct 8, 2025
…#72141) Co-authored-by: ellatrix <ellatrix@git.wordpress.org> Co-authored-by: Mamaduka <mamaduka@git.wordpress.org>
|
I just cherry-picked this PR to the release/21.8 branch to get it included in the next release: e266b75 |
cbravobernal
removed
the
Backport to Gutenberg RC
6 tasks
sidharthpandita1
pushed a commit
to sidharthpandita1/gutenberg
that referenced
this pull request
Oct 10, 2025
…WordPress#72141) Co-authored-by: ellatrix <ellatrix@git.wordpress.org> Co-authored-by: Mamaduka <mamaduka@git.wordpress.org>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
You can’t perform that action at this time.
What?
Restores the behaviour from WP 6.8: only actual custom templates should be listed in the template modal for specific posts/pages.
Why?
This is the current WP behavior: when you make edits to a theme template file, the old template or the edit is not visible as an option. Only templates that are "custom" according to the hierarchy can be picked. This is not ideal, but it's a currently limitation because the template property of a post has to be a slug instead of the template ID.
How?
There's actually an error in the changes I made before: the new templates endpoint doesn't have a
post_typequery option because it's a normal post endpoint. That's why I have to expose theis_wp_suggestionmeta (which was not correctly set anymore after the rewrite), which seems to be used currently to determine if a template is custom.A template is custom if
is_wp_suggestionis not set and if the template slug does not appear in the default template types.Testing Instructions
Create a specific template for a certain post, this template should not appear in the template options for other posts.
Duplicate the page template for the theme (or any other template). These user templates should also not appear in the options in the post/page editors.
Now create a "custom" templates (last option) after clicking "Add template". This one should appear in the options!
Testing Instructions for Keyboard
Screenshots or screencast