Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

[Bug] Support window.print() #13214

Closed
yoasif opened this issue Aug 2, 2020 · 8 comments
Closed

[Bug] Support window.print() #13214

yoasif opened this issue Aug 2, 2020 · 8 comments
Labels
feature request 🌟 New functionality and improvements needs:gv GeckoView bug required to fix the issue. See bugzilla.mozilla.org

Comments

@yoasif
Copy link
Contributor

yoasif commented Aug 2, 2020

Steps to reproduce

  1. Navigate to https://www.w3schools.com/jsref/tryit.asp?filename=tryjsref_print
  2. Tap "Print this page"

Expected behavior

Print dialog appears.

Actual behavior

Nothing happens.

See also #10217

Also in issue in Reference Browser.

Device information

  • Android device: Pixel 2
  • Fenix version: latest nightly

┆Issue is synchronized with this Jira Task

@yoasif yoasif added the 🐞 bug Crashes, Something isn't working, .. label Aug 2, 2020
@github-actions github-actions bot added the needs:triage Issue needs triage label Aug 2, 2020
@makotokato
Copy link

@liuche liuche added feature request 🌟 New functionality and improvements and removed needs:triage Issue needs triage 🐞 bug Crashes, Something isn't working, .. labels Aug 8, 2020
@data-sync-user data-sync-user changed the title [Bug] Support window.print() FNX3-22964 ⁃ [Bug] Support window.print() Aug 11, 2020
@data-sync-user data-sync-user changed the title FNX3-22964 ⁃ [Bug] Support window.print() FNX-14394 ⁃ [Bug] Support window.print() Aug 11, 2020
@data-sync-user data-sync-user changed the title FNX-14394 ⁃ [Bug] Support window.print() FNX2-18854 ⁃ [Bug] Support window.print() Aug 11, 2020
@kbrosnan kbrosnan changed the title FNX2-18854 ⁃ [Bug] Support window.print() [Bug] Support window.print() Aug 29, 2020
@neontorrent
Copy link

Any update on this?

@karlcow
Copy link

karlcow commented Jan 5, 2022

Another duplicate for this.
webcompat/web-bugs#77486

@headbank
Copy link

headbank commented Jan 5, 2022

An observation made in #10217 that is probably even more relevant here:

While we await the actual print functionality's arrival, the current state of this hurts web developers. If I understand correctly, window.print is totally MIA, so a method in which it is called will just quietly puke, which may clobber subsequent js code. (Whether window.print() should really only be called in isolation, or always wrapped in a try/catch, is arguable but besides the point.)

Even when the user-facing upshot is limited to "Clicking the Print This Page button and nothing happens", it's still likely to be assumed by many users that the site is broken and the web dev is to blame.

This is driven by the twin likelihoods (IMHO) that (a) most users and (b) a great many web developers will tend to assume that any smartphone browser does support printing. Fenix really is an outlier in this respect, as many people in the other ticket have pointed out.

I would very much like Fenix to give UI feedback to the user that printing is currently unsupported.

@karlcow
Copy link

karlcow commented Jan 31, 2022

@headbank
Copy link

headbank commented Mar 9, 2022

There is also https://bugzilla.mozilla.org/show_bug.cgi?id=1659818

Which in turn appears to await https://bugzilla.mozilla.org/show_bug.cgi?id=1659819
(If I have understood what's been said there)

@jonalmeida jonalmeida added the needs:gv GeckoView bug required to fix the issue. See bugzilla.mozilla.org label Mar 10, 2022
@headbank
Copy link

Stuff is happening: https://bugzilla.mozilla.org/showdependencygraph.cgi?id=1659818

The top two ellipses went from green to grey (resolved) this morning :)

@cpeterso
Copy link

Closing this issue because we're tracking window.print in https://bugzilla.mozilla.org/show_bug.cgi?id=1659818

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature request 🌟 New functionality and improvements needs:gv GeckoView bug required to fix the issue. See bugzilla.mozilla.org
Projects
None yet
Development

No branches or pull requests

8 participants