Closed Bug 1829366 Opened 1 year ago Closed 1 year ago

Downloading PDFs is unintuitive on android

Categories

(Firefox :: PDF Viewer, defect, P1)

All
Android
defect

Tracking

()

VERIFIED FIXED
114 Branch
Tracking Status
firefox113 --- verified
firefox114 --- verified

People

(Reporter: marco, Assigned: calixte)

References

Details

Attachments

(4 files)

+++ This bug was initially created as a clone of Bug #1823164 +++

We have decided on a slightly different solution to bug 1823164.

Assignee: nobody → cdenizet
Severity: -- → S3
Status: NEW → ASSIGNED
Priority: -- → P1
Depends on: 1829638

Comment on attachment 9330026 [details]
Bug 1829366 - Add a toolbar with a download button in pdf.js in GeckoView r=#pdfjs-reviewers,#geckoview-reviewers

Beta/Release Uplift Approval Request

  • User impact if declined: The way to download a pdf on Fenix is not really user-friendly right now and this patch will help to make it a way better.
  • Is this code covered by automated tests?: No
  • Has the fix been verified in Nightly?: No
  • Needs manual test from QE?: Yes
  • If yes, steps to reproduce: - check that there's a toolbar with a Download button on the top of the pdf viewer
  • open a pdf in Fenix
  • tap the button "Download" and the pdf should be saved in the Downloads folder on the device
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): It's a small change.
    The toolbar itself has been introduced in nightly few days ago with no specific issues and we just changed the css today to make more friendly with the help UI/UX team.
  • String changes made/needed:
  • Is Android affected?: Yes
Attachment #9330026 - Flags: approval-mozilla-beta?
Attachment #9329975 - Flags: approval-mozilla-beta?
Pushed by cdenizet@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/734186ff0bc5 Enable the new PDF.js toolbar on GeckoView r=geckoview-reviewers,amejiamarmol
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 114 Branch

Comment on attachment 9329975 [details]
Bug 1829366 - Enable the new PDF.js toolbar on GeckoView r=#geckoview-reviewers

Approved for 113.0b8.

Attachment #9329975 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Attachment #9330026 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Flags: qe-verify+

Verified as fixed on Nightly 114.0a1 and Beta 113.0b8.
Devices used:

Oneplus 5 (Android 10)
OnePlus A3000 (Android 6)
Lenovo Tab P11 Plus (Android 12)
Lenovo tab M10 (Android 10)
Xiaomi 12 Pro (Android 13)
Huawei MediaPad M2 (Android 5.1.1)
Google Pixel 7 (Android 13)

Status: RESOLVED → VERIFIED
Flags: qe-verify+
Attached video download.mp4 (deleted) —

:Adina, could you check that it's fine with a device with a low-resolution screen ?

Flags: needinfo?(apetridean)

:Calixte, we've also checked this issue on devices with low-resolution screens using the latest Nightly 114.0a1 from 26.04.2023 and Firefox 113.0b8.

Devices used:
Xperia 10 (Android 12)
Samsung Galaxy S9 (Android 8)
OnePlus A3000 (Android 6)

Flags: needinfo?(apetridean)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: