coracle/flotilla
1
6
Fork 24
Code Issues 42 Pull Requests 2 Actions Packages Projects 5 Releases Wiki Activity

Fix landscape layout on mobile devices #117

Closed
nayan9617 wants to merge 1 commits from nayan9617/flotilla:fix/landscape-viewport into dev
pull from: nayan9617/flotilla:fix/landscape-viewport
merge into: coracle:dev
Conversation 3 Commits 1 Files Changed 6
+6 -6
nayan9617 commented 2026-04-02 14:58:00 +00:00
Contributor
Copy Link
Copy Source

Summary

This PR addresses issue #108 and improves how Flotilla behaves in mobile landscape mode. A few full-height layout containers were using static viewport sizing, which can cause awkward cropping or overflow when the browser UI changes size on phones.

What changed

-Updated the main app shell to use dynamic viewport height.
-Aligned the affected sidebar and full-screen/detail views with the same sizing approach.
-Kept the change limited to layout code only, with no logic or data-flow changes.

Why this helps

-Mobile landscape is especially sensitive to browser chrome and safe-area changes.
-Dynamic viewport units track the visible screen more accurately than static screen height.
-The fix should make the app feel much more stable on Pixel-style devices and similar phones.

## Summary This PR addresses issue #108 and improves how Flotilla behaves in mobile landscape mode. A few full-height layout containers were using static viewport sizing, which can cause awkward cropping or overflow when the browser UI changes size on phones. ## What changed -Updated the main app shell to use dynamic viewport height. -Aligned the affected sidebar and full-screen/detail views with the same sizing approach. -Kept the change limited to layout code only, with no logic or data-flow changes. ## Why this helps -Mobile landscape is especially sensitive to browser chrome and safe-area changes. -Dynamic viewport units track the visible screen more accurately than static screen height. -The fix should make the app feel much more stable on Pixel-style devices and similar phones.
nayan9617 commented 2026-04-02 14:59:16 +00:00
Author
Contributor
Copy Link
Copy Source

hello @hodlbod,
This PR fixes the issue #108. Please review the same!

hello @hodlbod, This PR fixes the issue #108. Please review the same!
nayan9617 added 1 commit 2026-04-02 15:36:02 +00:00
make landscape mode behave better on phones b162d39f7b
nayan9617 force-pushed fix/landscape-viewport from cf8bbd4bab to b162d39f7b 2026-04-02 15:36:02 +00:00 Compare
hodlbod commented 2026-04-02 17:49:00 +00:00
Owner
Copy Link
Copy Source

This is worth doing, but it doesn't solve the original issue I was seeing, see below:

This is worth doing, but it doesn't solve the original issue I was seeing, see below:
signal-2026-04-02-104808.png
176 KiB
signal-2026-04-02-104808.png
nayan9617 commented 2026-04-02 18:04:46 +00:00
Author
Contributor
Copy Link
Copy Source

oh okay got it! let me try to resolve the same.

oh okay got it! let me try to resolve the same.
hodlbod closed this pull request 2026-04-04 00:17:57 +00:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
design
For issues related to design work. Development should be done in another issue and refer back to this one.
 dev
Issues related to development work
 easy
Issues that can be executed on by new contributors
 idea
For issues that aren't yet actionable
 priority
For issues related to high priority features and bugs
 usability
Issues that have to be validated with real live human beings.
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
hodlbod (hodlbod) mplorentz (Matt Lorentz) sakshamjain (Saksham Jain) sondreb triesap userAdityaa (Aditya Chaudhary)
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: coracle/flotilla#117
Delete Branch "nayan9617/flotilla:fix/landscape-viewport"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?