Attach every image from a multi-file Finder paste
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 52/100
- Issue type
- Bug
- Clarity
- Mostly clear
- Activity status
- Active
- Tech stack
- macos, typescript
- Domain
- desktop
Research direction
Start from the paste handling added in PR #104 (the clipboard-event path feeding useAttachments().add) and compare it with the drop path that already accepts multiple files. Then locate the desktop shell's native bridge/IPC for delivering files to the web view, since the fix needs a native macOS pasteboard read of public.file-url items. Done when a Finder multi-file copy pastes every image like a drop does, while single-file and in-memory clipboards from #104 keep working.
Written by the indexing model from the issue text.
Description
Follow-up to #91 and PR #104. That PR attaches pasted images, but a Finder copy of several files attaches only one image.
Confirmed behavior
A standalone WKWebView probe was given a real Finder copy of a.png, b.png and c.txt, all selected in Finder and copied with Cmd+C. The pasteboard held one public.file-url item per file, but the page's paste event saw only:
types: ["Files"], files: ["a.png image/png 14451"], items: ["file image/png"]
- WebKit exposes only the first copied file to
clipboardData.files/itemsand hides the file-name text. - The Ace-dev build from PR #104 attached one image for a single file-URL paste, as expected.
- #104 passes every exposed image file to
useAttachments().addin one call, so in-memory image clipboards with several images aren't affected. Only multi-file Finder copies lose files.
Expected
Pasting several images copied in Finder attaches all of them, as dropping them does. This probably needs the desktop shell to read the pasteboard's file URLs natively and hand the image files to the composer. The web clipboard event can't see them.
Related: #26 (non-image uploads). Part of #5.
- Dominant language
- TypeScript
- Stars
- 1
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- No contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from githubnext/ace2
-
Difficulty 1/5 Under an hour Newbie friendliness 94/100
githubnext/ace2#487 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 90/100
githubnext/ace2#486 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 67/100
githubnext/ace2#459 ·
-
Difficulty 1/5 Under an hour Newbie friendliness 88/100
githubnext/ace2#437 ·
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
githubnext/ace2#442 ·
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
ccch1mneyyy/dsh-TUI#1631 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
MiniMax-AI/minimax-code#507 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
-
Transient OAuth login failure persists in getLoginStatus and masks active account in dashboardMay be free again A pull request for this issue was closed without being merged. Openaccount-pool bug gui
Difficulty 2/5 1-3 hours Newbie friendliness 67/100
lidge-jun/opencodex#7378 · 2 comments ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
Maintainers usually reply within 1 day