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

Improve UX when cropping image size is less than needed #105

Open
arogachev opened this issue Sep 19, 2016 · 2 comments
Open

Improve UX when cropping image size is less than needed #105

arogachev opened this issue Sep 19, 2016 · 2 comments

Comments

@arogachev
Copy link

arogachev commented Sep 19, 2016

Instead of applying styles (red border around the crop frame as in current implementation), we can just use minSizeoption of Jcrop (which is now hardcoded to [5, 5]). That way user can not initialize crop frame or resize it to size smaller than minSize. With warning approach user needs to repeat resizing multiple times to figure out what the correct size is because warning state is changed only after releasing mouse.

Related: #106.

@anrie
Copy link
Member

anrie commented Sep 25, 2016

As mentioned in #106 it would be helpful if you would demonstrate your approach in a PR.

@MRigal
Copy link
Contributor

MRigal commented May 7, 2020

ping @arogachev

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

No branches or pull requests

3 participants