Skip to content

Conversation

@rebelchris
Copy link
Contributor

@rebelchris rebelchris commented Jan 24, 2026

Summary

Fixes spacing issue where the floating plus button was too close to the mobile footer navigation bar.

Changes

  • Increased bottom margin from bottom-24 (96px) to bottom-28 (112px)
  • Adds 16px more spacing between the plus button and footer bar

Before/After

Before: Plus button directly attached to footer bar
After: Proper spacing between plus button and footer bar

Testing

  1. Open mobile view
  2. Navigate to any page with the plus button
  3. Verify there's proper spacing between button and footer

Preview domain

https://fix-plus-button-spacing.preview.app.daily.dev

Increased bottom margin from bottom-24 to bottom-28 to prevent
the plus button from being too close to the navigation bar.
@rebelchris rebelchris requested a review from a team as a code owner January 24, 2026 04:24
@vercel
Copy link

vercel bot commented Jan 24, 2026

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

Project Deployment Review Updated (UTC)
daily-webapp Ready Ready Preview Jan 24, 2026 4:47am
1 Skipped Deployment
Project Deployment Review Updated (UTC)
storybook Ignored Ignored Jan 24, 2026 4:47am

Request Review

@CLAassistant
Copy link

CLAassistant commented Jan 24, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ rebelchris
❌ Chris


Chris seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

- Updated FooterWrapper to use bottom-safe-or-0 instead of !bottom-0
- This should fix iOS PWA safe area issues where plus button appears attached
- Also fixes webapp scroll behavior where button snaps to footer
Chris added 2 commits January 24, 2026 04:37
Reduced from bottom-28 to bottom-20 since FooterWrapper now
properly handles safe area with bottom-safe-or-0
- Reverted to fixed !bottom-0 so footer stays at bottom when scrolled
- Added pb-safe to handle iOS safe area with padding instead of positioning
- This should fix both scrolling behavior and iOS PWA safe area issues
- Use env(safe-area-inset-bottom, 0px) directly in inline style
- Keeps footer at fixed bottom-0 for proper scroll behavior
- Should work consistently across devices and PWA contexts
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.

4 participants