Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Check availability of canvas & PDF before printing #6088

Merged
merged 1 commit into from
Jun 6, 2015

Conversation

Rob--W
Copy link
Member

@Rob--W Rob--W commented Jun 5, 2015

Fixes #6083

@timvandermeij
Copy link
Contributor

@Snuffleupagus #6086 appears to fix the same issue, but this patch also addresses the polyfill. Can we go with this patch instead?

@Snuffleupagus
Copy link
Collaborator

Can we go with this patch instead?

Sure. (I can address the indentation issue some other time.)

/botio-linux preview

@pdfjsbot
Copy link

pdfjsbot commented Jun 6, 2015

From: Bot.io (Linux)


Received

Command cmd_preview from @Snuffleupagus received. Current queue size: 0

Live output at: http://107.21.233.14:8877/e900b63d906891f/output.txt

Snuffleupagus added a commit that referenced this pull request Jun 6, 2015
Check availability of canvas & PDF before printing
@Snuffleupagus Snuffleupagus merged commit f943b29 into mozilla:master Jun 6, 2015
@Snuffleupagus
Copy link
Collaborator

Thanks for the patch.

@plh97
Copy link

plh97 commented Sep 24, 2019

image
this issue hasn't fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Print document before it loaded throws exception and completely disabled the print functionality
5 participants