Skip to content

Fix offline banner overlapping mobile nav; add dismiss button#126

Merged
AdamJ merged 1 commit into
mainfrom
claude/fix-offline-banner-mobile-ZWenu
May 14, 2026
Merged

Fix offline banner overlapping mobile nav; add dismiss button#126
AdamJ merged 1 commit into
mainfrom
claude/fix-offline-banner-mobile-ZWenu

Conversation

@AdamJ
Copy link
Copy Markdown
Owner

@AdamJ AdamJ commented May 14, 2026

Summary

  • Raises the "App is ready to work offline" banner to bottom-20 on mobile so it clears the 64px bottom nav, and restores bottom-4 on md+ breakpoints where the nav is hidden
  • Adds an X dismiss button to the offline-ready banner so users can close it manually

Test plan

  • Trigger the offline-ready state (serve the PWA, go offline, reload) and confirm the banner appears above the mobile nav on a small viewport
  • Confirm the banner sits at the normal bottom-4 position on a desktop-width viewport
  • Click the X button and confirm the banner dismisses
  • Confirm the "Update Available" card (needRefresh path) is unaffected

https://claude.ai/code/session_01BcCgnBbCntUd4tpNYq9azJ


Generated by Claude Code

On mobile the banner was fixed at bottom-4, sitting behind the
bottom nav (h-16). Raise it to bottom-20 on mobile screens and
restore bottom-4 on md+ where the nav is hidden. Also adds an X
button so users can dismiss the banner manually.

https://claude.ai/code/session_01BcCgnBbCntUd4tpNYq9azJ
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying timetrackerpro with  Cloudflare Pages  Cloudflare Pages

Latest commit: fb5f64f
Status: ✅  Deploy successful!
Preview URL: https://a809ff97.timetrackerpro.pages.dev
Branch Preview URL: https://claude-fix-offline-banner-mo.timetrackerpro.pages.dev

View logs

@AdamJ AdamJ added duplicate This issue or pull request already exists enhancement New feature or request fix For general fixes or bundled fixes labels May 14, 2026
@AdamJ AdamJ merged commit ff30330 into main May 14, 2026
1 check passed
@AdamJ AdamJ deleted the claude/fix-offline-banner-mobile-ZWenu branch May 22, 2026 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

duplicate This issue or pull request already exists enhancement New feature or request fix For general fixes or bundled fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants