fix(my-account): v2 demo polish pass#4689
Merged
thomasguillot merged 18 commits intoApr 30, 2026
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Polish pass on the My Account v2 demo prototype to better match Figma spacing/typography and improve a few interaction/layout details (donations list layout, newsletters header spacing, delete-account modal sizing, sidebar footer spacing).
Changes:
- Adds scoped v2-demo CSS overrides for newsletters row button widths, donations viewport anchoring + table styling, and sidebar footer spacing.
- Updates delete-account modal JS to toggle
newspack-ui__modal--smallon the success step and reset on close. - Tweaks v2-demo newsletters/donations templates to align section spacing, typography tokens, and the billing-history “Button Card” markup/icon.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
src/my-account-v2-demo/style.scss |
Adds new scoped layout/spacing overrides for newsletters, donations list/table, and sidebar footer. |
src/my-account-v2-demo/account-settings.js |
Toggles modal width class on success step and removes it on reset. |
includes/plugins/woocommerce/my-account/templates/my-account-v2-demo/newsletters.php |
Tightens header stack gap for section headers and “Unsubscribe from all” block. |
includes/plugins/woocommerce/my-account/templates/my-account-v2-demo/donations.php |
Restructures donations list layout for bottom-anchored billing history; adjusts typography and billing-history button card markup/icon. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
6 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
All Submissions:
Changes proposed in this Pull Request:
A polish pass on the My Account v2 demo prototype, addressing several visual nits surfaced while reviewing the merged Phases 2-9 against Figma. Stacks on top of
prototype/my-account-demo(umbrella PR #4679).Donations list (Figma
2636:46467)__stack--justify-betweenon the outer wrapper plus a viewport-basedmin-height(offset for admin bar + the.woocommercewrapper's vertical padding so the page never overflows).__font--lto__font--mto match the design token; recurring card amount likewise from--mto--s.--color-border), lighter row dividers (neutral-5),cursor: pointeron rows, and roomier padding.__stack--gap-11. Required overriding a v1:not(.__stack--gap-11) section + sectionrule that fired on our consecutive<section>elements (the outer wrapper isn't--gap-11, only the inner one is) and zeroing the__box's defaultmargin-bottomplus theme<h2>margins so the parent gap is the single source of inter-section spacing.<button>styled with__button--secondary --widerather than a__boxwrapped in custom CSS — picks up bg, padding, hover, focus-visible outline, and transition for free. Inner label block stretches to fill width viaflex: 1since newspack-ui has no flex-grow utility.Account settings - Delete account modal (Figma
4865:92398)__modal--small. Init keeps the wider default because of the alternatives list. Toggle is JS-driven on step transition and resets on close.Sidebar nav footer
spacer-1(8px) gap; previously sat too tight against each other.Newsletters list
min-widthso the right edge stays steady when toggling subscribed state and the loading-state spinner doesn't reveal a wider label after the flip. The bottom "Unsubscribe from all" CTA is unaffected (different context).__stack--gap-0.How to test the changes in this Pull Request:
Log in as administrator. Each surface lives behind the demo flag.
/my-account/donations/?my-account-v2-demo=1. Verify: Recurring + Previous section titles read at medium size; the gap between them is exactlyspacer-11(no extra phantom whitespace); billing history sits at the bottom of the viewport (not floating right under Previous donations); page height does not overflow the viewport. Hover the billing history card - it should darken toneutral-30and show a focus-visible outline on tab. Click it (no-op, just visual). Confirm the receipt icon matches the new SVG, and the URL bar does not pick up a#fragment when clicking./my-account/donations/?my-account-v2-demo=billing-history. Verify the inline billing history table replaces the Button Card and still sits at the bottom./my-account/donations/?my-account-v2-demo=no-donationsand=empty. Verify the empty notice + billing history layout still spaces correctly./my-account/newsletters/?my-account-v2-demo=1. Verify Sign up and Unsubscribe row buttons render at the same width (tight against each label, but identical right edge across rows). Toggle a row - the loading spinner sits inside the same-width pill before the label flips. The "Unsubscribe from all" row at the bottom stays its natural width. Confirm the section header (e.g., "Featured") sits flush against its description./my-account/edit-account/?my-account-v2-demo=1. Click Delete account. Verify the modal opens at default width with the alternatives list. Click Delete account inside the modal. Verify it transitions to the "Your account deletion has been requested" success step at--smallwidth. Close the modal and reopen - should reset to init at default width./my-account/<endpoint>/?my-account-v2-demo=1URL with the v1 sidebar showing FAQ / Contact us / Privacy Policy / Terms of Service / Sign out. Verify there is aspacer-1(8px) gap between each item./my-account/donations/(no flag) - should 404 / fall through to v1 unchanged. Visit/my-account/edit-account/- v1 form renders unchanged.Other information: