Skip to content

Paste-to-attach: converting an already-attached URL keeps the one attachment - #352

Open
dmnyc wants to merge 1 commit into
fix/alt-flush-closefrom
fix/attach-dedup
Open

dmnyc wants to merge 1 commit into
fix/alt-flush-closefrom
fix/attach-dedup

Conversation

@dmnyc

@dmnyc dmnyc commented Sep 22, 2026

Copy link
Copy Markdown
Owner

Fixes #343. Stacked on #351 (→ #341) — the diff shown here is just this fix.

Accepting the paste offer pushed into draft.media unconditionally: paste the same image URL again, accept again, and the note carried the URL twice — two identical thumbnails, a duplicated line in the published content, two imeta tags for one picture.

Accepting a URL the draft already holds now converts — removes the prose line, keeps the existing entry and its description — instead of adding a second. Guard pinned in both composers.

Accepting the paste offer pushed into the media slot unconditionally,
so pasting the same URL a second time and accepting again appended the
image twice: two identical thumbnails, the URL twice in the published
content, and two imeta tags describing one picture.

Accepting a URL the draft already holds now takes the prose line and
keeps the existing entry — and the description already written for it —
rather than adding a second.

Fixes #343.
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