Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
| <head> | ||
| <meta charset="UTF-8" /> | ||
| <meta name="viewport" content="width=device-width, initial-scale=1.0" /> | ||
| <meta name="viewport" content="width=device-width, initial-scale=1.0, viewport-fit=cover" /> |
There was a problem hiding this comment.
viewport-fit=cover added without safe area inset padding
Medium Severity
Adding viewport-fit=cover tells the browser to extend the layout viewport into device safe areas (home indicator, rounded corners), but no env(safe-area-inset-*) padding is applied anywhere in the app's CSS. Previously, without viewport-fit=cover, the browser automatically constrained the viewport to avoid these regions. Now content at the screen edges—especially the bottom home indicator area on modern iPads—can be obscured or hard to interact with in standalone mode.


What Changed
Improves the iPad/iOS PWA shell so the app stays in standalone mode more reliably and does not expose Safari chrome during normal use.
This change:
manifest.webmanifestwithdisplay: "standalone"index.htmlviewport-fit=cover100dvhin standalone/fullscreen display modes and fixes the body to the viewport in that modeWhy
On iPadOS, the installed PWA could still reveal browser chrome during scroll or certain interactions, which breaks the standalone-app feel.
This approach keeps the fix focused at the app-shell level instead of changing individual screens:
UI Changes
Behavior change only. In installed PWA mode on iPad/iOS, the app shell should remain in standalone presentation more consistently instead of allowing Safari chrome to appear.
Checklist
Before:

After:

Note
Low Risk
Low risk: changes are limited to PWA metadata and CSS app-shell sizing/scroll behavior, with the main risk being unintended layout/scroll regressions in standalone/fullscreen modes.
Overview
Improves iPad/iOS installed-PWA behavior by making the app explicitly standalone-capable and reducing cases where Safari chrome appears.
Adds a
manifest.webmanifest(withdisplay: "standalone") and wires it up via new mobile/Apple meta tags (includingviewport-fit=cover) inindex.html. Updates the global shell CSS to use100dvhand fix thebodyto the viewport only instandalone/fullscreendisplay modes, preventing overscroll/scroll-induced chrome.Written by Cursor Bugbot for commit 00c4f3e. This will update automatically on new commits. Configure here.
Note
Keep iPad PWA running in standalone mode
display: standaloneso browsers can install the app correctly.apple-mobile-web-app-capable, status bar style, title) to keep the app in standalone mode on iOS/iPadOS.display-mode: standalonemedia query that locks the body to the full dynamic viewport height and disables scroll bounce.viewport-fit=coverso content extends into safe areas on notched devices.Macroscope summarized 00c4f3e.