Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add original width and height #90

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

Closed
wants to merge 4 commits into from
Closed

Add original width and height #90

wants to merge 4 commits into from

Conversation

doc-l
Copy link

@doc-l doc-l commented Apr 4, 2019

We need to have the original width and height of the image, and it seems to be impossible to get this information from the library without reloading the original image as an image. This is a big overhead, because you can also extract this information from within the lib.

I added two lines that append the width and height to the file object, is this the best place to put it?

@blueimp blueimp closed this in f27268d Apr 5, 2019
@blueimp
Copy link
Owner

blueimp commented Apr 5, 2019

Thanks for your contribution, @L-Hendriks.
Adding the properties to the file object would not be the ideal location.
I've pushed a new version that provides the original image width+height in the callback data.
Since the callback data wasn't well documented, I've also updated the README.

@doc-l
Copy link
Author

doc-l commented Apr 6, 2019

Hi Sebastian,

Thanks for that! I agree the location is not be ideal, but I didn't want to change too much on the library. Have a nice weekend!

@blueimp
Copy link
Owner

blueimp commented Apr 8, 2019

Thanks @L-Hendriks ☺️

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.

2 participants