Support cropping based on original image size

[rharter]

Support cropping based on original image size

[eveliotc]

Although having output to match view dimensions was intended (literally wysiwyg), agreed that is not ideal and this would be a great enhancement for third party developers, handling original source is a bit of a challenge and we should approach it using subsampling.

[soxunyi]

@nshmura Do best

[konecnyna]

I would love to have this feature too. The way the current cropping works
is using the viewport dimensions instead of the original bitmap dimensions.

    @Nullable
    public Bitmap crop() {
        if (bitmap == null) {
            return null;
        }


        if (true) {
            return cropFullResolution();
        }

        final Bitmap src = bitmap;
        final Bitmap.Config srcConfig = src.getConfig();
        final Bitmap.Config config = srcConfig == null ? Bitmap.Config.ARGB_8888 : srcConfig;
        final int viewportHeight = touchManager.getViewportHeight();
        final int viewportWidth = touchManager.getViewportWidth();

        final Bitmap dst = Bitmap.createBitmap(viewportWidth, viewportHeight, config);

        Canvas canvas = new Canvas(dst);
        final int left = (getWidth() - viewportWidth) / 2;
        final int top = (getHeight() - viewportHeight) / 2;
        canvas.translate(-left, -top);

        drawBitmap(canvas);

        return dst;
    }

It's using the viewportWidth to create the cropped bitmap which I would think change per the screen density of the phone. Two identical crops on two different phones would have different resolutions.

We have access to the original bitmap in this scope but i'm having trouble trying to translate cropping logic to respect the original bitmap.

[dschaller]

Thank you for you contribution to this repository.

Closing this contribution as this repository is being archived.