Skip to content
This repository has been archived by the owner on Jan 7, 2023. It is now read-only.

Rotation issue Fix #221

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

Rotation issue Fix #221

wants to merge 12 commits into from

Conversation

Lawgrin
Copy link

@Lawgrin Lawgrin commented Apr 15, 2016

Fixes issue where the savedUri rotation was incorrect
Along with builder and sdk updates

Fixes issue where the savedUri rotation was incorrect
@Lawgrin
Copy link
Author

Lawgrin commented Apr 18, 2016

A problem occurred configuring project ':example'.

failed to find Build Tools revision 23.0.3

Please update to use the latest build tools

Grant added 2 commits April 20, 2016 13:35
Changes buildToolsVersion to 23.0.2
Changes build tools version again, to 23.0.1
Can’t believe that it hasn’t been updated
Grant added 2 commits April 28, 2016 21:13
Added the withMinSize function to allow for min size will also throw
exception/error if the selected image is less then the given amount
@jdamcd jdamcd added the bug label May 27, 2016
Grant added 4 commits May 27, 2016 21:30
# Conflicts:
#	build.gradle
#	lib/src/main/java/com/soundcloud/android/crop/Crop.java
# Conflicts:
#	build.gradle
#	lib/src/main/java/com/soundcloud/android/crop/Crop.java
5 lint errors removed
@Lawgrin
Copy link
Author

Lawgrin commented May 27, 2016

Lint and travis are conflicting, please sort this out

Grant added 2 commits May 29, 2016 19:58
Replaces almost all icons.
Added "disable 'Registered', 'IconDensities'" for lint ignore
Updated travis CI
@Lawgrin
Copy link
Author

Lawgrin commented May 29, 2016

@jdamcd Please review as there is nothing more for me to fix. Update your travis CI to the latest build tools

@thameemulansari
Copy link

@jdamcd , @Lawgrin Any update on this pull request with this rotation fix.
My Problem is,
When choosing the picture through gallery(Pictures taken through device camera) and after cropping it the cropped image is not retaining the actual orientation. Hope this pull request will fix my problem.

Please revert if you have any comments.

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

Successfully merging this pull request may close these issues.

3 participants