Skip to content

fix(shared): prevent composer modal content overflow with vertical scrolling - #6647

Open
Rishiii57 wants to merge 1 commit into
dailydotdev:mainfrom
Rishiii57:fix/composer-modal-overflow
Open

fix(shared): prevent composer modal content overflow with vertical scrolling#6647
Rishiii57 wants to merge 1 commit into
dailydotdev:mainfrom
Rishiii57:fix/composer-modal-overflow

Conversation

@Rishiii57

@Rishiii57 Rishiii57 commented Sep 12, 2026

Copy link
Copy Markdown

Fixes dailydotdev/daily#2146

Changes

  • Added overflow-y-auto to TextForm's flex container (packages/shared/src/components/post/composer/TextForm.tsx).
  • When writing or sharing a post/comment with long content or cover image in the "New post" modal, the content now scrolls cleanly within the card boundaries instead of overflowing outside the modal.

Events

No new tracking events introduced.

Experiment

No new experiments introduced.

Manual Testing

On those affected packages:

  • Have you done sanity checks in the webapp?
  • Have you done sanity checks in the extension?
  • Does this not break anything in companion?

Did you test the modified components media queries?

  • MobileL (420px)
  • Tablet (656px)
  • Laptop (1020px)

@vercel

vercel Bot commented Sep 12, 2026

Copy link
Copy Markdown

@Rishiii57 is attempting to deploy a commit to the Daily Dev Team on Vercel.

A member of the Team first needs to authorize it.

@CLAassistant

CLAassistant commented Sep 12, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

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.

🐛 BUG: "New post" share modal content overflows the card instead of scrolling (missing overflow-y)

2 participants