Skip to content

Feat/fe payment page a11y#604

Open
AbuJulaybeeb wants to merge 7 commits into
Pulsefy:mainfrom
AbuJulaybeeb:feat/fe-payment-page-a11y
Open

Feat/fe payment page a11y#604
AbuJulaybeeb wants to merge 7 commits into
Pulsefy:mainfrom
AbuJulaybeeb:feat/fe-payment-page-a11y

Conversation

@AbuJulaybeeb

Copy link
Copy Markdown
Contributor

What was done

Implemented PayPageShell to provide a consistent page container, a skip-to-content accessibility link, and a semantic

tag for screen readers.
Standardized text contrast across all states (Active, Expired, Paid, Refunded, Loading, Error) to support both light and dark modes out-of-the-box using the --background and --foreground theme tokens.
Replaced non-existent internal icons with lucide-react icons.
Removed generateMetadata from layout.tsx to resolve the Next.js parameter parsing crash on the server and safely refactored the page into a server-side entry point.
Created an elegant top-right NetworkBadge with theme-aware amber/emerald/blue highlighting.
Why it was done To ensure the public payment links represent QuickEx well during contributor demos, fulfilling the WCAG AA contrast standards and ensuring keyboard accessibility on the transaction pipeline.

How it was verified Added a mockState query parameter to cleanly test each payment step without depending on the backend DB. Ran an automated browser subagent through mockState=ACTIVE|EXPIRED|PAID|REFUNDED|ERROR to verify visual aesthetics and generate session screenshots/recordings. Verified local Next.js turbopack builds and completed eslint checks with exit code 0.

Closes #575

@drips-wave

drips-wave Bot commented Jun 26, 2026

Copy link
Copy Markdown

@AbuJulaybeeb Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@Cedarich

Copy link
Copy Markdown
Contributor

@AbuJulaybeeb fix conflict

@Cedarich

Copy link
Copy Markdown
Contributor

Fix workflow

@AbuJulaybeeb AbuJulaybeeb force-pushed the feat/fe-payment-page-a11y branch from 1722b0c to 90ffbc9 Compare June 26, 2026 07:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FE-40: Public Payment Page Theme + Accessibility Pass

2 participants