Skip to content

style: Improve PWA experience on iOS devices #162

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

dorage
Copy link

@dorage dorage commented Aug 17, 2025

Summary

  • Enhanced PWA experience for iOS devices by adding proper safe area handling
  • Fixed layout issues where content was hidden behind iOS home indicator and status bar

Problem

There was an issue where the update available button kept showing because version matching failed due to the remaining 'V' prefix. Additionally, on iOS devices running as PWA, content was being obscured by system UI elements.

Changes

  • Fixed app height to prevent overflow issues in PWA mode
  • Added ios-top-safe and ios-bottom-safe classes for proper safe area padding
pwa_before pwa_after
  • Added padding adjustments for:
    • Sidebar component (top/bottom safe areas)
sidebar_before sidebar_after
  • MainContent header (top safe area)
main_before main_after
  • FileTree component (bottom safe area with extra spacing)
    Forget to take screenshots

  • CodeEditor modal (top/bottom safe areas)

code_before code_after
  • QuickSettingsPanel (top safe area)
quick_before quick_after
  • New Project mobile form (bottom safe area positioning)
new_project_before new_project_after
  • Adjusted z-index for QuickSettingsPanel pull tab to prevent overlap issues

Before

before.mov

After

after.mov

Test plan

  • Test on iOS devices in PWA mode
  • Test on regular desktop/mobile browsers for regression
  • Verify all modals and overlays display correctly

@dorage dorage changed the title Improve PWA experience on iOS devices style: Improve PWA experience on iOS devices Aug 17, 2025
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