-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Site Editor: Add a fallback template showing the title and content for the post only mode. #56509
Site Editor: Add a fallback template showing the title and content for the post only mode. #56509
Conversation
…r the post only mode
Size Change: -31 B (0%) Total Size: 1.7 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.
Good fallback plan.
Before
2023-11-27.11.26.59.mp4
After
2023-11-27.11.28.25.mp4
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.
Nice, this feels like a good default to me, too!
One potential issue to smooth out (but not a blocker for this PR as it's only partly related) is what we do with the content only mode when you initially go into a page that doesn't have any content blocks. If you have the list view open by default, it appears empty, and if you click anywhere on the page, you're nudged to switch to template editing mode. I'm not too sure what the best behaviour would be here, but in any case, the user can either go to edit their template, or switch the template preview mode off if they wish to edit the content for the page directly:
2023-11-27.12.42.07.mp4
🤔
This change LGTM, though! ✨
Related #52632
Follow-up #56418
What?
If your page is using a template with a query loop block rather than using directly post title and post content, disabling the template preview (under "template" panel in the sidebar) resulted in a white page because no post blocks were found in the template.
This PR adds a fallback showing the post title and post content of the page (kind of mimicking the post editor).
My ultimate goal (not sure how possible) is to make the "post-only" mode strictly equivalent to the post editor which would allow us to refactor the post editor to use it instead of handing the switching between template and post manually.
Testing instructions