Mobile downloaded content display#1
Conversation
Co-authored-by: Navdeep Singh <Navdeepannu@users.noreply.github.com>
|
Cursor Agent can help with this pull request. Just |
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment Tip You can make CodeRabbit's review stricter and more nitpicky using the `assertive` profile, if that's what you prefer.Change the |
Fix mobile PDF/image blob download handling to ensure files are displayed correctly on mobile devices.
The previous blob download implementation was unreliable on mobile browsers (especially iOS Safari) because it clicked an
<a download>and immediately revoked the blob URL. This PR updates thedownloadBlobfunction to usewindow.openfor iOS devices and delaysURL.revokeObjectURLby 60 seconds to prevent race conditions where mobile browsers fail to load the blob if it's revoked too quickly.