| CARVIEW |
Select Language
HTTP/2 200
date: Sat, 11 Oct 2025 10:47:00 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=436.124081,conversation_content-fragment;desc="conversation_content fragment";dur=2216.435236,conversation_sidebar-fragment;desc="conversation_sidebar fragment";dur=384.197664,nginx;desc="NGINX";dur=1.281136,glb;desc="GLB";dur=138.328214
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=OJWRBtKA5C6j13Df6q2TMz%2F%2FOxqSDWA7kpdJJhh7usovsYU0zzUPipUKn99ru%2BaF7j2bxqGKpodBQTwQoYsCLGSx9ICMmYMuq5P0FmxqwxsutGqm3Y%2FymBLRjxwJYPo3funf0xQUw9IbDp8t894onEAqI%2FsKbr%2F1mYdXqV011khnBGe99UAM2FlDBTuq8TvRwbqAH2F%2Bzb0BM%2F8%2BIJa55jCytVzEdyydMvqvLFmNdHWp9HnVXBtuFqY%2FOZV%2Fjq66s4%2BZLxCur9NuUV%2BnM0SrTw%3D%3D--0HC5AnWAooXBpa4p--WC%2BuFEziDKrtYzJ9ECGxSg%3D%3D; Path=/; HttpOnly; Secure; SameSite=Lax
set-cookie: _octo=GH1.1.1876120243.1760179617; Path=/; Domain=github.com; Expires=Sun, 11 Oct 2026 10:46:57 GMT; Secure; SameSite=Lax
set-cookie: logged_in=no; Path=/; Domain=github.com; Expires=Sun, 11 Oct 2026 10:46:57 GMT; HttpOnly; Secure; SameSite=Lax
x-github-request-id: D12A:12893F:5BFD3D:7B7890:68EA35A1
Template activation: don't set site option on edit by ellatrix · Pull Request #71811 · WordPress/gutenberg · GitHub
ellatrix
force-pushed
the
fix/template-activation-site-option
branch
from
September 30, 2025 07:55
Skip to content
Navigation Menu
{{ message }}
-
Notifications
You must be signed in to change notification settings - Fork 4.6k
Template activation: don't set site option on edit #71811
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
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
|
Size Change: +267 B (+0.01%) Total Size: 1.95 MB
ℹ️ View Unchanged
|
kmanijak
reviewed
Sep 22, 2025
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 confirm this resolves #71724 🙌
packages/edit-site/src/components/template-save-dialog/index.js
Outdated
Show resolved
Hide resolved
6 tasks
|
Flaky tests detected in 93a638c. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/18123574499
|
|
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. |
youknowriad
reviewed
Sep 29, 2025
packages/editor/src/components/post-publish-panel/postpublish.js
Outdated
Show resolved
Hide resolved
packages/editor/src/components/post-publish-panel/postpublish.js
Outdated
Show resolved
Hide resolved
packages/editor/src/components/post-publish-panel/postpublish.js
Outdated
Show resolved
Hide resolved
youknowriad
reviewed
Sep 29, 2025
| .getEditorSettings(); | ||
|
|
||
| // Don't open for focused entity. | ||
| if ( editorSettings.onNavigateToPreviousEntityRecord ) { |
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 probably not the right fix to be honest. It's an indirect check. I'd be ok keeping it but I think the commend should be a lot more clear than that. we want to check "custom template". It's not out of the question that a regular template is also accessed as a focused entity, and also in the site editor you can also create custom templates without going through the page first.
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'm surprised we don't have previous cases of "isHierarchyTemplate" or something like that.
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 don't think isHierarchyTemplate will help us tbh. It would be possible to have multiple post or page templates and edit them as a focussed entity here without it being the active post/page template. While it needs to be possible to set it active generally, I don't think showing the activation panel in the focussed mode is expected.
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.
So in other words, the goal is not to check for "custom" templates, but rather if you're editing it outside of a page/post editor context.
youknowriad
reviewed
Sep 29, 2025
youknowriad
reviewed
Sep 29, 2025
packages/editor/src/components/post-publish-panel/postpublish-next-panel-template.js
Outdated
Show resolved
Hide resolved
ellatrix
force-pushed
the
fix/template-activation-site-option
branch
from
b3ecb11 to
54424ed
Compare
youknowriad
reviewed
Sep 30, 2025
|
|
||
| await editor.saveSiteEditorEntities(); | ||
| await editor.saveSiteEditorEntities( { | ||
| isOnlyCurrentEntityDirty: true, |
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.
What is this flag about?
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 think it's to signal that it's not a multi-entity save. This just restores the test to pre-#67125.
youknowriad
approved these changes
Sep 30, 2025
kmanijak
reviewed
Sep 30, 2025
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 ran WooCommerce E2E tests with GB built on this feature branch (woocommerce/woocommerce#61046). We have better result than before. Some tests are still failing, especially related to template customisations but I need to add a step to activate the template through the new flow.
Tomorrow morning I'll add this step so we have better confidence everything works and check few remaining tests that are failing. Two of them fail definitely due to UI change in new templates creation and there just fe that require manual checking. I don't think I'll be able to test the PR deeply due to other assignments but E2E tests run is promising! (FYI: @sunyatasattva)
22 tasks
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?
Closes #71724
Why?
Editing a template should not be setting site settings at the same time.
Additionally, I had to do some major refactoring of the back-end filters due to an e2e correctly failing for post templates. Somehow the test in trunk passes while even though there's an error in the test. Basically, when the template resolution found a specific page template to show (vs the hierarchy), it just adds the template slug to the top of the hierarchy. I decided to reimplement the way Gutenberg intercepts template resolution with a way earlier filter to make the logic a lot more solid and prepare us better for a core backport. It requires us to copy over some core code, but the logic has become a lot clearer.
To do: we need to find a way to NOT show the activation post-publish panel when saving a template after creating one specifically for a post in the post editor.How?
Let's ask the user after editing a theme template if they want to activate the template.
Testing Instructions
With a theme template active for home/index, navigate to the root of the site editor. Edit the template and save. You should see a notice asking whether to activate the new template.
Similarly for theme templates in the templates list, when you edit it and save, you should see the same notice.
Testing Instructions for Keyboard
Screenshots or screencast
Before
After
template-activate-notice.mov