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

"Save Image" option is not working in Firefox #20

Closed
micnic opened this issue Sep 25, 2017 · 9 comments
Closed

"Save Image" option is not working in Firefox #20

micnic opened this issue Sep 25, 2017 · 9 comments
Labels

Comments

@micnic
Copy link

micnic commented Sep 25, 2017

As the title says, when click on Save Image button in Firefox nothing happens.

@nozzlegear
Copy link

Also does not work in Edge (neither does the Tweet Image button).

@briandennis
Copy link
Collaborator

@micnic I pushed a fix, it should be working on Firefox now.

@nozzlegear Is the Save Image button working on Edge now? I can check myself later today, but don't have access to Edge at the moment...

@micnic
Copy link
Author

micnic commented Sep 25, 2017

@briandennis, thanks it works fine, you are awesome !

@briandennis
Copy link
Collaborator

@micnic glad I could help!

@briandennis
Copy link
Collaborator

Going to close this. Will reopen another issue if the problem on Edge persists

@nozzlegear
Copy link

@briandennis I've done some investigating with Edge and it looks like the dom-to-image package isn't tested in and doesn't support Edge (or Safari I guess): tsayen/dom-to-image#78. Specifically the .toPng function fails with Edge complaining that the canvas is dirty.

I switched out .toPng() with .toSvg() which got me further, the browser was able to create the data URL. However I was unable to actually navigate to that data URL, download it or even just copy it to my clipboard to navigate there manually. It would just crash Edge instead.

Not sure how much work you want to put into this issue, it's probably just easier to add a note that Carbon doesn't support Edge!

@mfix22
Copy link
Contributor

mfix22 commented Sep 25, 2017

@nozzlegear thanks so much for investigating. We plan to replace dom-to-image with a cross-browser solution. We really appreciate the work you did.

@nozzlegear
Copy link

No problem! Looking forward to a cross browser Carbon.

@mfix22
Copy link
Contributor

mfix22 commented Sep 25, 2017

@nozzlegear opened #28 so you can track it (:

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

No branches or pull requests

4 participants