TOCropViewController icon indicating copy to clipboard operation
TOCropViewController copied to clipboard

Some CropView bottom is hided if TOCropViewController is not full height

Open dobiho5 opened this issue 5 years ago • 1 comments

Describe the bug

Some CropView Bottom is hided when CropViewController is not full height.


// View has another toolbar below TOCropViewController
TOCropViewController *cropController;

self.cropController = =  [[TOCropViewController alloc] initWithCroppingStyle:self.croppingStyle image:self.image];

self.cropController.view.frame = CGRectMake(0, 0, self.view.bounds.size.width, self.toolBar.frame.origin.y);

[self addChildViewController:self.cropController];
    [self.view addSubview:self.cropController.view];
    [self.cropController didMoveToParentViewController:self];
    [self.view bringSubviewToFront:self.toolBar];

I can fixed it in TOCropViewController.m with +100

- (CGRect)frameForCropViewWithVerticalLayout:(BOOL)verticalLayout
{
if (!verticalLayout) {
        frame.origin.x = kTOCropViewControllerToolbarHeight + insets.left;
        frame.size.width = CGRectGetWidth(bounds) - frame.origin.x;
		frame.size.height = CGRectGetHeight(bounds);
    }
    else { // Vertical layout
        frame.size.height = CGRectGetHeight(bounds);
        frame.size.width = CGRectGetWidth(bounds);

        // Set Y and adjust for height
        if (self.toolbarPosition == TOCropViewControllerToolbarPositionBottom) {
           // +100
            frame.size.height -= (insets.bottom + kTOCropViewControllerToolbarHeight + 100);
        } else if (self.toolbarPosition == TOCropViewControllerToolbarPositionTop) {
			frame.origin.y = kTOCropViewControllerToolbarHeight + insets.top;
            frame.size.height -= frame.origin.y;
        }
    }
}

If TOCropViewController frame is changed, would you change bottom height?

dobiho5 avatar Aug 21 '19 07:08 dobiho5

Thanks for the report @dobiho5.

Um. The library has full support for arbitrary frame sizes. How exactly are you presenting it? Can you attach a screenshot?

Simply adding 100 points is a very naive way of fixing it and might create more bugs than it fixes. 😅

TimOliver avatar Sep 16 '19 03:09 TimOliver