[ListView] Fix RCTScrollView stickyHeader touch passing.

Tj · Tj · commit d8f43105ed99 · 2015-08-04T17:10:19.000-08:00
Summary: Point must be converted to the stickyHeader's coordinate space and then passed to the stickyHeaders hitTest:withEvent: method in order to be correctly routed to any subviews of the stickyHeader. This resolves this [issue](facebook/react-native#2075). Closes facebook/react-native#2224 Github Author: Tj <tfallon@Tjs-MBP.local>
diff --git a/React/Views/RCTScrollView.m b/React/Views/RCTScrollView.m
@@ -339,20 +339,16 @@ - (void)dockClosestSectionHeader
 
 - (UIView *)hitTest:(CGPoint)point withEvent:(UIEvent *)event
 {
-  __block UIView *stickyHeader;
+  __block UIView *hitView;
 
   [_stickyHeaderIndices enumerateIndexesWithOptions:0 usingBlock:^(NSUInteger idx, BOOL *stop) {
-    stickyHeader = [self contentView].reactSubviews[idx];
+    UIView *stickyHeader = [self contentView].reactSubviews[idx];
     CGPoint convertedPoint = [stickyHeader convertPoint:point fromView:self];
-
-    if ([stickyHeader hitTest:convertedPoint withEvent:event]) {
-      *stop = YES;
-    } else {
-      stickyHeader = nil;
-    }
+    hitView = [stickyHeader hitTest:convertedPoint withEvent:event];
+    *stop = (hitView != nil);
   }];
 
-  return stickyHeader ?: [super hitTest:point withEvent:event];
+  return hitView ?: [super hitTest:point withEvent:event];
 }
 
 @end
