#232: remove duplicate filename under attachment pill, widen pill #234

Merged

fen merged 2 commits from fix-232 into dev 2026-09-10 22:06:25 +00:00

Conversation 0 Commits 2 Files Changed 2

+19 -11

fen commented 2026-09-10 22:02:54 +00:00

Collaborator

Copy Link

Copy Source

Fixes #232

Non-image attachments rendered an .attachment-preview img whose src fails to load (e.g. CSV served as text/plain), so the browser painted the alt text: the filename duplicated under the download pill. Preview is now rendered only for real image mimes (png/jpeg/gif/webp, matching the AttachmentImage flag). Also widened the chip filename max-width 320px -> 560px so long names show more before ellipsizing.

Verified locally: non-image paste serves no preview div; image paste keeps preview; go build/test pass.

Fixes #232 Non-image attachments rendered an .attachment-preview img whose src fails to load (e.g. CSV served as text/plain), so the browser painted the alt text: the filename duplicated under the download pill. Preview is now rendered only for real image mimes (png/jpeg/gif/webp, matching the AttachmentImage flag). Also widened the chip filename max-width 320px -> 560px so long names show more before ellipsizing. Verified locally: non-image paste serves no preview div; image paste keeps preview; go build/test pass.

fen added 1 commit 2026-09-10 22:02:54 +00:00

fix-232: drop attachment-preview img for non-image files (alt text painted duplicate filename); widen chip name max-width to 560px ca59c19fa4

fen added 1 commit 2026-09-10 22:05:24 +00:00

#232: file pill shows filename twice on non-image files; drop duplicate, let long names truncate wider and reveal on hover 9a5eaed00b

fen merged commit 1dd8a3b28a into dev 2026-09-10 22:06:25 +00:00

fen referenced this pull request 2026-09-10 22:09:22 +00:00

File View #232

fen referenced this pull request 2026-09-10 22:09:45 +00:00

File View #232

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

Assigned

Owner assigned: agent may pick this up

Critical

critical severity (pentest triage)

High

high severity (pentest triage)

In Progress

Agent actively working on this

Low

low severity (pentest triage)

Medium

medium severity (pentest triage)

Owner Review

Blocked on a human decision or design call - poslop must review before work proceeds

QA

Implementation done, awaiting independent review agent

No labels

Milestone

No items

No Milestone

Projects

Clear projects

No projects

Assignees

Clear assignees

fen poslop

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: poslop/palette#234