poslop/palette
1
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases 2 Wiki Activity

Fix #229: padding between image and pill border #231

Closed
fen wants to merge 0 commits from fix-229 into dev
pull from: fix-229
merge into: :dev
Conversation 1 Commits - Files Changed -
fen commented 2026-09-10 19:56:48 +00:00
Collaborator
Copy Link
Copy Source

Adds 12px padding to .attachment-bar so the image preview no longer touches the pill border, and drops the .attachment-preview inner border/radius (the .float wrapper already frames the pill; the inner border got corner-clipped per the nested-border rule).

Verified locally: go build/test pass, rendered CDP probe shows 13px (12px padding + 1px border) clearance between the image and the pill border on all sides.

Ref #229

Adds 12px padding to .attachment-bar so the image preview no longer touches the pill border, and drops the .attachment-preview inner border/radius (the .float wrapper already frames the pill; the inner border got corner-clipped per the nested-border rule). Verified locally: go build/test pass, rendered CDP probe shows 13px (12px padding + 1px border) clearance between the image and the pill border on all sides. Ref #229
fen closed this pull request 2026-09-10 20:00:00 +00:00
fen deleted branch fix-229 2026-09-10 20:00:00 +00:00
fen commented 2026-09-10 20:02:18 +00:00
Author
Collaborator
Copy Link
Copy Source

Superseded duplicate: this PR's head commit 86eb821 was already merged into dev via PR #230. Live-verified on palette-dev: served /static/app.css sha256 matches dev tip, and a CDP probe on an image paste shows the preview 13px from the pill top/left edge (12px padding) with border/radius 0 on .attachment-preview. QA PASS.

Superseded duplicate: this PR's head commit 86eb821 was already merged into dev via PR #230. Live-verified on palette-dev: served /static/app.css sha256 matches dev tip, and a CDP probe on an image paste shows the preview 13px from the pill top/left edge (12px padding) with border/radius 0 on .attachment-preview. QA PASS.

Pull request closed

This pull request cannot be reopened because the branch was deleted.
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
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: poslop/palette#231