fix(my-account): v2 demo subscriptions polish + expired variant#4692
Merged
thomasguillot merged 1 commit intoApr 30, 2026
Merged
Conversation
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.
Polish pass on the v2-demo subscriptions list against Figma
2636:46117, plus a small fix to the detail-pageexpiredvariant. Targetsprototype/my-account-demo(umbrella tracker #4679). Devlog rows taggedSubscriptions list polishcapture the why for each call.Summary
Subscriptions list (Figma 2636:46117):
__font--l __font--bold→__font--m __font--bold(Figma 20/32; matches the typography migration donations + newsletters got earlier today).__font--m __font--bold→__font--s __font--bold(Figma 16/24 SemiBold).__font--normalfor explicit weight parity with donations.__stack--gap-9(48px) →__stack--gap-11(64px). Side benefit: satisfies v1's:not(.__stack--gap-11) section + section { margin-top: spacer-11 }exclusion so the second section's 64px margin-top stops compounding on top of the gap (prior cascade was producing ~112px between Active and Previous)..newspack-my-account-v2-demo__separatorprimitive —1px × 1emvertical hairline — replaces the literal|glyph between plan name and price (Figma renders this as a1px × 16px #ddddiv, not a character). Page-agnostic so future cards / headers can reuse without per-page rule duplication.Sidebar:
payment-information.phpshell. Slug stayspayment-methods(URL routing unchanged); only the display label changes.Subscription detail (
/sub-expired/):$is_expiredwas an undeclared fall-through — the page rendered with no header badge and no actions any time a reader clicked an Expired card in Previous.__badge--error).Subscription detail audit (Figma 2636:46149): came back clean — existing scoped CSS already pins header h2 to
--font-size-m, kv tables to--font-size-s, Billing history h2 to--font-size-s. No further pinning needed this pass.Test plan
/my-account/subscriptions/?my-account-v2-demo=1— section titles drop a notch, "Member · £71.16 / year" reads at 16/24 SemiBold, the gap between Active and Previous opens to 64px (no more compounded margin), the|glyph is replaced with a thin gray vertical hairline./my-account/subscriptions/sub-expired/?my-account-v2-demo=1— header shows EXPIRED badge next to product name + a Renew subscription button. Clicking Renew opens the renew-subscription modal./my-account/subscriptions/sub-cancelled/?my-account-v2-demo=1— unchanged (Cancelled badge + Renew, sanity check that the elseif didn't break the cancelled branch)./my-account/subscriptions/sub-001/?my-account-v2-demo=1— detail page renders unchanged (audit confirmed no changes needed).:php,:scss); build green.