opened
CrackerakiUA
username_0
I have modified one of the forks, so it can work with type rectangle and keep specific aspect ratio. Feel free to try and if you see that something is wrong, please tell me.
https://github.com/username_0/ngImgCropExtended
Fork with cool rectangle with aspect ratio
issue
created
raysefo
username_1
Hi,

I guess there is some padding around the cursor in the middle. Can I narrow it? And also there is a message while cropping. Can I change the message?

Best Regards.
comment
created
CrackerakiUA
username_0
Hello @username_1, about the cursor, i am not sure which padding do you mean. About the text, yeah, i will add option  to modify that test. Good point, thanks
comment
created
raysefo
username_1
Here is a sample, Is there a way to make it narrower in height? And the cursor is in the middle of the crop area.

![sample](https://cloud.githubusercontent.com/assets/859957/11021607/bee84902-864f-11e5-9374-e90ddb2f477d.PNG)
comment
created
CrackerakiUA
username_0
you want to make smaller height and keep same width? If i understand you, you can set result size, and if you have problem with min size, you have to set it also.
comment
created
raysefo
username_1
And how to change the text after choosing the image?
comment
created
CrackerakiUA
username_0
Which text do you mean, the "Chargement"?
comment
created
raysefo
username_1
yep
comment
created
CrackerakiUA
username_0
At the moment you can't by options, i will add option on new release.
You can search in the code and change it if you like.
comment
created
raysefo
username_1
Hi,

I searched the Chargement on the code but i found only in readme files.
Where is it in the code?
comment
created
raysefo
username_1
Hi,

I changed the size but it can not get any smaller in height.

<img-crop image="myImage" result-image="myCroppedImage" change-on-fly="false"
                          area-type="rectangle"
                          area-min-size="80"
                          result-image-format="image/jpeg"
                          result-image-quality="1"
                          result-image-size="{w:300, h:100}"></img-crop>
comment
created
CrackerakiUA
username_0
I will have a look tomorrow.
comment
created
CrackerakiUA
username_0
@username_1 ok, "Chargement" it's not simple text, it's not me that implement that. When i sit to fix bugs and add features i will add this as option.
Have you tried `area-min-size{w:80,h:80}` ?
comment
created
raysefo
username_1
@username_0 , I just need to narrow the height not width. Does your code make it narrower in height?
comment
created
CrackerakiUA
username_0
@username_1 you need to make narrow height in min size case? It will be awesomne if you fork any of mine codepen and show your problem there.
comment
created
raysefo
username_1
Hello @username_0 , I am selecting the picture from IOS photo library. How can I make it work in the fork? The picture in my second reply of this issue explains what I want to do. I couldn't get only 1 row of numbers. Here is a sample picture. If I try to crop only one row of numbers (lets say 01, 02, 03, 09, 15, 30) , I can not fit only those numbers inside of the cropped area.

![4](https://cloud.githubusercontent.com/assets/859957/11089165/6ab6fdb0-8870-11e5-8b79-cea7427cce56.jpg)
comment
created
CrackerakiUA
username_0
Ok, i see your problem. My fork don't support yet fixed height of crop area. I will add this to feature implement. Meanwhile as not good solution you can use simple rectangle without to set any aspect ration, also as one more know issue is that the result image come with size of square and you need to fix this with css. Have a look at my screenshot.
![screenshot 2015-11-11 14 55 12](https://cloud.githubusercontent.com/assets/6815721/11091589/b2afdc9a-8884-11e5-810e-b51b1032ef89.png)
comment
created
raysefo
username_1
@username_0 may be is it due to on mobile?
comment
created
CrackerakiUA
username_0
@username_1  I have tested it on my android and it worked well, haven''t tested it on iphone.
comment
created
raysefo
username_1
@username_0 what did you change? I am not using aspect ratio as well. Here is my img crop:
```
<img-crop image="myImage" result-image="myCroppedImage" change-on-fly="false"
                          area-type="rectangle"
                          area-min-size="80"
                          result-image-format="image/jpeg"
                          result-image-quality="1"
                          result-image-size="{w:300, h:100}"></img-crop>
```
comment
created
CrackerakiUA
username_0
This is what i used: http://codepen.io/Crackeraki/pen/vNQzEb
comment
created
CrackerakiUA
username_0
If you test on my codepen and then test on your project, if you see that it's not the same you should check that you have latest version of my fork.
comment
created
raysefo
username_1
@username_0 ,I guess I need to get the latest version. Is this yours right?

https://github.com/username_0/ngImgCropFullExtended
comment
created
CrackerakiUA
username_0
@username_1 yeah, this is mine.
comment
created
CrackerakiUA
username_0
@username_1 i have added custom chargement, you can check how to add it here: http://codepen.io/Crackeraki/pen/avYNKP?editors=100
comment
created
raysefo
username_1
@username_0 thanks so much, is it enough to get the latest ng-img-crop.js?
comment
created
CrackerakiUA
username_0
@username_1 yes
comment
created
raysefo
username_1
Hi @username_0 , unfortunately adding only this ng-img-crop breaks my code :( Selecting image is not working. Can you please check it?
comment
created
CrackerakiUA
username_0
@username_1 I can't see your code, you can check codepen's, how they are implemented.
comment
created
raysefo
username_1
@username_0 here is my code

http://codepen.io/cenkkizildag/pen/GpPxWx

But as I told you before I am selecting the picture from Iphone photo library. It was working before updating the js file.
comment
created
raysefo
username_1
As you can see from the Iphone screenshot, there is no take picture button and select picture (resim seç) is not working @username_0 

![img_7997](https://cloud.githubusercontent.com/assets/859957/11146922/37d22008-8a1b-11e5-8713-4570af481d3d.PNG)
comment
created
CrackerakiUA
username_0
Ok, i will check your codepen from my phone as soon as i can.
comment
created
CrackerakiUA
username_0
@username_1 on the newest version i have pushed today you don't have to manage result image with css, it get aspect ration of crop area. I will try to test you codepen tomorrow.
comment
created
raysefo
username_1
@username_0 have u tried it?
comment
created
CrackerakiUA
username_0
@username_1 sorry, i had crazy days, i will try to today later or tomorrow. In meanwhile we have updated the cropper again, so if you want, you can try the newest.
comment
created
raysefo
username_1
@username_0 whats new in croping?
comment
created
evdoks
username_2
@username_0 Don't you want to open your repo for issues/pull requests, so that we could submit issues and pull requests for it. Or take over that repo :)
comment
created
fernoftheandes
username_3
@username_0 @username_2 +1
comment
created
CrackerakiUA
username_0
Sorry, didn't noticed i had that closed. It's open
comment
created
raysefo
username_1
@username_0 still same problem ( there is no take picture button and select picture (resim seç) is not working)
comment
created
raysefo
username_1
@username_0 any news?
comment
created
raysefo
username_1
@username_0 ,Something in your new code breaks mine :) Here is my app.js:
```
angular.module('starter', ['ionic', 'ngCordova', 'ngImgCrop', 'ngIOS9UIWebViewPatch', 'ngFileUpload'])

.controller("ExampleController", ['$scope', '$cordovaCamera', 'Upload', '$timeout', function ($scope, $cordovaCamera, Upload, $timeout) {...
```

Somehow my first button below does NOT appear!:
```
<ion-content ng-controller="ExampleController" padding="true">
        <button class="button button-full button-balanced icon-right ion-ios-camera" ng-click="takePhoto()">
            Resim Çek
        </button>

        <button class="button button-full button-balanced icon-right ion-images" ng-click="choosePhoto()">
            Resim Seç
        </button>
        <center>
            <div class="cropArea">
...
```
comment