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

Add chooseText and cancelText properties #618

Closed
wants to merge 6 commits into from

Conversation

smsmdida
Copy link

added ability to change control buttons texts in crop

@ivpusic
Copy link
Owner

ivpusic commented Feb 20, 2018

thanks for PR. Could you prefix properties with ‘cropper’ to have consistent naming with opet cropper related properties?

@smsmdida
Copy link
Author

@ivpusic done

@pistonsky
Copy link

Will this be merged? We are making an app that supports multiple languages and this feature is a must. Please! @ivpusic

@ivpusic
Copy link
Owner

ivpusic commented May 15, 2018

@pistonsky could you fix merge conflicts and I will then merge it?

@istro
Copy link
Contributor

istro commented May 15, 2018

@ivpusic & @pistonsky - I just rebased it on master & opened a separate PR - figured that would be quicker than reaching out to @smsmdida to ask him to rebase.

As a side note - thanks for great work!

@ivpusic
Copy link
Owner

ivpusic commented May 16, 2018

tnx all. merged here #699

@ivpusic ivpusic closed this May 16, 2018
@dinhbang
Copy link

added ability to change control buttons texts in crop

Thanks. Could you translate the "Rotate" and "Scale" on edit image with android device?

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.

5 participants