Page Editor: Fix Text global styles for block themes in simple sites #90784
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.
Related to #85798 p1715801793910199-slack-C03TY6J1A
Proposed Changes
normalize_theme_slug
fromadd_default_editor_styles_for_classic_themes
Note
normalize_theme_slug()
is not required becauseget_stylesheet()
returns the theme directory correctly prefixed for simplepub/{ theme directory }
and atomic{ theme directory }
.Why are these changes being made?
Testing Instructions
bedrock
themeinstall-plugin.sh etk fix/default-styles-for-classic-themes-in-simple
Styles > Typography > Text
and set the font family toPixeloid mono
https://wordpress.com/pages/{ SITE }
, make sure it has some regular paragraphsTest on Atomic:
https://wordpress.com/pages/{ SITE }
, make sure it has some regular paragraphsPre-merge Checklist