Podcast: align bottom Stats cards with wpcom Stats card module look#48761
Podcast: align bottom Stats cards with wpcom Stats card module look#48761robertbpugh wants to merge 4 commits into
Conversation
Restyles the SectionCard wrapper used by Top episodes, By app, and By location so the cards read like the Calypso Stats card modules: real 1px border, no box-shadow, 20px title, and 24px content padding. Scoped off the Downloads chart card so its layout stays unchanged. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
|
Are you an Automattician? Please test your changes on all WordPress.com environments to help mitigate accidental explosions.
Interested in more tips and information?
|
|
Thank you for your PR! When contributing to Jetpack, we have a few suggestions that can help us test and review your patch:
This comment will be updated as you work on your PR and make changes. If you think that some of those checks are not needed for your PR, please explain why you think so. Thanks for cooperation 🤖 🔴 Action required: Please include detailed testing steps, explaining how to test your change, like so: 🔴 Action required: We would recommend that you add a section to the PR description to specify whether this PR includes any changes to data or privacy, like so: Follow this PR Review Process:
If you have questions about anything, reach out in #jetpack-developers for guidance! |
There was a problem hiding this comment.
Pull request overview
This PR updates the Podcast dashboard Stats tab styling so the non-chart stats cards (Top episodes, By app, Locations, and similar “bottom” cards) visually match the WordPress.com “card module” appearance while keeping the downloads chart card unchanged.
Changes:
- Restyles
.podcast-stats__section-card(excluding.podcast-stats-chart) to use a 1px border, 5px radius, no shadow, and updated internal padding. - Increases section title typography within the affected cards to a larger 20px/500-weight style.
- Adds a changelog entry documenting the visual alignment update.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
projects/packages/podcast/src/dashboard/stats/style.scss |
Applies the new card-module styling to non-chart Stats cards while explicitly excluding the downloads chart card. |
projects/packages/podcast/changelog/update-podcast-stats-bottom-cards-wpcom-styling |
Documents the UI styling change for the Podcast Stats cards. |
| Significance: patch | ||
| Type: changed | ||
|
|
||
| Podcast: visually align the Top episodes, By app, and Locations Stats cards with the wpcom Stats card module look (real border, larger header, 24px padding). |
Code Coverage SummaryThis PR did not change code coverage! That could be good or bad, depending on the situation. Everything covered before, and still is? Great! Nothing was covered before? Not so great. 🤷 |
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
|
@copilot apply changes based on the comments in this thread |
Agent-Logs-Url: https://github.com/Automattic/jetpack/sessions/8bd8ddd0-a863-47f5-8657-d1d07b160ff0 Co-authored-by: robertbpugh <52668747+robertbpugh@users.noreply.github.com>
Addressed in 7c9a441: updated the changelog entry to use user-facing wording (“WordPress.com”) and aligned it with the existing “Stats tab:” style from the review thread. This is a changelog-only update, so there are no UI changes to screenshot. |
Summary
Visually align the Top episodes, By app, and Locations Stats cards with the wpcom Stats card module look so the Stats tab reads as one consistent surface.
&:not(.podcast-stats-chart)so the Downloads chart card stays as-is.Test plan