Print preview always shows 30% scale when it is actually Shrink To Fit

VERIFIED FIXED in Firefox 63

Status

()

P1
normal
VERIFIED FIXED
4 months ago
4 months ago

People

(Reporter: gordon, Assigned: dao)

Tracking

({regression})

63 Branch
mozilla65
regression
Points:
---

Firefox Tracking Flags

(firefox-esr60 unaffected, firefox63 verified, firefox64 verified, firefox65 verified)

Details

Attachments

(1 attachment)

(Reporter)

Description

4 months ago
User Agent: Mozilla/5.0 (Windows NT 10.0; rv:63.0) Gecko/20100101 Firefox/63.0

Steps to reproduce:

If I view a web page and select Print Preview the scale ALWAYS shows 30% even though it is actually giving me a Shrink To Fit. If I select 30% from the drop down menu the scale changes, proving it was NOT at 30% but was at Shrink To Fit.


Actual results:

Regardless of the print scale I select the drop down menu ALWAYS shows 30% next time but is actually performing Shrink To Fit. The default scale shown is WRONG, showing 30% when it is actually "Shrink To Fit". Naturally when you use the drop down menu to select Shrink To Fit nothing changes as this is already what is happening.


Expected results:

The print scale used to stay at the last value chosen and if Shrink To Fit was selected then this is what would have been shown in the closed drop down box.

On version 63.0 the print scale defaults to Shrink To Fit. I am happy with this rather than it sticking to the last scale set but it should show "Shrink To Fit" in the unopened drop down scale menu. Currently it shows 30% which is WRONG.

Comment 1

4 months ago
[Tracking Requested - why for this release]:
Status: UNCONFIRMED → NEW
status-firefox63: --- → wontfix
status-firefox64: --- → affected
status-firefox65: --- → affected
status-firefox-esr60: --- → unaffected
Component: Untriaged → Print Preview
Ever confirmed: true
Keywords: regression
Product: Firefox → Core

Comment 2

4 months ago
Regression window:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=81f6183eff6587a4e57bfa02bb0c2df9ae9ba2ab&tochange=13ec544a537b1339d262b81845ab65802367d6dd

Regressed by:414ca83ed73f	Dão Gottwald — Bug 1473265 - Clean up print preview toolbar ids and classes. r=bgrins

@Dão, Your patch seems to cause the regression. Could you please look into this?
Blocks: 1473265
Flags: needinfo?(dao+bmo)
(Assignee)

Updated

4 months ago
Assignee: nobody → dao+bmo
Status: NEW → ASSIGNED
Component: Print Preview → Printing
Flags: needinfo?(dao+bmo)
Product: Core → Toolkit
(Assignee)

Updated

4 months ago
Priority: -- → P1
(Assignee)

Comment 3

4 months ago
Created attachment 9019975 [details]
Bug 1501952 - Make mScaleCombobox point to the right element. r?bgrins

Comment 4

4 months ago
Pushed by dgottwald@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0e96d6002321
Make mScaleCombobox point to the right element. r=bgrins
(Assignee)

Updated

4 months ago
Flags: qe-verify+
Summary: The print dialogue always shows 30% scale when it is actually Shrink To Fit → Print preview always shows 30% scale when it is actually Shrink To Fit

Comment 5

4 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/0e96d6002321
Status: ASSIGNED → RESOLVED
Last Resolved: 4 months ago
status-firefox65: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
(Assignee)

Comment 6

4 months ago
Comment on attachment 9019975 [details]
Bug 1501952 - Make mScaleCombobox point to the right element. r?bgrins

[Beta/Release Uplift Approval Request]

Feature/Bug causing the regression: Bug 1473265

User impact if declined: see comment 0

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: see comment 0

List of other uplifts needed: None

Risk to taking this patch: Low

Why is the change risky/not risky? (and alternatives if risky): trivial fix

String changes made/needed:
Attachment #9019975 - Flags: approval-mozilla-beta?
Comment on attachment 9019975 [details]
Bug 1501952 - Make mScaleCombobox point to the right element. r?bgrins

Trivial fix for print preview, approved for 64.0b5.
Attachment #9019975 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

Comment 8

4 months ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-beta/rev/a265524c6b4e
status-firefox64: affected → fixed

Comment 9

4 months ago
If the user has disabled Shrink to Fit and uses a standard or custom percentage -- which Firefox remembers -- the next use of Print Preview also will show 30% instead of the true value that will be used. It may be difficult for the average user to know what scale they used previously (they would need to open the Page Setup dialog).

Any chance of uplift if there is a Firefox 63.0.1?

SuMo thread: https://support.mozilla.org/questions/1238497
(Assignee)

Comment 10

4 months ago
(In reply to jscher2000 from comment #9)
> Any chance of uplift if there is a Firefox 63.0.1?

Ryan?
Flags: needinfo?(ryanvm)
Seems like a decent candidate, though Fx63 is Pascal's release. Go ahead and nominate the patch for release uplift :).
status-firefox63: wontfix → fix-optional
Flags: needinfo?(ryanvm)
(Assignee)

Comment 12

4 months ago
Comment on attachment 9019975 [details]
Bug 1501952 - Make mScaleCombobox point to the right element. r?bgrins

[Beta/Release Uplift Approval Request]

Feature/Bug causing the regression: Bug 1473265

User impact if declined: see comment 0

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: see comment 0

List of other uplifts needed: None

Risk to taking this patch: Low

Why is the change risky/not risky? (and alternatives if risky): trivial fix

String changes made/needed:
Attachment #9019975 - Flags: approval-mozilla-release?
status-firefox64: fixed → verified
status-firefox65: fixed → verified
(Assignee)

Updated

4 months ago
Status: RESOLVED → VERIFIED
Comment on attachment 9019975 [details]
Bug 1501952 - Make mScaleCombobox point to the right element. r?bgrins

Trivial fix for a regression, already uplifted to beta, approved for 63.0.1. Thanks
Attachment #9019975 - Flags: approval-mozilla-release? → approval-mozilla-release+

Comment 14

4 months ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-release/rev/f44ea719baf6
status-firefox63: fix-optional → fixed
I have managed to reproduce this issue using Firefox 65.0a1 (BuildId:20181024221315) on Windows 10 64bit.

This issue is verified fixed using Firefox 63.0.1 (BuildId:20181030165643) on Windows 10 64bit and Ubuntu 16.04 64bit.
status-firefox63: fixed → verified
Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.