react-native/React
Nick Lockwood 688bb17777 Fixed sticky header touch handling bug
Summary:In order to ensure that the docked sticky header in a ListView receives touches correctly, RCTScrollView has a custom hitTest implementation that checks the sticky headers for touches prior to checking any other views.

There was a bug in this implementation that meant that sticky views would get touch priority even if the touch was outside the bounds of the scrollView. This meant that sticky headers that scrolled off the top of the list would intercept touches intended for views placed above the scrollView.

This diff fixes that bug by checking that the touch is inside the scrollview before checking for sticky header touches. I've also limited the custom hit test logic to just the currently docked header, as the other sticky header views do not require special treatment.

Reviewed By: javache

Differential Revision: D3041236

fb-gh-sync-id: a2a3474dda03d5b51688bd575195a67956184bbe
shipit-source-id: a2a3474dda03d5b51688bd575195a67956184bbe
2016-03-14 04:08:21 -07:00
..
Base Declare RCTPerformanceLogger functions RCT_EXTERN 2016-03-11 03:18:21 -08:00
Executors Sourcemaps support for RAM 2016-03-13 11:14:32 -07:00
Layout updated css-layout and fixed callsites 2015-12-21 10:18:41 -08:00
Modules Updating AppState to support the inactive state. 2016-03-10 08:30:24 -08:00
Profiler Add systrace controls window 2016-03-11 06:21:26 -08:00
React.xcodeproj Fix OSS iOS build 2016-02-17 07:38:33 -08:00
Views Fixed sticky header touch handling bug 2016-03-14 04:08:21 -07:00