Support iPhone X

[TimOliver]

Jumping in here before anyone else does.

Due to the extremely custom layout of this view controller, it is hilariously broken on iPhone X.

For the most part, it should just be a matter of making the tool bar and the crop box insets obey the safe area of iOS 11. It shouldn't be TOO bad to fix.

[mugunth87]

@TimOliver When will this issue be fixed?

[TimOliver]

It's virtually finished in master already. The only thing remaining is I need to set up the toolbar background to extend under the notch, but even that isn't too much of a problem since it isn't noticable.

I plan to close this issue off after I've finished that part. But I've got some other pressing stuff, so I won't commit to any timeframe except 'before the iPhone X is released'. :)

[inPhilly]

The iPhone X is due out November 3. I am working on another issue with this repository, but I was also hoping to support iPhone X in my app that will be released in the next couple of weeks. Do you happen to have any updates on this? Thanks.

[TimOliver]

Nope. No updates at the moment sorry. I'm at multiple conferences this week without my dev setup. XD

[TimOliver]

Okay. I'm heading home from my conference week now. I had a quick chance to fill in the rest of the iPhone X logic I was missing.

It's all checked into master now. Try pulling it down and see how it looks to you! If all is good, I'll cut a new CocoaPods release after I get back home.

[Theome]

Just tested the master branch. The notch or the system bar overlaps the UI in some situations. This issue affects all devices (iPhone X, iPad, iPhone 8), and also devices running iOS 10.

[yojique]

sorry for bothering. any good news on the issue?

[inPhilly]

Sorry to bother again Tim. Any updates with this issue? Thanks for this great controller, again.

[TimOliver]

Sorry for the delay folks! I was Mac-less for all of last week, but I'm back in business now!

Oh jeez, okay. I see the issue. The top inset isn't account for the notch properly still on iPhone X, and not even handling the status bar properly on other iOS devices.

I think I know how to fix it. I'll see if I can smooth it all out today. :)

[TimOliver]

Okay! I THINK I've smoothed out the majority of issues. The implementation is a bit of a mess so I'll go through it tomorrow and properly refine it to make sure it works properly (It's a matter of testing landscape/portrait on iPad, iPhone and iPhone X. Each have different behaviours with the status bar.)

In the meantime, please grab the latest version from master and see how that looks to you. :)

[Theome]

Hi Tim, thanks for the update! With the new version from master (commit 4835a0), it looks correct on portrait under iOS 11. Configurations that use landscape or iOS 10 still look wrong.

[TimOliver]

Alright. It's done. I ended up needing to spend a whole day on it, but it's finally done. :D

There were a lot of little edge cases that broke by sheer virtue of the fact that the iPhone X notch introduces a pretty substantial offset. A lot of images broke when performing rotation animations on them and the likes.

I tested it very thoroughly on my iPhone X, my iPad and iOS 10.3 in the Simulator, and I'm happy with it.

It's been pushed to CocoaPods under version 2.2. Enjoy!