Skip to content

fix: keep the Landing newsletter prompt visible at the bottom - #132

Merged
suguanYang merged 1 commit into
mainfrom
fix/suguan/newsletter-prompt-bottom
Sep 22, 2026
Merged

suguanYang merged 1 commit into
mainfrom
fix/suguan/newsletter-prompt-bottom

Conversation

@suguanYang

Copy link
Copy Markdown
Contributor

Summary

  • Pin the Landing newsletter prompt to the bottom of the viewport so the header no longer covers its title on small screens.
  • Lift the prompt when another fixed control overlaps that corner, so the subscribe action stays fully visible.

Test plan

  • Open the landing page on a phone-width viewport and confirm the prompt sits at the bottom with the title, close control, and Subscribe button fully visible.
  • Scroll the page and confirm the prompt stays fixed to the bottom of the viewport.
  • Widen the window past 768px and confirm the prompt remains a bottom-right card.
  • Subscribe and dismiss still work, and the prompt respects the safe-area inset.

Made with Cursor

…viewport

The prompt sat under the header on small screens. Anchor it to the bottom and lift it clear of anything else fixed in that corner.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vercel

vercel Bot commented Sep 22, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
knowhere-dashboard-staging Ready Ready Preview Sep 22, 2026 2:39am UTC

Request Review

@suguanYang
suguanYang merged commit 86a5ec0 into main Sep 22, 2026
4 checks passed

This branch was successfully deployed

1 active deployment
Preview — 6c8715b9 Deployed Sep 22, 2026 by vercel[bot]
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