-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
"Customize CSS" should not be in Command Palette for non-block themes #64932
Comments
@MadtownLems Can you please specify the Theme where you found the issue. I tested it on Twenty Twenty One theme and was not able to reproduce it. Thanks! |
Hi there!
Unfortunately core doesn't maintain a theme that fits the description, but
the kind of theme talked about here:
https://gutenbergtimes.com/block-based-template-parts-for-classic-themes/
…On Mon, Sep 2, 2024, 5:24 AM Rutvik Savsani ***@***.***> wrote:
@MadtownLems <https://github.com/MadtownLems> Can you please specify the
Theme where you found the issue. I tested it on Twenty Twenty One theme and
was not able to reproduce it.
Thanks!
—
Reply to this email directly, view it on GitHub
<#64932 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAHGYBLUO2M5SVEVBJM6HTLZUQ4HHAVCNFSM6AAAAABNMUDAUSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMRUGM3TMMJQHA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
I can't reproduce the problem 🤔
|
Yes, exact those steps to reproduce.
No, not in the theme directory.
I will revisit this week with more information. Thank you!
…On Mon, Sep 2, 2024, 11:02 AM Aki Hamano ***@***.***> wrote:
I can't reproduce the problem 🤔
@MadtownLems <https://github.com/MadtownLems>
- Just to be sure, did you launch the command palette after accessing
Apperance > Patterns > All template parts > any template part?
- Is the theme you're having problems with exposed to the theme
directory <https://wordpress.org/themes/>?
—
Reply to this email directly, view it on GitHub
<#64932 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAHGYBOCWMUJ5W5WBW7ZXOTZUSDZVAVCNFSM6AAAAABNMUDAUSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMRVGAZTKMZSGQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Complete steps to reproduce:
AND adding something to theme.json to register one: `{ }` Once you do, you don't even need to do anything with Template Parts. Simply Add New Pattern, access the command palette, and find "Customize CSS" is now present |
Thanks for the reply. I was able to reproduce this issue. All that's needed to reproduce the issue seems to be to add a theme.json to the classic theme. Simply create a |
Thank you for confirming and for making the steps to reproduce even easier! I ran through what I believe are all the commands, and that was the only one that seemed out of place for our type of theme. Until core stops adding it, I'm trying to figure out how to unregister it myself: #65021 |
Description
"Customize CSS" is available as a command palette function in the context of editing Template Parts, but it shouldn't be there (as that shouldn't be relevant to non-block themes). The good news is that it just seems to fail silently, but it shouldn't be present in the UI.
Step-by-step reproduction instructions
Screenshots, screen recording, code snippet
Environment info
Tested on 6.6.1, both without Gutenberg and with Gutenberg 19.1.0
Please confirm that you have searched existing issues in the repo.
Please confirm that you have tested with all plugins deactivated except Gutenberg.
The text was updated successfully, but these errors were encountered: