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

add Windows 10 support #174

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

add Windows 10 support #174

wants to merge 2 commits into from

Conversation

3h3c
Copy link

@3h3c 3h3c commented Sep 5, 2016

No description provided.

@alok932
Copy link

alok932 commented Sep 6, 2016

Is it working for windows? will it work on windows 8.1 as well? I am getting missing command error.

@3h3c
Copy link
Author

3h3c commented Sep 6, 2016

@alok932
i only test it for win10. pls post the complete error message with stacktrace.thx

@alok932
Copy link

alok932 commented Sep 6, 2016

Yes it worked on windows (not on phone) but it only opens file explorer and when an image is selected then my app is getting closed. For phone nothing is happening on click (i need to try on phone with windows 10, currently i tried on 8.1).
I will let you know once i succeed to get any stacktrace, Thanks Mate,

when height and width not specified it will be result in error. now its
fixed
@alok932
Copy link

alok932 commented Sep 7, 2016

thnx, i was just going to tell you about that.

@alok932
Copy link

alok932 commented Sep 8, 2016

Ok, so this hasn't solved problem yet!
Error says:-
0x88982f8c - JavaScript runtime error: The buffer allocated is insufficient.

@alok932
Copy link

alok932 commented Sep 8, 2016

changed line 45 to "return decoder.getPixelDataAsync();" and buffer issue is solved 😃

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