Theme build process#212
Merged
kimcoleman merged 27 commits intostrangerstudios:memberlite-7-0from Feb 16, 2026
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR replaces the deprecated “Blank” page template with per-page editor toggles (hide header/footer), introduces a small JS build pipeline for editor assets, and adds an upgrade routine to migrate existing pages that used the blank template.
Changes:
- Removed
templates/blank.phpand added page-level header/footer visibility toggles via block editor Document Settings. - Added a webpack/wp-scripts build setup for editor JS (source in
src/, output expected inbuild/). - Added an update routine to migrate published pages using the deprecated blank template to the new meta-based settings.
Reviewed changes
Copilot reviewed 12 out of 15 changed files in this pull request and generated 9 comments.
Show a summary per file
| File | Description |
|---|---|
| webpack.config.js | Adds wp-scripts-based webpack entry for editor custom settings bundle. |
| templates/blank.php | Removes the deprecated Blank page template. |
| src/editor/custom-settings.js | Registers a Gutenberg document settings panel with header/footer toggles. |
| package.json | Introduces npm scripts and @wordpress/scripts dependency. |
| inc/updates.php | Adds migration to set new meta flags for pages using the old blank template; updates db version gating. |
| inc/editor-settings.php | Registers the meta keys, enqueues the built editor script, and exposes helpers for templates. |
| header.php | Switches from template-based checks to meta-based header visibility checks; adjusts wrapper logic. |
| footer.php | Switches from template-based checks to meta-based footer visibility checks; adjusts wrapper logic. |
| components/page/content-page.php | Updates page navigation condition to use new header/footer visibility helpers. |
| functions.php | Loads the new editor settings module. |
| .gitignore | Ignores build/ output directory. |
| docs/build-process.md | Documents the new build workflow. |
| docs/project-structure.md | Adds repository structure documentation. |
| docs/customizer.md | Adds customizer architecture documentation. |
| docs/README.md | Adds an index/entry point for docs. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…nditional with helper
… to the new post meta that matches our new settings
… own file in includes
…king the returns on the helpers clearer
…move the filters from helpers
…calization to editor settings JS
…separate update file for memberlite 7.0
… still there for backwards compatibility but added a comment
… active for reference, no new setting added
…iding prev/next on pages
…ally registered the existing post meta because the REST API needs to read it (the block editor)
7f6b895 to
1e58c94
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 18 out of 21 changed files in this pull request and generated 8 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
kimcoleman
requested changes
Feb 13, 2026
…script for fluid-width template to use template setting; putting latest color migration script in place before merge
RachelRVasquez
added a commit
to RachelRVasquez/memberlite
that referenced
this pull request
Mar 12, 2026
* added simple build process so we can add custom settings * in the process of replacing conditionals where we check for the blank templat * created a small helper to get the post meta, swapped page template conditional with helper * delete blank template * separate hiding the header and footer into two separate toggles * add function to replace old post meta tied to the blank page template to the new post meta that matches our new settings * renamed existing docs to lowercase, moved the editor settings to it's own file in includes * added docs for the build process * return early if updates function doesn't find any pages with the blank template * changes based on co-pilot feedback, mainly re-naming functions and making the returns on the helpers clearer * fix to path in docs, return early if post type isn't a page in JS, remove the filters from helpers * return early if assets file doesn't exist for editor settings, add localization to editor settings JS * renamed function where we register post meta for settings, created a separate update file for memberlite 7.0 * remove interstitial template and any references in code to it, CSS is still there for backwards compatibility but added a comment * added a function to set post meta where the interstitial template was active for reference, no new setting added * corrected how I was adding the setting to the document settings for hiding prev/next on pages * removed checkbox in the metabox we'll be deprecating soon, and officially registered the existing post meta because the REST API needs to read it (the block editor) * update php doc comments * tweak comment on function to remove dark css theme mod after branch rebase * fixes based on more co-pilot feedback * tweak to meta login conditional * changes from PR feedback * added a comment to logic that shows page nav in content-page for clarity * Adjusting update script to handle swapping page template; New update script for fluid-width template to use template setting; putting latest color migration script in place before merge * Small wording change for editor panel * Incrementing db version so pre-release sites get the latest script run * Moving conditional page nav checks into the function --------- Co-authored-by: Kim Coleman <kim@strangerstudios.com>
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
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.
All Submissions:
Changes proposed in this Pull Request:
-- Note that this navigation only appears on child pages. Is this something we want to make clearer in the setting's label and in customizer?
How to test the changes in this Pull Request:
Other information:
Changelog entry