Skip to content

Fix iOS top bar breaking edge-to-edge webview layout#77

Draft
shanerbaner82 wants to merge 2 commits intomainfrom
fix/ios-topbar-edge-to-edge
Draft

Fix iOS top bar breaking edge-to-edge webview layout#77
shanerbaner82 wants to merge 2 commits intomainfrom
fix/ios-topbar-edge-to-edge

Conversation

@shanerbaner82
Copy link
Copy Markdown
Contributor

Summary

  • Changed top bar from safeAreaInset(edge: .top) to a ZStack overlay so the webview stays edge-to-edge behind the status bar and top bar
  • Updated --inset-top CSS injection to include nav bar height (52pt) when top bar is present, so .nativephp-safe-area padding still works
  • Removed top-bar-dependent ignoresSafeArea conditions — webview now always ignores all safe areas

Test plan

  • Verify red/colored backgrounds extend edge-to-edge behind status bar with top bar present
  • Verify content text has padding below the top bar (not flush against it)
  • Verify layout still works correctly without a top bar
  • Verify bottom nav layouts (iOS 26+ and iOS 18) are unaffected
  • Test with side nav — menu button should still open sidebar

🤖 Generated with Claude Code

shanerbaner82 and others added 2 commits March 30, 2026 16:04
The safeAreaInset(edge: .top) modifier pushed the webview down below
the top bar, leaving a white gap behind the status bar. Changed to a
ZStack overlay approach so the webview stays edge-to-edge and the top
bar sits on top. Updated --inset-top CSS injection to include the nav
bar height so .nativephp-safe-area padding still works correctly.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The top bar was in an outer ZStack which caused it to render on top of
the side navigation drawer. Moved it inside the NativeSideNavigation
content closure so the drawer properly covers it when opened.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@shanerbaner82 shanerbaner82 marked this pull request as draft March 31, 2026 02:20
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