Closed
Bug 1973142
Opened 2 months ago
Closed 2 months ago
Save to PDF shows dialog to save a zero byte file
Categories
(Firefox for Android :: Downloads, defect)
Tracking
()
VERIFIED
FIXED
142 Branch
People
(Reporter: jonalmeida, Assigned: giorga)
References
(Regression)
Details
(Keywords: regression, Whiteboard: [fxdroid] [group4])
Attachments
(4 files)
Steps to reproduce
- Navigate to duck.com
- Open the menu and choose 'Save to PDF' (or via the share icon).
- Observe the dialog.
Expected behaviour
- The dialog should have no mention of file size if it's not possible to determine it.
Actual behaviour
- The dialog shows a size of 0 bytes instead. See screenshot.
Device information
- Firefox version: 141.0a1-20250616214059
- Android device model: Pixel 8a
- Android OS version: Android 16
Any additional information?
n/a
Comment 1•2 months ago
|
||
This is likely a regression from Bug 1962682, when we switched from a bottom sheet to an alert dialog. I also attached a screenshot of how the original bottom sheet displayed this kind of scenario - the file size is not shown as mentioned in the expected behaviour.
Comment 2•2 months ago
|
||
Updated•2 months ago
|
Comment 3•2 months ago
|
||
:giorga, since you are the author of the regressor, bug 1962682, could you take a look?
For more information, please visit BugBot documentation.
Flags: needinfo?(giorga)
Assignee | ||
Updated•2 months ago
|
Assignee: nobody → giorga
Flags: needinfo?(giorga)
Assignee | ||
Comment 4•2 months ago
|
||
Pushed by giorga@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/7e8e43182965
https://hg.mozilla.org/integration/autoland/rev/fb49e4eae195
Save dialog should have no mention of file size if it's not possible to determine it. r=android-reviewers,npoon,android-l10n-reviewers,delphine
Comment 6•2 months ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 2 months ago
status-firefox142:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 142 Branch
Comment 7•2 months ago
|
||
Set release status flags based on info from the regressing bug 1962682
status-firefox140:
--- → affected
Comment 8•2 months ago
|
||
The patch landed in nightly and beta is affected.
:giorga, is this bug important enough to require an uplift?
- If yes, please nominate the patch for beta approval.
- See https://wiki.mozilla.org/Release_Management/Requesting_an_Uplift for documentation on how to request an uplift.
- If no, please set
status-firefox141
towontfix
.
For more information, please visit BugBot documentation.
Flags: needinfo?(giorga)
Assignee | ||
Updated•2 months ago
|
Flags: needinfo?(giorga)
Updated•2 months ago
|
Updated•2 months ago
|
Flags: qe-verify+
Comment 9•2 months ago
|
||
Verified as fixed on the latest Nightly 142.0a1 from 5.07.2025 with Poco M4 Pro (Android 12).
Updated•2 months ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•