Only the header and footer are Printed for emails in Outlook Webapp
Categories
(Core :: Printing: Output, defect, P2)
Tracking
()
People
(Reporter: luc.mauchamp, Assigned: emilio)
References
(Regression)
Details
(Keywords: regression)
Attachments
(4 files, 4 obsolete files)
(deleted),
text/html
|
Details | |
(deleted),
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
|
Details |
(deleted),
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
|
Details |
(deleted),
patch
|
RyanVM
:
approval-mozilla-release+
RyanVM
:
approval-mozilla-esr68+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:67.0) Gecko/20100101 Firefox/67.0
Steps to reproduce:
When you print an email in Outlook Web App
Actual results:
Only the header and footer are Printed
Comment 1•5 years ago
|
||
according to some user reports this is regressing in firefox 68.
Hi philipp,
I see you marked this ticket as duplicate of two closed tickets. If you are still experiencing this issue, please make sure to test if it's reproducible on the latest Nightly version. You can download it from here: https://nightly.mozilla.org/.
If you still have the issue please create a new profile, you have the steps here:https://support.mozilla.org/en-US/kb/profile-manager-create-and-remove-firefox-profiles?redirectlocale=en-US&redirectslug=Managing-profiles#w_starting-the-profile-manager
Also, please check if this problem is reproduce on other browsers, like: chrome, internet explorer, etc and let us know your results.
It would be useful to have more steps to reproduce this bug, if possible can you try to print the email to a pdf file and attach a screenshot of the actual and the expected results?
For example expected result: the complete email, actual result: the incomplete email you get when printing.
Regards,
Assignee | ||
Comment 7•5 years ago
|
||
I'll take a look, thanks for finding the regression range.
Assignee | ||
Comment 8•5 years ago
|
||
[Tracking Requested - why for this release]:
Assignee | ||
Comment 9•5 years ago
|
||
I couldn't repro this from the free outlook.com web app (which AIUI is the same).
Two questions...
- Does this happen when printing to file? Or only when printing to an actual printer? I don't have a printer handy so I was doing the first.
- Does this happen in all mails or only on some? If not on all of them, is there any chance you could send me one of the emails that reproduces it for you?
Comment 10•5 years ago
|
||
It happens when printing to a printer AND to file.
Btw: It doesn't happen when using Google Chrome
Comment 11•5 years ago
|
||
It happens in all mails and on all Clients.
Assignee | ||
Comment 12•5 years ago
|
||
Hmm, then there is something I'm missing. You're on Windows, I guess, right? Maybe outlook is doing something different here on Linux. I'll try later today to repro on Windows later today...
Any hint to reproduce? I'll update the screencast with what I'm seeing.
Assignee | ||
Comment 13•5 years ago
|
||
Is this the right thing to test? Do I need a more complex email? I just wrote some simple email from Outlook's UI and sent it to myself.
Sorry for the spanish UI, this is a really old email account I only use for testing these days :)
Comment 14•5 years ago
|
||
Yes, that's the right thing.
We only have Windows (10) clients.
Assignee | ||
Comment 15•5 years ago
|
||
Ok, so I just tried to reproduce this on Windows 10 (my Windows version is 1809), and I cannot reproduce there on Firefox 68 nor on Nightly.
I could reproduce the printing issue but only once, and only because the email preview hadn't finished loaded (I'm on a pretty bad internet connection today).
I assume you're using some kind of private domain / paid service for Outlook, would there be any chance you could provide some testing account where you reproduce the issue in order to discard the cause being differences on my free account vs. yours? Would there be any specific Windows or Outlook configuration that I'm missing? Anything that like that that could come to mind?
Assignee | ||
Comment 16•5 years ago
|
||
(I also tried printing fairly more complex emails, and no issue in any of those either)
Comment 17•5 years ago
|
||
I created an account on our Exchange server for you:
https://outlook.uni-polster.de
user: emilio
password: Alvarez19
Assignee | ||
Comment 18•5 years ago
|
||
Alright, I can indeed reproduce there, thank you!
Assignee | ||
Comment 19•5 years ago
|
||
I think I managed to reduce it to a particular stylesheet in the print dialog
Assignee | ||
Comment 20•5 years ago
|
||
Assignee | ||
Comment 21•5 years ago
|
||
I hate quirks mode so much. My patch cloned the stylesheets a bit earlier, and as a result a wrong quirks mode is used, which makes a selector incorrectly match, and ends up with a layout that clips all content.
This is a sad bug :(
FWIW, the most reduced version of the stylesheet that I came up with before realizing what was going on is this:
body.printMail {
height: auto;
position: relative!important
}
.conductorContent {
position: absolute;
width: 100%;
height: 100%
}
.printMail .conductorContent {
height: auto
}
._n_x4 {
overflow: hidden
}
._n_X4 {
position: absolute;
width: 100%;
height: 100%
}
._n_X4
is supposed to match, but ._n_x4
isn't. We match it and as such make it incorrectly (a) be zero-sized, and (b) clip its content. Which means that it hides the whole page. :((
Assignee | ||
Comment 22•5 years ago
|
||
So that the stylist and CSS loader react appropriately.
Bug 1535788 causes the stylist to be created earlier, so it stopped grabbing the
already-updated compat mode.
I think the CSS loader stuff could already cause some issues before bug 1535788,
for what is worth.
Assignee | ||
Comment 23•5 years ago
|
||
Thanks so much for helping me figure this out itservice@! Vielen Dank!
Assignee | ||
Comment 24•5 years ago
|
||
The reftest-paged tests don't trigger the document clone code-path (I realized
that after writing them), but I guess they don't hurt, the printpreview test
does fail without the previous patch.
Depends on D39053
Assignee | ||
Comment 25•5 years ago
|
||
Comment on attachment 9080079 [details]
Bug 1567105 - When cloning a document for printing, call SetCompatMode rather than silently copying over the compat mode. r=heycam,#style,jwatt
Beta/Release Uplift Approval Request
- User impact if declined: Outlook OWA print mode doesn't print anything.
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: Yes
- If yes, steps to reproduce: open any of the test-cases attached to the bug, and try to print / print preview.
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): One-liner that prevents some quirks-mode state from getting out of sync.
- String changes made/needed: none
ESR Uplift Approval Request
- If this is not a sec:{high,crit} bug, please state case for ESR consideration: Outlook OWA doesn't print.
- User impact if declined: See above.
- Fix Landed on Version:
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): One-liner that prevents some quirks-mode state from getting out of sync.
- String or UUID changes made by this patch: none
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Comment 26•5 years ago
|
||
Comment 27•5 years ago
|
||
Not sure there's going to be another 68 dot release at this point, but the patch is simple enough that I'll keep it on the radar for now in case there is.
Comment 28•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/c260ef59bde9
https://hg.mozilla.org/mozilla-central/rev/2e8116048302
Updated•5 years ago
|
Comment 29•5 years ago
|
||
I successfully reproduced the issue on Firefox Nightly 70.0a1 (2019-07-18) under Windows 10 (x64) using the "totally reduced testcase".
The issue is no longer reproducible on latest Nightly 70.0a1 (2019-07-24). Tests were performed under WIndows 10 (x64), Ubuntu 18.04 (x64) and macOS 10.14.
Updated•5 years ago
|
Comment 30•5 years ago
|
||
Comment on attachment 9080079 [details]
Bug 1567105 - When cloning a document for printing, call SetCompatMode rather than silently copying over the compat mode. r=heycam,#style,jwatt
Fixes a printing regression in the Outlook web app. Verified on Nightly and includes automated tests. Approved for 69.0b8 and 68.1esr.
Updated•5 years ago
|
Comment 31•5 years ago
|
||
bugherder uplift |
Comment 32•5 years ago
|
||
bugherder uplift |
Comment 33•5 years ago
|
||
Backed out from esr68 for bustage at dom/base/Document.cpp:
https://hg.mozilla.org/releases/mozilla-esr68/rev/c08e3b8c55b2bb7062d51cdda4a01b0c74e296b2
Push with bustage: https://treeherder.mozilla.org/#/jobs?repo=mozilla-esr68&resultStatus=usercancel%2Ctestfailed%2Cbusted%2Cexception&revision=22c7a4ad22904acda04db7b947fbf6374194565f
Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=258166314&repo=mozilla-esr68
/builds/worker/workspace/build/src/dom/base/Document.cpp:8465:10: error: no member named 'SetCompatibilityMode' in 'mozilla::dom::Document'
Assignee | ||
Comment 34•5 years ago
|
||
Mind uplifting this version instead? The function lives in nsHTMLDocument
in ESR.
Comment 35•5 years ago
|
||
The issue is no longer reproducible on Firefox 69.0b8 (20190724202328 from Treeherder). Tests were performed under Windows 10 (x64), Ubuntu 18.04 (x64) and macOS 10.14.
Updated•5 years ago
|
Comment 36•5 years ago
|
||
bugherder uplift |
Comment 37•5 years ago
|
||
The issue is verified on Firefox 68.1.0esr (20190725195252 from Treeherder). Tests were performed under Windows 10 (x64), Ubuntu 18.04 (x64) and macOS 10.14.
Comment 38•5 years ago
|
||
Backed out changeset ed2ee021980b (Bug 1567105) for test_printpreview.xul failures
Backout link: https://hg.mozilla.org/releases/mozilla-esr68/rev/59b5294b8c047097b4575d3c5de86a2f1530d3aa
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=258327750&repo=mozilla-esr68&lineNumber=1869
[task 2019-07-25T16:05:56.775Z] 16:05:56 INFO - TEST-START | layout/base/tests/chrome/test_printpreview.xul
[task 2019-07-25T16:05:56.941Z] 16:05:56 INFO - GECKO(1200) | JavaScript error: chrome://mochitests/content/chrome/layout/base/tests/chrome/printpreview_helper.xul, line 126: ReferenceError: counterTimeout is not defined
[task 2019-07-25T16:06:02.749Z] 16:06:02 INFO - GECKO(1200) | JavaScript error: chrome://mochitests/content/chrome/layout/base/tests/chrome/printpreview_helper.xul, line 399: ReferenceError: frameElts is not defined
[task 2019-07-25T16:11:19.468Z] 16:11:19 INFO - TEST-INFO | started process screentopng
[task 2019-07-25T16:11:19.747Z] 16:11:19 INFO - TEST-INFO | screentopng: exit 0
[task 2019-07-25T16:11:19.748Z] 16:11:19 INFO - <snipped 41 output lines - if you need more context, please use SimpleTest.requestCompleteLog() in your test>
[task 2019-07-25T16:11:19.749Z] 16:11:19 INFO - Buffered messages logged at 16:05:59
[task 2019-07-25T16:11:19.751Z] 16:11:19 INFO - TEST-PASS | layout/base/tests/chrome/test_printpreview.xul | Should have called afterprint listener!
....
[task 2019-07-25T16:11:19.849Z] 16:11:19 INFO - TEST-PASS | layout/base/tests/chrome/test_printpreview.xul | Should have called beforeprint listener!
[task 2019-07-25T16:11:19.850Z] 16:11:19 INFO - TEST-PASS | layout/base/tests/chrome/test_printpreview.xul | Should have called afterprint listener!
[task 2019-07-25T16:11:19.852Z] 16:11:19 INFO - Buffered messages finished
[task 2019-07-25T16:11:19.860Z] 16:11:19 INFO - TEST-UNEXPECTED-FAIL | layout/base/tests/chrome/test_printpreview.xul | Test timed out.
[task 2019-07-25T16:11:19.860Z] 16:11:19 INFO - SimpleTest.ok@chrome://mochikit/content/tests/SimpleTest/SimpleTest.js:275:18
[task 2019-07-25T16:11:19.862Z] 16:11:19 INFO - reportError@chrome://mochikit/content/tests/SimpleTest/TestRunner.js:121:22
[task 2019-07-25T16:11:19.862Z] 16:11:19 INFO - TestRunner._checkForHangs@chrome://mochikit/content/tests/SimpleTest/TestRunner.js:142:7
[task 2019-07-25T16:11:20.467Z] 16:11:20 INFO - GECKO(1200) | MEMORY STAT | vsize 3122MB | residentFast 310MB | heapAllocated 101MB
[task 2019-07-25T16:11:20.468Z] 16:11:20 INFO - TEST-OK | layout/base/tests/chrome/test_printpreview.xul | took 323698ms
Assignee | ||
Comment 39•5 years ago
|
||
The test is timing out in ESR, but I think we should leave the fix there.
Comment 40•5 years ago
|
||
Comment on attachment 9080080 [details]
Bug 1567105 - Tests. r=heycam,#style,jwatt
Sounds like we're leaving the test for 69+ only.
Comment 41•5 years ago
|
||
Updated•5 years ago
|
Updated•5 years ago
|
Comment 42•5 years ago
|
||
bugherder uplift |
Comment 43•5 years ago
|
||
bugherder uplift |
FIREFOX_ESR_68_0_X_RELBRANCH https://hg.mozilla.org/releases/mozilla-esr68/rev/52b6010173ef
Comment 44•5 years ago
|
||
Added to the 68.0.2 relnotes:
Printing emails from the Outlook web app no longer prints only the header and footer
Comment 46•5 years ago
|
||
The issue is no longer reproducible on Firefox 68.0.2 (20190802130755 from Treeherder) and 68.1.0esr (20190806222858 from Treeherder). Tests were performed under Windows 10 (x64), Ubuntu 18.04 (x64) and macOS 10.13.
Updated•5 years ago
|
Comment 48•5 years ago
|
||
Hello,
Seeing as this was not covered in our tests, I made a new testcase to verify this.
Updated•3 years ago
|
Description
•