fix: remote double measurement on load, as the observer takes care of that now

Author: olzzon

packages/webui/src/client/lib/VirtualElement.tsx

@@ -64,7 +64,6 @@ export function VirtualElement({
 	className?: string
 }>): JSX.Element | null {
 	const resizeObserverManager = ElementObserverManager.getInstance()
-	const [waitForInitialLoad, setWaitForInitialLoad] = useState(true)
 	const [inView, setInView] = useState(initialShow ?? false)
 	const [isShowingChildren, setIsShowingChildren] = useState(inView)
 
@@ -197,26 +196,6 @@ export function VirtualElement({
 	}, [ref, inView])
 
 	useEffect(() => {
-		if (inView === true) {
-			setIsShowingChildren(true)
-
-			// Schedule a measurement after a short delay
-			if (waitForInitialLoad && ref) {
-				const initialMeasurementTimeout = window.setTimeout(() => {
-					const measurements = measureElement(ref, placeholderHeight)
-					if (measurements) {
-						setMeasurements(measurements)
-						setWaitForInitialLoad(false)
-					}
-				}, 800)
-
-				return () => {
-					window.clearTimeout(initialMeasurementTimeout)
-				}
-			}
-			return
-		}
-
 		let idleCallback: number | undefined
 		let optimizeTimeout: number | undefined
 

packages/webui/src/client/ui/SegmentTimeline/SegmentTimelineContainer.tsx

@@ -547,7 +547,6 @@ const SegmentTimelineContainerContent = withResolvedSegment(
 
 			this.visibilityChangeTimeout = setTimeout(() => {
 				if (entries[0].intersectionRatio < 0.99 && !isMaintainingFocus() && Date.now() - this.mountedTime > 2000) {
-					console.log('Segment out of view :', this.props.segmentId)
 					if (typeof this.props.onSegmentScroll === 'function') this.props.onSegmentScroll()
 					this.isVisible = false
 				} else {