Don't show minimap when view is not scrollable (#10062)

blakeblackshear · Feb 26, 2024 · 1dd904d · 1dd904d
1 parent 76ad91f
commit 1dd904d
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/web/src/views/events/DesktopEventView.tsx b/web/src/views/events/DesktopEventView.tsx
@@ -86,6 +86,14 @@ export default function DesktopEventView({
  return current;
  }, [reviewItems, severity]);
 
+ const showMinimap = useMemo(() => {
+ if (!contentRef.current) {
+ return false;
+ }
+
+ return contentRef.current.scrollHeight > contentRef.current.clientHeight
+ }, [contentRef.current?.scrollHeight])
+
  // review interaction
 
  const pagingObserver = useRef<IntersectionObserver | null>();
@@ -310,7 +318,7 @@ export default function DesktopEventView({
  timestampSpread={15}
  timelineStart={timeRange.before}
  timelineEnd={timeRange.after}
- showMinimap
+ showMinimap={showMinimap}
  minimapStartTime={minimapBounds.start}
  minimapEndTime={minimapBounds.end}
  events={reviewItems.all}