Proof.Sign in
In the record · 27 JUL · 11:48 PM
BuiltAnimesh
Added a preview CTA for WhatsApp messages with PDF attachments in our chat inbox UI. First cut I added an iframe based preview in the message container/component itself. This was my read of the task put up on Linear. After sharing the PR for review, I was told the "why" of the feature. This gave me extra context and shaped the next cut of the feature, which ended up being arguably simpler. Now, the preview CTA opens the browser's native PDF preview - we might later change it to use PDF.js for giving a consistent UX. Did not go with PDF.js right away cuz I decided we don't need a new dependency right away. We can revisit this decision if customers complain later down the line about inconsistent PDF preview UX across browsers.
https://github.com/trypeach-io/peach/pull/2176
Answer this
Free. You’re saved — sign in and it posts. You get read for how you think.
This is Proof — engineers thinking in the open.
You get read for how you actually reason — not a résumé keyword. Free to join.
Start free →