Closed Bug 1274834 Opened 3 years ago Closed 3 years ago

Printing in full screen mode gives a screen with no access to the print button

Categories

(Firefox :: Untriaged, defect)

43 Branch
defect
Not set

Tracking

()

VERIFIED FIXED
Firefox 49
Tracking Status
firefox46 --- wontfix
firefox47 --- wontfix
firefox48 + wontfix
firefox49 + verified
firefox-esr38 --- unaffected
firefox-esr45 --- wontfix

People

(Reporter: rwza01, Assigned: xidorn)

References

(Blocks 1 open bug)

Details

(Keywords: regression)

Attachments

(1 file)

User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:43.0) Gecko/20100101 Firefox/43.0
Build ID: 20151223140742

Steps to reproduce:

(Using FirefoxPortable_43.0.3_English downloaded from portableapps.com on Jan 5 2016 at 3:12 pm SAST)

1) View a web page in full screen mode
2) Print the page (move cursor to top of screen so that tabs and menus appear, click on three horizontal bars, click on print)
3) Try to press the print button that (should, but does not) appear


Actual results:

4) Moving the cursor to the top of the screen doesn't make the buttons appear and hence it is not possible to press the print button and so you have to hit <escape> to leave full screen mode - then the print button (and the other buttons) appears and you can print


Expected results:

4) Either moving the cursor to the top of the screen should make the print button appear, or printing in full screen mode should not give a full screen print preview but rather should automatically make the print button visible
Regression window:
https://hg.mozilla.org/integration/fx-team/pushloghtml?fromchange=391241169bec&tochange=1f7db5d87e80

Regressed by 	1f7db5d87e80	Dão Gottwald — Bug 1176233 - Simplify fullscreen code. r=xidorn
Blocks: 1176233
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(dao+bmo)
Keywords: regression
Blocks: 1208939
Assignee: nobody → bugzilla
Flags: needinfo?(dao+bmo)
Attachment #8755268 - Flags: review?(dao+bmo) → review+
Keywords: checkin-needed
Too late for 46/47, we are just a couple of weeks away from 47 release.
But this could still make it into 48 if the fix is verified and you request uplift.
https://hg.mozilla.org/mozilla-central/rev/aee22493e3c2
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Xidorn, are you going to request for an uplift in beta? Thanks
Flags: needinfo?(bugzilla)
Comment on attachment 8755268 [details]
MozReview Request: Bug 1274834 - Make print preview toolbar a fullscreentoolbar. r?dao

Approval Request Comment
[Feature/regressing bug #]: bug 1176233
[User impact if declined]: cannot access print preview toolbar in fullscreen mode
[Describe test coverage new/current, TreeHerder]: no test coverage
[Risks and why]: low risk, should not affect anything other than this specific case
[String/UUID change made/needed]: n/a


I don't think it's worth uplifting... But it isn't risky either. So if you think it's worth, then it's worth...
Flags: needinfo?(bugzilla)
Attachment #8755268 - Flags: approval-mozilla-beta?
Comment on attachment 8755268 [details]
MozReview Request: Bug 1274834 - Make print preview toolbar a fullscreentoolbar. r?dao

I trust your judgment! Let it ride the train from 49 then! Thanks for your time
Attachment #8755268 - Flags: approval-mozilla-beta? → approval-mozilla-beta-
I've reproduced the initial issue on Nightly 49 (2016-05-22). 

Verified fixed on Firefox 49 Beta 1 (buildID: 20160802125918) on Windows 7 x64.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.