-
Notifications
You must be signed in to change notification settings - Fork 9.1k
fix(app): tighten up session padding-top for mobile #11247
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
Conversation
|
The following comment was made by an LLM, it may be inaccurate: Based on my search, I found one potentially related PR: Related PR:
This PR might be related since it addresses overall mobile UX improvements, which could include session padding and layout adjustments. However, PR #11247 appears to be a more specific, targeted fix for session padding on mobile screens, so it's likely a distinct improvement rather than a duplicate. The search didn't find other PRs specifically addressing session padding or mobile responsive padding adjustments. |
bd214cc to
a7d1cc1
Compare
00637c0 to
71e0ba2
Compare
f1ae801 to
08fa7f7
Compare
|
imo a little bit of top padding here would still be nice, perhaps |
going from pt-6 -> pt-2 is a nice conservative improvement for mobile so i'm for it (no padding is my pref but will take the win :) ) |
What does this PR do?
tightens up the extra padding in primary session pane. Current styles give you more padding on smaller screens. Should be less padding on small screens IMO.
Mobile has much less space to work with, needs all the session space it can get.
Before:
After:
Video showing breakpoint transition:
Screen.Recording.2026-02-03.at.9.46.18.PM.mov
How did you verify your code works?
Tested
Fixes: #11245