Skip to content

Improve/non native/2.2.8 prs integration#812

Merged
severianremi merged 12 commits intodevelop-non-nativefrom
improve/non_native/2.2.8-prs_integration
Jan 27, 2022
Merged

Improve/non native/2.2.8 prs integration#812
severianremi merged 12 commits intodevelop-non-nativefrom
improve/non_native/2.2.8-prs_integration

Conversation

@dmitriy1morozov
Copy link
Copy Markdown
Contributor

Please review this PR as an integrational part of 2.2.8 release. Once PR is accepted we will start testing on the develop-non-native branch. Once tested these changes will be released to master-non-native as 2.2.8 version.

dmytro1morozov and others added 12 commits January 21, 2022 17:43
Signed-off-by: dmorozov <dmytro.morozov@yalantis.net>
- change fallback for EXTRA_ASPECT_RATIO_X and EXTRA_ASPECT_RATIO_Y to "-1" as "0" has special meaning
- fix usage of useSourceImageAspectRatio (now properly hides mWrapperStateAspectRatio)
- fix setTargetAspectRatio at processOptions due to Float.NaN (allows custom aspect ratio definitions with default ratio set to x=CropImageView.DEFAULT_ASPECT_RATIO and y=CropImageView.DEFAULT_ASPECT_RATIO)
- fix sanity check at setAspectRatioOptions
@dmitriy1morozov dmitriy1morozov self-assigned this Jan 26, 2022
@severianremi severianremi merged commit 2f52086 into develop-non-native Jan 27, 2022
@dmitriy1morozov dmitriy1morozov deleted the improve/non_native/2.2.8-prs_integration branch January 27, 2022 13:04
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.

10 participants