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

Suggestion : Add image rotate left/right #699

Closed
stonecourier opened this issue Sep 8, 2015 · 4 comments
Closed

Suggestion : Add image rotate left/right #699

stonecourier opened this issue Sep 8, 2015 · 4 comments

Comments

@stonecourier
Copy link

Please add so functionality in the core code so users can rotate the images taken with the camera module.

I have posted code how to do it here, but it would be great if this was built in for others.

http://stonecourier.blogspot.com.au/2015/09/adding-image-rotate-with-nativescript.html

@enchev enchev assigned enchev and unassigned enchev Sep 9, 2015
@bradmartin
Copy link
Contributor

+1

@bradmartin
Copy link
Contributor

Link to the idea: https://nativescript.ideas.aha.io/ideas/NS-I-62

I'd like to see the camera module support some of the options present in the Cordova plugin. I don't know the native aspects of achieving this but it would definitely improve the camera module to correct the orientation, allow cropping before using the image and so forth. Those two options would probably be the most beneficial and needed by those coming from a hybrid background.

@N3ll N3ll added this to the 1.5 (Under Review) milestone Oct 12, 2015
@hshristov hshristov modified the milestones: 1.6 (Under Review), 1.5 (Under Review) Oct 16, 2015
@hshristov hshristov modified the milestones: 1.7.0 (Under Review), 1.6.0 Jan 6, 2016
@valentinstoychev valentinstoychev modified the milestone: 1.7.0 (Under Review) Feb 24, 2016
@hshristov hshristov added this to the 2.4.0 milestone Oct 10, 2016
@hshristov
Copy link
Contributor

@stonecourier @bradmartin We have done this in the @next of widgets. So Image component now handles the rotation. Do you think this is enough?

@enchev enchev added the ready for test TSC needs to test this and confirm against live production apps and automated test suites label Oct 19, 2016
@enchev enchev closed this as completed Oct 19, 2016
@SvetoslavTsenov SvetoslavTsenov self-assigned this Oct 27, 2016
@SvetoslavTsenov SvetoslavTsenov added done and removed ready for test TSC needs to test this and confirm against live production apps and automated test suites labels Nov 9, 2016
@SvetoslavTsenov SvetoslavTsenov removed their assignment Nov 9, 2016
@lock
Copy link

lock bot commented Aug 29, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked and limited conversation to collaborators Aug 29, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

7 participants