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

switch to jpgjs-dist #40

Closed
wants to merge 1 commit into from
Closed

Conversation

hipstersmoothie
Copy link
Contributor

@hipstersmoothie hipstersmoothie commented Aug 16, 2018

switch to jpgjs that's maintained. increases JS by 40kb.

closes #39

@photopea

@rxluz
Copy link

rxluz commented Aug 21, 2018

Lads, this package is breaking when I try to upload my application to AWS, is possible replace this to jpeg-js?

@hipstersmoothie hipstersmoothie changed the title switch to jpeg-js switch to jpgjs-dist Aug 21, 2018
@hipstersmoothie
Copy link
Contributor Author

Not at the moment. jpeg-js/jpeg-js#43 feel free to give this issue some love and the may fix it

@hipstersmoothie
Copy link
Contributor Author

@photopea you’ll get a lot less notifications once you merge this. If literally is the same dependency. Just the maintained version by Mozilla

@photopea
Copy link
Owner

Hi, I inserted JpegDecoder directly into UTIF.js , so there are no JPEG-related dependencies now.

@photopea photopea closed this Aug 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

installation failed on Windows
3 participants