Skip to content

fix(membership): let Qgiv embed height flex on membership/supporting-member - #528

Merged
mohanadft merged 2 commits into
mainfrom
fix/qgiv-flexible-height
Aug 18, 2026
Merged

fix(membership): let Qgiv embed height flex on membership/supporting-member#528
mohanadft merged 2 commits into
mainfrom
fix/qgiv-flexible-height

Conversation

@mohanadft

Copy link
Copy Markdown
Collaborator

Summary

  • Removed the max-height: 720px CSS cap on the Qgiv payment iframe in QgivJoin.tsx
  • The embed is shared by /membership and /supporting-member (via LegacyJoinSection), so both pages now let the form's height flex to its actual content instead of clipping/leaving dead space at a fixed height

Test plan

  • Visit /membership, click "Become a member", reach the payment step, confirm the Qgiv form renders fully without clipping
  • Visit /supporting-member, repeat the same flow
  • Confirm no dead whitespace or scroll clipping on both desktop and mobile viewports

The 720px max-height clipped taller variants of the Qgiv form; letting
the iframe follow Qgiv's own resize behavior fixes both /membership and
/supporting-member since they share QgivJoin via LegacyJoinSection.
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 18, 2026

Copy link
Copy Markdown

Deploying website with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6c7ce53
Status: ✅  Deploy successful!
Preview URL: https://fd3d0a9e.website-aun.pages.dev
Branch Preview URL: https://fix-qgiv-flexible-height.website-aun.pages.dev

View logs

… unbounded

720px clipped taller form variants; removing the cap entirely left
Qgiv's unresized 1000px default showing as dead space. 950px splits
the difference.
@mohanadft
mohanadft merged commit c471f53 into main Aug 18, 2026
5 checks passed
@mohanadft
mohanadft deleted the fix/qgiv-flexible-height branch August 18, 2026 19:32
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.

1 participant