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.

Sign up for GitHub

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

Jump to bottom

image reduction for beauty #618

Open
OmlineEditor opened this issue Nov 28, 2021 · 5 comments
Open

image reduction for beauty #618

OmlineEditor opened this issue Nov 28, 2021 · 5 comments
Labels
common: uncommon Most users are unlikely to come across this or unexpected workflow severity: tolerable Low/no impact on users state: help wanted I looked into this issue but couldn't solve it quickly type: enhancement New feature or request

Comments

@OmlineEditor
Copy link

now it is impossible to reduce the image so that it fits completely and has transparent edges.
now it is impossible to reduce the image, it looks bad, part is cropped.
you need to zoom out in order to have transparent areas

0
2
@TheLastProject TheLastProject added the type: enhancement New feature or request label Nov 28, 2021
@TheLastProject
Copy link
Member

It is a bug that it still crops it in the main view when you use the "card" size in the cropper and I will fix that.

I am not completely sure what your other comments mean though.

@TheLastProject
Copy link
Member

I fixed one cropping issue in 19f7af1, it will not crop any further on the main screen than the card size in the cropper.

The rest I don't understand well, so I am changing this issue to "needs info".

@TheLastProject TheLastProject added state: needs info and removed type: enhancement New feature or request labels Nov 28, 2021
@OmlineEditor
Copy link
Author

is it possible to crop so that there are transparent areas? that is, there was no image in the transparency area before.

@TheLastProject TheLastProject added type: enhancement New feature or request state: help wanted I looked into this issue but couldn't solve it quickly and removed state: needs info labels Nov 28, 2021
@TheLastProject
Copy link
Member

Ah... yeah that is a fair request. I don't know if uCrop can do that, if it can it is probably just a setting change.

@roshavagarga
Copy link

These might be related - Yalantis/uCrop#622 and Yalantis/uCrop#803

@TheLastProject TheLastProject added common: uncommon Most users are unlikely to come across this or unexpected workflow severity: tolerable Low/no impact on users labels Feb 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
common: uncommon Most users are unlikely to come across this or unexpected workflow severity: tolerable Low/no impact on users state: help wanted I looked into this issue but couldn't solve it quickly type: enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@TheLastProject @OmlineEditor @roshavagarga