-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Patterns: Reveal the Featured pattern category, and make it the first in the list #49907
Patterns: Reveal the Featured pattern category, and make it the first in the list #49907
Conversation
This PR does not affect the size of JS and CSS bundles shipped to the user's browser. Generated by performance advisor bot at iscalypsofastyet.com. |
Caution: This PR affects files in the Editing Toolkit Plugin on WordPress.com D56791-code has been created so you can easily test it on your sandbox. See this FieldGuide page about developing the Editing Toolkit Plugin for more info: PCYsg-ly5-p2 |
Thanks for testing @glendaviesnz! I've updated it to sort by the label instead of the slug, and to be case insensitive just in case there are strings with the wrong case (like we have for |
apps/editing-toolkit/editing-toolkit-plugin/block-patterns/class-block-patterns-from-api.php
Outdated
Show resolved
Hide resolved
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.
Works for me - over to you if you want to explore the union operator option mentioned
e97e7ac
to
cd4fe68
Compare
Changes proposed in this Pull Request
Screenshot
Testing instructions
label
instead of the slug 🤔Related to 120-gh-Automattic/view-design
CC: @joanrho