alabid / flylatex

FlyLatex: A Realtime Collaborative Environment (with a concurrent editor) in node-js.
766 stars 85 forks source link

Uses pdf.js if available in the browser #6

Closed victornoel closed 11 years ago

victornoel commented 11 years ago

Hi,

Firefox now ships with pdf.js, could'nt it be used?

alabid commented 11 years ago

This software is supposed to be compatible with most browsers. So I/we can't assume that pdf.js is in the browser.

victornoel commented 11 years ago

Hi,

This is maybe testable at the browser level and have the generated image as a fallback, or maybe pdf.js can be provided by the flylatex, as it is javascript… actually I don't know how that work but I guess that if you want to integrate pdf.js in your web app, you need to ship it with it on the server side. The only problem will be check if the browser supports all the required features for pdf.js.

alabid commented 11 years ago

I honestly don't think I'll worry about all these. This is only a slight enhancement. It'd be nice if you could implement this and submit a pull request. Thanks.