-
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
Template Editor Mode: Hide editor mode switcher #37359
Conversation
Size Change: +17 B (0%) Total Size: 1.11 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.
I feel that this is the right decision, short term. The PR works well in my manual test.
Thanks for testing, @carolinan. I agree this is a short-term solution. Let's try to solve #22528 in the next release. @noisysocks should we backport this into beta 4? |
Is there a decision about the backport? |
|
I think it's okay to backport this. It's not a significant code change. |
Since it is currently broken, I think it would be more confusing if the short term fix is not backported. |
Description
Resolves #37320.
Code Editor isn't supported in Site Editor nor correctly works in the Template Editing Mode. PR hides editor mode switcher. We should enable this feature back once Code Editor mode is fully supported - #22528.
How has this been tested?
Screenshots
Types of changes
Bugfix
Checklist:
*.native.js
files for terms that need renaming or removal).