-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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 theme.json to export file #39048
Conversation
To the point of collapsing user/theme data here is how we have experimented with it in a Block Creation plugin. This is the resulting JSON I expect for 'theme provided' and 'user-overrided' values. |
@Mamaduka As we talked yesterday, I've pushed some changes that add a new method |
Thank you, @oandregal 🦸 |
@oandregal I made small code style change in 7f4f48d. Should we land this to get more feedback from testing? |
@scruffian, I was also thinking about changing the text. Any suggestions? |
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.
LGTM
Size Change: +405 B (0%) Total Size: 1.15 MB
ℹ️ View Unchanged
|
Thanks, @scruffian. |
Description
Resolves #27528
Supersedes #36619
Adds global style settings (as theme.json) to the site export file.
PR also overrides the core REST API endpoint for export functionality.
Testing Instructions
Types of changes
Enhancement
Checklist:
*.native.js
files for terms that need renaming or removal).