Skip to content

fix(web): tighten the composer on narrow screens - #15

Open
Antisophy wants to merge 1 commit into
CyberShadow:masterfrom
Antisophy:fix/mobile-composer-spacing
Open

fix(web): tighten the composer on narrow screens#15
Antisophy wants to merge 1 commit into
CyberShadow:masterfrom
Antisophy:fix/mobile-composer-spacing

Conversation

@Antisophy

Copy link
Copy Markdown
Contributor

The composer keeps its desktop spacing on a phone, where it takes a noticeable slice of the screen: 12px/16px around the box, 10px/14px inside the textarea, and buttons padded 10px/20px.

This halves that below 768px and gives the composer's buttons equal padding on all sides.

The textarea's min-height: 40px is one line's worth at the full padding, so it drops to 30px to match. Left alone it holds open roughly ten pixels of dead space under a single line, which disappears only once word wrap outgrows the floor, so the box looks correct while typing a long message and wrong while typing a short one.

Nothing outside the max-width: 768px block changes, so desktop layout is untouched.

The composer keeps its desktop spacing on a phone, where it takes a
noticeable slice of the screen: 12px/16px around the box, 10px/14px inside
the textarea, and buttons padded 10px/20px.

Halve that below 768px and give the buttons equal padding on all sides. The
textarea's 40px floor is one line's worth at the full padding, so it drops to
30px to match; left alone it would hold open roughly ten pixels of dead space
under a single line, disappearing only once wrapping outgrew the floor.
@Antisophy

Antisophy commented Aug 21, 2026

Copy link
Copy Markdown
Contributor Author

One of five independent changes for using cydo on a phone, split apart so each can be judged on its own:

They touch different code and can merge in any order. #15 and #17 both add rules inside the same @media (max-width: 768px) block, so whichever lands second may need a trivial context rebase; nothing semantic overlaps.

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