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

Status: VERIFIED FIXED

(Toolkit :: Printing, defect, P1)

Description

[[e-mail]]

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

[Alice0775 White]

[Tracking Requested - why for this release]:

Comment 2

[Alice0775 White]

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?

Comment 3

[Dão Gottwald [:dao]]

Attachment: Bug 1501952 - Make mScaleCombobox point to the right element. r?bgrins

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0e96d6002321
Make mScaleCombobox point to the right element. r=bgrins

Comment 5

[Cristina Coroiu [:ccoroiu]]

https://hg.mozilla.org/mozilla-central/rev/0e96d6002321

Comment 6

[Dão Gottwald [:dao]]

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:

Comment 7

[Ryan VanderMeulen [:RyanVM]]

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.

Comment 8

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/releases/mozilla-beta/rev/a265524c6b4e

Comment 9

[jscher2000]

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

Comment 10

[Dão Gottwald [:dao]]

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

Ryan?

Comment 11

[Ryan VanderMeulen [:RyanVM]]

Seems like a decent candidate, though Fx63 is Pascal's release. Go ahead and nominate the patch for release uplift :).

Comment 12

[Dão Gottwald [:dao]]

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:

Comment 13

[Pascal Chevrel:pascalc]

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

Comment 14

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-release/rev/f44ea719baf6

Comment 15

[Emil Ghitta, QA [:emilghitta]]

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.