Scroll view behavior fix

[jasonaibrahim]

Current behavior

The scroll view will jump to the middle when a keystroke is received by the keyboard no matter where the scroll view currently is.

Fix

Remove calls to setNeedsLayout to avoid the scroll view from jumping

[rnystrom]

@jasonaibrahim can you detail this bug a little more? I'm not sure I've seen it before. How do I repro?

[jasonaibrahim]

@rnystrom sorry for slow replies, been travelling. i will put together a little demo repo in the next few days

[jasonaibrahim]

Alright I have a demo repo up and able to reproduce.

Reproduce Issue

1. Clone https://github.com/jasonaibrahim/MessageViewControllerScrollViewBugExample
2. Pod install
3. Open MessageViewControllerScrollViewBugExample.xcworkspace
4. Run in any simulator or physical device
5. Scroll down to the bottom of the scroll view so that "99" is the last item visible
6. Tap into message view to open keyboard (make sure keyboard is visible on screen)
7. Type a letter
8. See the scroll view jump so now "91" is the last item visible

Fix

1. rm -rf Pods
2. open Podfile, change pod to pod 'MessageViewController', :git => 'https://github.com/jasonaibrahim/MessageViewController.git', :branch => 'scroll-view-behavior-fix'
3. pod install
4. Run steps 4-7 from above
5. Scroll view no longer jumps