Open
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Member
Author
|
Updated the remaining Nostr grant/client posts that were still using the generic recurring CTA so they now render the purple v2 banner for
I also checked the Vercel preview on those paths and confirmed they now use the purple |
Member
Author
Prevent MDX prose from styling the banner link and flex overflow from pushing copy past the wedge on narrow widths; relax mobile nowrap.
Remove deprecated center wrappers site-wide under data/. Donate banners already use horizontal margin auto; add mx-auto on fund/ops modal buttons so they stay centered in prose without <center>.
Shrink-wrap with max-width min(480px, 100%), min-width 0 in narrow prose; scale wedge slightly under clip and keep rotate on hover to avoid seams.
Newsletter and blog content use Tailwind typography; the banner link alone could sit like a block glued to the prose margin. Add a full-width flex shell with justify-center and move vertical margins to the shell so layout matches legacy <center> behavior without touching MDX.
Avoid flex column weighting that truncated the CTA word; rely on pseudo geometry + matching hover rotation at max-width 480px.
Member
Author
|
I think I fixed most issues now, so I'll mark this as ready for review @Arvin21M |
Arvin21M
requested changes
May 8, 2026
Contributor
There was a problem hiding this comment.
Mobile view: button looks tall.
The following each contain at least one of the old donation buttons.
- /blog/thirteenth-wave-of-bitcoin-grants
- /blog/fourteenth-wave-of-bitcoin-grants
- /blog/twelfth-wave-of-bitcoin-grants
- /blog/thirteenth-wave-of-nostr-grants
- /blog/fourteenth-wave-of-nostr-grants
- /donate
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.


This routes legacy
DonateRecurringButtonusage through the v2 recurring donation banner so older static pages and blog posts pick up the new CTA without touching the landing page.nostrdonations to the purple v2 banner and preserves custom CTA textCloses #715
Build preview: