Skip to content

Conversation

@kylecarbs
Copy link
Member

Adds a file picker button for attaching images on mobile devices.

Changes

  • Added attach button (📎) positioned in bottom-right of input area
  • Hidden file input with accept='image/*' and multiple selection support
  • Clicking button opens native file picker (works on mobile and desktop)
  • Reuses existing processImageFiles logic for consistent image handling
  • Button properly disabled when chat input is disabled

Why

Desktop users can paste images, but mobile devices need a button to open the file picker. This provides a consistent cross-platform experience for image attachments.

The button is always visible (on mobile and desktop) since it provides a convenient alternative to paste/drag-and-drop.


Generated with cmux

- Add attach button (📎) positioned in bottom-right of input area
- Hidden file input with accept='image/*' and multiple selection
- Clicking button opens native file picker
- Reuses existing processImageFiles logic for consistent handling
- Button disabled when chat input is disabled
Hide the attach button on desktop (>768px width) since desktop users
can paste or drag-and-drop images. Mobile users need the button to
access the file picker.
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

- Remove absolute positioning from attach button
- Make it a flex sibling of VimTextArea instead
- Add flex-shrink: 0 to maintain button size
- Button now sits beside the input instead of overlaying it
@kylecarbs kylecarbs merged commit bbde9c6 into main Oct 21, 2025
9 checks passed
@kylecarbs kylecarbs deleted the attach-mobile branch October 21, 2025 18:44
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