Skip to content

Commit 89bd386

Browse files
authored
Make all motion segments clickable (#10488)
1 parent c14f3c3 commit 89bd386

File tree

1 file changed

+4
-22
lines changed

1 file changed

+4
-22
lines changed

web/src/components/timeline/MotionSegment.tsx

Lines changed: 4 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ export function MotionSegment({
3939
shouldShowRoundedCorners,
4040
} = useEventSegmentUtils(segmentDuration, events, severityType);
4141

42-
const { getMotionSegmentValue, interpolateMotionAudioData, getMotionStart } =
42+
const { getMotionSegmentValue, interpolateMotionAudioData } =
4343
useMotionSegmentUtils(segmentDuration, motion_events);
4444

4545
const { alignStartDateToTimeline, alignEndDateToTimeline } = useEventUtils(
@@ -66,15 +66,6 @@ export function MotionSegment({
6666
[shouldShowRoundedCorners, segmentTime],
6767
);
6868

69-
const startTimestamp = useMemo(() => {
70-
const eventStart = getMotionStart(segmentTime);
71-
if (eventStart) {
72-
return alignStartDateToTimeline(eventStart);
73-
}
74-
// we know that these deps are correct
75-
// eslint-disable-next-line react-hooks/exhaustive-deps
76-
}, [getMotionStart, segmentTime]);
77-
7869
const timestamp = useMemo(() => new Date(segmentTime * 1000), [segmentTime]);
7970
const segmentKey = useMemo(() => segmentTime, [segmentTime]);
8071

@@ -179,19 +170,10 @@ export function MotionSegment({
179170
};
180171

181172
const segmentClick = useCallback(() => {
182-
if (
183-
startTimestamp &&
184-
setHandlebarTime &&
185-
(firstHalfSegmentWidth > 1 || secondHalfSegmentWidth > 1)
186-
) {
187-
setHandlebarTime(startTimestamp);
173+
if (setHandlebarTime) {
174+
setHandlebarTime(segmentTime);
188175
}
189-
}, [
190-
startTimestamp,
191-
setHandlebarTime,
192-
firstHalfSegmentWidth,
193-
secondHalfSegmentWidth,
194-
]);
176+
}, [segmentTime, setHandlebarTime]);
195177

196178
return (
197179
<div

0 commit comments

Comments
 (0)