Add image resolution to done function about cropper

[fengyuanchen]

It's not good to add image info here. If you realy need the image info, maybe i can add a new method named "getImgInfo" to do this.

[jsguy]

It seems ok to have it as part of the data passed into the "done" function as it is static and related to the other info that comes through.

If you really wanted to, you could add a "getImgInfo" method to the cropper, but then you'd need to access a reference to the cropper instance inside the "done" function, which doesn't quite seem right.
Unless perhaps if you add it as part of the data? ie:

data = {
    x1: dragger.left,
    y1: dragger.top,
    x2: dragger.left + dragger.width,
    y2: dragger.top + dragger.height,
    height: dragger.height,
    width: dragger.width,
    image: this.getImgInfo()
};

That way you could still get to it via data.image, yet it is not exposing the image object directly, if that was your concern?

What do you reckon?

[jsguy]

I see you've added the "getImgInfo" function, however it is private and not called anywhere - are you able to update the getData function to include it, ie:

    getData: function() {
        var dragger = this.dragger,
            data = {};

        if (this.active) {
            data = this.transformData({
                x1: dragger.left,
                y1: dragger.top,
                width: dragger.width,
                height: dragger.height,
                x2: dragger.left + dragger.width,
                y2: dragger.top + dragger.height
            }, "get");
        }

        data.image = this.getImgInfo();

        return data;
    },

Or I can re-fork and add that line if you like? probably easier to just have you add it :)

[fengyuanchen]

Sorry, I just saw this comment..
The result data is dynamic, but the image information is static and not so useful. As this reason, I can't put them together.