Closed
Bug 1280159
Opened 5 years ago
Closed 5 years ago
Page Setup Margin Widths use Millimeters Instead of Inches (for paper Legal US e.g.)
Categories
(Core :: Printing: Setup, defect)
Tracking
()
VERIFIED
FIXED
mozilla50
People
(Reporter: castingman, Assigned: bobowen)
References
Details
(Keywords: regression)
Attachments
(1 file)
58 bytes,
text/x-review-board-request
|
jimm
:
review+
gchang
:
approval-mozilla-aurora+
gchang
:
approval-mozilla-beta+
|
Details |
User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:47.0) Gecko/20100101 Firefox/47.0 Build ID: 20160604131506 Steps to reproduce: Select File, Page Setup..., Margins & Header/Footer. Actual results: The margins are in millimeters. Prior to version 46, the margins were in inches. Expected results: The margins should be in inches as I'm still using Letter size (8.5" x 11") paper and not a metric size. Using about:config, I reset all prefs with "print, printer, margins, or paper" to default. The margins remain in millimeters (10mm) in the Page Setup... dialog box although in about:config they are shown in inches (0.393750011920929" which is a close approximation to 10mm). I have found no way to change back to inches. See https://support.mozilla.org/en-US/questions/1126411
Is it reproducible with a fresh profile? https://support.mozilla.org/en-US/kb/profile-manager-create-and-remove-firefox-profiles
Component: Untriaged → Printing: Setup
Flags: needinfo?(castingman)
Product: Firefox → Core
Nevermind, I'm able to reproduce it. Indeed, Print Setup displays margin sizes in mm instead of in when the paper size is Legal US e.g. (I tested on Win 7 with MS XPS Document Writer). Regression range: https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=955993 ba4a2a6ed9c85b9fc6c3a4c0fe582b178f&tochange=fb4737474a9c50b44e0ec38e43d42ed8a119 b7ee
Blocks: 1238964
Status: UNCONFIRMED → NEW
Has Regression Range: --- → yes
Has STR: --- → yes
status-firefox47:
--- → affected
status-firefox48:
--- → affected
status-firefox49:
--- → affected
status-firefox50:
--- → affected
tracking-firefox48:
--- → ?
tracking-firefox49:
--- → ?
tracking-firefox50:
--- → ?
Ever confirmed: true
Flags: needinfo?(castingman) → needinfo?(bobowen.code)
Keywords: regression
OS: Unspecified → All
Hardware: Unspecified → All
Summary: Page Setup Margin Widths use Millimeters Instead of Inches → Page Setup Margin Widths use Millimeters Instead of Inches (for paper Legal US e.g.)
Version: 47 Branch → 46 Branch
I note that the title of this Bug report now refers to US Legal Size paper. The problem I've encountered is with my printer set to US Letter Size paper. Not sure if this is significant but thought I should mention it. Oh, and I did try a new profile and had the same problem.
Given that this is not a new regression in 47, we will have to live with it for another ~3 weeks. Hopefully we address this in 48 release.
Assignee | ||
Comment 5•5 years ago
|
||
I know what I need to do to sort this out, I'll get to it soon.
Assignee: nobody → bobowen.code
Flags: needinfo?(bobowen.code)
Assignee | ||
Updated•5 years ago
|
Status: NEW → ASSIGNED
Regression from 46, tracking.
Assignee | ||
Comment 7•5 years ago
|
||
Also, changed the other paper size conversion code to be similar, as I think it is easier to follow. Review commit: https://reviewboard.mozilla.org/r/61518/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/61518/
Attachment #8766708 -
Flags: review?(jmathies)
![]() |
||
Updated•5 years ago
|
Attachment #8766708 -
Flags: review?(jmathies) → review+
![]() |
||
Comment 8•5 years ago
|
||
Comment on attachment 8766708 [details] Bug 1280159: Let paper type drive page size units on Windows. https://reviewboard.mozilla.org/r/61518/#review58440 lgtm
Pushed by bobowencode@gmail.com: https://hg.mozilla.org/integration/autoland/rev/a4999a2b771d Let paper type drive page size units on Windows. r=jimm
Comment 10•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/a4999a2b771d
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
Comment 11•5 years ago
|
||
Hi Bob, do you want to uplift this for 48/49 if this patch is not too risky.
Flags: needinfo?(bobowen.code)
Assignee | ||
Comment 12•5 years ago
|
||
(In reply to Gerry Chang [:gchang] from comment #11) > Hi Bob, do you want to uplift this for 48/49 if this patch is not too risky. Yes, I'll just wait for it to hit Nightly and give it another test, then I'll request uplift, thanks.
Assignee | ||
Comment 13•5 years ago
|
||
Comment on attachment 8766708 [details] Bug 1280159: Let paper type drive page size units on Windows. Approval Request Comment [Feature/regressing bug #]: Bug 1238964 changed to just use millimetres on Windows. (Firefox 46) [User impact if declined]: Margin sizes unit will continue not to relate to the paper type. [Describe test coverage new/current, TreeHerder]: Manual testing, it would be good to get QA verification in Beta. [Risks and why]: Low(ish) - fairly simple change that maps paper type to size units and uses that for calculations. [String/UUID change made/needed]: None
Flags: needinfo?(bobowen.code)
Attachment #8766708 -
Flags: approval-mozilla-beta?
Attachment #8766708 -
Flags: approval-mozilla-aurora?
Comment 14•5 years ago
|
||
Comment on attachment 8766708 [details] Bug 1280159: Let paper type drive page size units on Windows. This patch fixes a regression. Take it in 48 beta 6 and aurora.
Attachment #8766708 -
Flags: approval-mozilla-beta?
Attachment #8766708 -
Flags: approval-mozilla-beta+
Attachment #8766708 -
Flags: approval-mozilla-aurora?
Attachment #8766708 -
Flags: approval-mozilla-aurora+
Comment 16•5 years ago
|
||
bugherderuplift |
https://hg.mozilla.org/releases/mozilla-aurora/rev/65cb053a14bc
Comment 17•5 years ago
|
||
bugherderuplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/f574e40a75b5
Comment 18•5 years ago
|
||
According to Comment 13 (and also based on my investigations), this bug is only affecting Windows platform. I managed to reproduce this issue on 50.0a1 (2016-06-14). Also, I investigated it on - Windows 10 x64 - Windows 7 x64 The bug is verified fixed on - latest Nightly 50.0a1 (2016-07-10) - latest Aurora 49.0a2 (2016-07-11) - 48.0b6 build 1 (20160706215822) I will set the flags accordingly.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
Updated•5 years ago
|
OS: All → Windows
You need to log in
before you can comment on or make changes to this bug.
Description
•