Merge pull request #1902 from lucky-chen/ios-feature-interaction_fixd…

…element * [iOS]ignore fixed element for interactionTime
apache · Dec 12, 2018 · 0a823bd · 0a823bd
2 parents f745b79 + db5340e
commit 0a823bd
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/ios/sdk/WeexSDK/Sources/Model/WXComponent.mm b/ios/sdk/WeexSDK/Sources/Model/WXComponent.mm
@@ -595,6 +595,7 @@ - (BOOL)_insertSubcomponent:(WXComponent *)subcomponent atIndex:(NSInteger)index
     pthread_mutex_unlock(&_propertyMutex);
 
     if (subcomponent->_positionType == WXPositionTypeFixed) {
+        subcomponent.ignoreInteraction = YES;
         [self.weexInstance.componentManager addFixedComponent:subcomponent];
     }
 

diff --git a/ios/sdk/WeexSDK/Sources/Model/WXSDKInstance_performance.m b/ios/sdk/WeexSDK/Sources/Model/WXSDKInstance_performance.m
@@ -45,7 +45,7 @@ - (void) recordComponentCreatePerformance:(double) diffTime forComponent:(WXComp
 /** on UI thread **/
 - (void)onViewLoad:(WXComponent *)targetComponent
 {
-    if (targetComponent.hasAdd) {
+    if (targetComponent.hasAdd || targetComponent.ignoreInteraction) {
         return;
     }
     targetComponent.hasAdd = true;
@@ -71,9 +71,6 @@ - (void) _handleRenderTime:(WXComponent*)targetComponent withModifyTime:(double)
     if (nil == targetComponent) {
         return;
     }
-    if (targetComponent.ignoreInteraction) {
-        return;
-    }
     double diff = modifyTime - self.renderTimeOrigin;
     if (diff > 8000) {
         return;