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

[TIMOB-19597] Android: Update for previewRect when no overlay is used #7660

Closed
wants to merge 1 commit into from
Closed

Conversation

m1ga
Copy link
Contributor

@m1ga m1ga commented Feb 1, 2016

See https://jira.appcelerator.org/browse/TIMOB-19597
When no overlay image is used it will return -1 (width, height)

@hansemannn
Copy link
Collaborator

Hey @m1ga, there are a lot of blank spaces that produce a diff change. Can you check and eliminate them? Thanks!

@hansemannn hansemannn changed the title update for previewRect when no overlay is used [TIMOB-19597] Android: Update for previewRect when no overlay is used Feb 1, 2016
@sriks
Copy link
Contributor

sriks commented Feb 2, 2016

@m1ga this PR is taken from your master branch? When you are reworking a fix you should work/fork from the fix branch itself i.e your TIMOB-19597 branch. That way only the required diff is captured. Other wise this PR is capturing a lot of other commits as well.

@hansemannn
Copy link
Collaborator

Closing in favor of #7665 and #7666, thank you!

@hansemannn hansemannn closed this Feb 2, 2016
@m1ga
Copy link
Contributor Author

m1ga commented Feb 2, 2016

@hansemannn sorry about all the blanks!
@sriks yes, it was from master. I thought it would only push my changes (the few lines). Will be more careful next time.

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

Successfully merging this pull request may close these issues.

None yet

3 participants