-
Notifications
You must be signed in to change notification settings - Fork 34
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
PLANET-7561 Move Blank Page pattern to the theme #2316
Conversation
db7d144
to
4305684
Compare
/unhold 379b279c-69b3-4022-b5f1-81024c641e70
Test instance is ready 🚀🌑 umbriel | admin | blocks report | CircleCI | composer-local.json ⌚ 2024.07.01 10:22:41 |
DeepDive::class, | ||
HighlightedCta::class, | ||
Issues::class, |
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've reorganised these so that they are in alphabetical order
@@ -17,17 +17,15 @@ | |||
class DeepDive extends BlockPattern | |||
{ | |||
/** | |||
* Returns the pattern name. | |||
* @inheritDoc |
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.
This notation allows us to avoid repeating ourselves
It used to be in the plugin
4305684
to
c5246e1
Compare
/unhold fde75baf-9e52-4c47-a508-f994bcec5bc1
Description
See PLANET-7561
It used to be in the plugin, see related PR
Testing
Either on local or on the umbriel test instance, you should be able to add a Blank Page pattern just like before.