Skip to content

Commit

Permalink
Remove "overlayRenderCount" from LazyCollection
Browse files Browse the repository at this point in the history
Summary:
This API has been broken for a while and brings confusion for our users multiple times, this diff is getting rid of it.

- https://fb.workplace.com/groups/litho.support/permalink/3792199454434728/
- https://fb.workplace.com/groups/litho.support/permalink/3618480355139973/

Reviewed By: pengj

Differential Revision: D58459773

fbshipit-source-id: eb43cc2ddd36b91700f582cc4238e2ee7e3f4581
  • Loading branch information
Andrew Wang authored and facebook-github-bot committed Jun 12, 2024
1 parent 5ad4b6e commit 6119376
Show file tree
Hide file tree
Showing 5 changed files with 1 addition and 79 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@ import com.facebook.litho.Handle
import com.facebook.litho.KComponent
import com.facebook.litho.LithoStartupLogger
import com.facebook.litho.Style
import com.facebook.litho.config.LithoDebugConfigurations
import com.facebook.litho.eventHandlerWithReturn
import com.facebook.litho.kotlinStyle
import com.facebook.litho.sections.ChangesInfo
Expand Down Expand Up @@ -87,7 +86,6 @@ class LazyCollection(
private val lazyCollectionController: LazyCollectionController? = null,
private val onDataRendered: OnDataRendered? = null,
private val childEquivalenceIncludesCommonProps: Boolean = true,
private val overlayRenderCount: Boolean = false,
private val alwaysDetectDuplicates: Boolean = false,
private val fadingEdgeLength: Dimen? = null,
private val shouldExcludeFromIncrementalMount: Boolean = false,
Expand Down Expand Up @@ -240,10 +238,7 @@ class LazyCollection(
parentHeightPercent(item.parentHeightPercent)
}
}
.component(
if (LithoDebugConfigurations.isDebugModeEnabled && overlayRenderCount)
component.overlayRenderCount
else component)
.component(component)
.build()
})
.onCheckIsSameItemEventHandler(eventHandlerWithReturn(::isSameID))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,6 @@ inline fun ResourcesScope.LazyGrid(
isReconciliationEnabled: Boolean =
context.lithoConfiguration.componentsConfig.isReconciliationEnabled,
childEquivalenceIncludesCommonProps: Boolean = true,
overlayRenderCount: Boolean = false,
alwaysDetectDuplicates: Boolean = false,
fadingEdgeLength: Dimen? = null,
preAllocationHandler: PreAllocationHandler? =
Expand Down Expand Up @@ -118,7 +117,6 @@ inline fun ResourcesScope.LazyGrid(
lazyCollectionController,
onDataRendered,
childEquivalenceIncludesCommonProps,
overlayRenderCount,
alwaysDetectDuplicates,
fadingEdgeLength,
shouldExcludeFromIncrementalMount,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -71,7 +71,6 @@ inline fun ResourcesScope.LazyList(
preAllocationHandler: PreAllocationHandler? =
context.lithoConfiguration.componentsConfig.preAllocationHandler,
childEquivalenceIncludesCommonProps: Boolean = true,
overlayRenderCount: Boolean = false,
alwaysDetectDuplicates: Boolean = false,
fadingEdgeLength: Dimen? = null,
shouldExcludeFromIncrementalMount: Boolean = false,
Expand Down Expand Up @@ -123,7 +122,6 @@ inline fun ResourcesScope.LazyList(
lazyCollectionController,
onDataRendered,
childEquivalenceIncludesCommonProps,
overlayRenderCount,
alwaysDetectDuplicates,
fadingEdgeLength,
shouldExcludeFromIncrementalMount,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,6 @@ inline fun ResourcesScope.LazyStaggeredGrid(
context.lithoConfiguration.componentsConfig.isReconciliationEnabled,
isIncrementalMountEnabled: Boolean = true,
childEquivalenceIncludesCommonProps: Boolean = true,
overlayRenderCount: Boolean = false,
alwaysDetectDuplicates: Boolean = false,
fadingEdgeLength: Dimen? = null,
preAllocationHandler: PreAllocationHandler? =
Expand Down Expand Up @@ -119,7 +118,6 @@ inline fun ResourcesScope.LazyStaggeredGrid(
lazyCollectionController,
onDataRendered,
childEquivalenceIncludesCommonProps,
overlayRenderCount,
alwaysDetectDuplicates,
fadingEdgeLength,
shouldExcludeFromIncrementalMount,
Expand Down

This file was deleted.

0 comments on commit 6119376

Please sign in to comment.