Image view for pastes #229
Notifications
Total Time Spent: 2 hours
fen
2 hours
No due date set.
Dependencies
No dependencies set.
Reference: poslop/palette#229
Reference in New Issue
Block a user
there should be slight padding between the image and the pill body border to help alleviate cut off
QA PASS on PR #230 (verified live on palette-dev after merge
e059ca1and dev rollout).7wht6s:.attachment-barcomputed padding = 12px on all sides; image inset exactly 12px from bar top/left at both widths..attachment-previewcomputed border 0px / radius 0px (nested-border pill rule respected);.floatwrapper frames the pill.Closing issue, labels cleared, fix-229 branch deleted.
QA PASS on palette-dev. Merge commit
e059ca1(PR #230) deployed: served /static/app.css sha256 5442702d... matches git show origin/dev. CDP probe on a fresh image paste (8tvr2r): .attachment-bar padding 12px on all sides, image inset 12px from pill edges, .attachment-preview border 0 (inner corner-clipped border removed). Verified rendered output at 1400x900 and 375x812, no horizontal overflow at mobile. Closing; fix-229 branch deleted.QA PASS. Merged PR #230 (
86eb821) into dev; CI docker green on dev tip; dev rollout restarted. Rendered probe on https://palette-dev.archfox.org/j43r3u: .attachment-bar padding 12px, image loads, gap image->pill border = 12px top/left, attachment-preview border/radius removed. Issue closed, fix-229 branch deleted (merge auto-delete, 404 on explicit delete).QA PASS (PR #230). Deployed dev tip
e059ca1to palette-dev; served /static/app.css sha256 5442702d matches git show origin/dev. CDP probe of an image paste at 1400x900 and 375x812: .attachment-bar padding 12px, image inset 12px from the pill top/left on both widths, .attachment-preview border 0px (no more corner-clipped inner border). No CSP console errors.QA PASS verified live on palette-dev (dev tip
e059ca1, image for merge commit86eb821, CI docker success).Fix as specified in the issue. Merging #230, closing, cleaning up fix-229.
QA PASS (PR #231 / dev tip
e059ca1).git show origin/dev:internal/web/static/app.css(1274dafa...); rollout restart confirmed current image.