-
Notifications
You must be signed in to change notification settings - Fork 22
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
feat: color palette adjustments #1487
feat: color palette adjustments #1487
Conversation
a973574
to
1e8c12e
Compare
1e8c12e
to
30c1581
Compare
const PUBLIC_POST_ID_META = 'newspack_nl_public_post_id'; | ||
const NEWSPACK_NEWSLETTERS_CPT = 'newspack_nl_cpt'; | ||
const EMAIL_HTML_META = 'newspack_email_html'; | ||
const NEWSPACK_NEWSLETTERS_PALETTE = 'newspack_newsletters_color_palette'; |
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.
Nit: just to be clear that the variable holds a meta key name and not "a color palette".
const NEWSPACK_NEWSLETTERS_PALETTE = 'newspack_newsletters_color_palette'; | |
const NEWSPACK_NEWSLETTERS_PALETTE_META_KEY = 'newspack_newsletters_color_palette'; |
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.
Fixed in 346714c
🎉 This PR is included in version 2.15.0-hotfix-tracking-logfile-fatals-and-mc-interests.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
# [2.15.0-alpha.4](v2.15.0-alpha.3...v2.15.0-alpha.4) (2024-05-20) ### Bug Fixes * respect buttons alignment and sizing ([#1492](#1492)) ([0a63a5a](0a63a5a)) ### Features * **layouts:** save sender and audience in the layout ([#1461](#1461)) ([aec7bce](aec7bce)) * newsletters color palette adjustments ([#1487](#1487)) ([2b0ecac](2b0ecac)) * **ras-acc:** add helper method to identify premium lists ([#1502](#1502)) ([2473ea7](2473ea7))
🎉 This PR is included in version 2.15.0-alpha.4 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
# [2.15.0](v2.14.4...v2.15.0) (2024-05-28) ### Bug Fixes * bad auto-merge ([fdfe15f](fdfe15f)) * **newsletter-editor:** tag/segment selection ([2589d3a](2589d3a)) * respect buttons alignment and sizing ([#1492](#1492)) ([0a63a5a](0a63a5a)) ### Features * **layouts:** save sender and audience in the layout ([#1461](#1461)) ([aec7bce](aec7bce)) * newsletters color palette adjustments ([#1487](#1487)) ([2b0ecac](2b0ecac)) * notify site admin on campaign send failure ([#1434](#1434)) ([4e95283](4e95283)) * **ras-acc:** add helper method to identify premium lists ([#1502](#1502)) ([2473ea7](2473ea7))
🎉 This PR is included in version 2.15.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
All Submissions:
Changes proposed in this Pull Request:
This PR is paired with the following Plugin PR: Automattic/newspack-plugin#3083
This makes the newsletter color palette a bit more flexible to work with by:
How to test the changes in this Pull Request:
These changes should not change any of the existing behavior of newsletters.
Other information: