-
Notifications
You must be signed in to change notification settings - Fork 178
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
MWPW-147173 - Add gap below Aside notification headings #2255
MWPW-147173 - Add gap below Aside notification headings #2255
Conversation
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## stage #2255 +/- ##
==========================================
- Coverage 95.71% 95.70% -0.01%
==========================================
Files 166 166
Lines 44009 44009
==========================================
- Hits 42124 42120 -4
- Misses 1885 1889 +4 ☔ View full report in Codecov by Sentry. |
This pull request is not passing all required checks. Please see this discussion for information on how to get all checks passing. Inconsistent checks can be manually retried. If a test absolutely can not pass for a good reason, please add a comment with an explanation to the PR. |
Verified, testing details: MWPW-147173 |
notification
variant that do NOT have body copy. Without body copy, the heading butts up right against the CTAs. This PR fixes that.Resolves: MWPW-147173
Test URLs:
Milo:
Bacom: