Skip to content
This repository has been archived by the owner on Aug 31, 2021. It is now read-only.

[[ Bug 21672 ]] Disable auto content inset adjustment on iOS 11 scroller #6776

Merged
merged 1 commit into from Nov 6, 2018

Conversation

livecodeali
Copy link
Member

No description provided.

@@ -752,7 +752,12 @@ bool MCScrollViewGetContentOffset(UIScrollView *p_view, int32_t &r_x, int32_t &r
t_view = [[UIScrollView alloc] initWithFrame: CGRectMake(0, 0, 0, 0)];
if (t_view == nil)
return nil;


if (@available(iOS 11.0, *)) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@livecodeali I think you also need to wrap this in a pre-processor macro because UIScrollViewContentInsetAdjustmentNever won't be available for some of the SDKs we still build against:

#if __IPHONE_OS_VERSION_MAX_ALLOWED > 110000

@montegoulding
Copy link
Member

@livecode-vulcan review ok bb81c3d

@livecode-vulcan
Copy link
Contributor

💙 review by @montegoulding ok bb81c3d

livecode-vulcan added a commit that referenced this pull request Nov 6, 2018
[[ Bug 21672 ]] Disable auto content inset adjustment on iOS 11 scroller
@livecode-vulcan
Copy link
Contributor

😎 test success bb81c3d

  • try-community-armv6-android-sdk26_ndk16r15: success
  • try-community-armv7-android-sdk26_ndk16r15: success
  • try-community-arm64-android-sdk26_ndk16r15: success
  • try-community-x86-android-sdk26_ndk16r15: success
  • try-community-x86_64-android-sdk26_ndk16r15: success
  • try-community-js-emscripten-sdk1.35: success
  • try-community-universal-ios-iphoneos12.0: success
  • try-community-universal-ios-iphonesimulator12.0: success
  • try-community-universal-mac-macosx10.9: success
  • try-community-x86-linux-debian8: success
  • try-community-x86_64-linux-debian8: success
  • try-community-x86-win32: success
  • try-community-x86_64-win32: success

@livecodepanos livecodepanos merged commit 74e6931 into livecode:develop-9.0 Nov 6, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
4 participants