From a40fb42fedf7c3c5a24da1f065892cb2c508263f Mon Sep 17 00:00:00 2001
From: David <33458145+davidmcpowell@users.noreply.github.com>
Date: Fri, 4 Oct 2024 10:15:46 +0100
Subject: [PATCH] Merge pull request #20712 from overleaf/dp-click-mini-panel

Clicking on an entry in the mini review panel opens the full panel

GitOrigin-RevId: 4c925435a09f56ae5b02c53951185c67eb0413a2
---
 .../review-panel-new/components/review-panel-entry.tsx     | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/services/web/frontend/js/features/review-panel-new/components/review-panel-entry.tsx b/services/web/frontend/js/features/review-panel-new/components/review-panel-entry.tsx
index c6b8937181..708c083df0 100644
--- a/services/web/frontend/js/features/review-panel-new/components/review-panel-entry.tsx
+++ b/services/web/frontend/js/features/review-panel-new/components/review-panel-entry.tsx
@@ -11,6 +11,7 @@ import {
   highlightRanges,
 } from '@/features/source-editor/extensions/ranges'
 import { useEditorManagerContext } from '@/features/ide-react/context/editor-manager-context'
+import { useLayoutContext } from '@/shared/context/layout-context'
 
 export const ReviewPanelEntry: FC<{
   position: number
@@ -34,9 +35,14 @@ export const ReviewPanelEntry: FC<{
   const view = useCodeMirrorViewContext()
   const { openDocId } = useEditorManagerContext()
   const [focused, setFocused] = useState(false)
+  const { setReviewPanelOpen } = useLayoutContext()
 
   const highlighted = isSelectionWithinOp(op, state.selection.main)
 
+  const openReviewPanel = useCallback(() => {
+    setReviewPanelOpen(true)
+  }, [setReviewPanelOpen])
+
   const focusHandler = useCallback(() => {
     if (selectLineOnFocus) {
       openDocId(docId, { gotoOffset: position, keepCurrentView: true })
@@ -55,6 +61,7 @@ export const ReviewPanelEntry: FC<{
 
   return (
     <div
+      onMouseDown={openReviewPanel} // Using onMouseDown rather than onClick to guarantee that it fires before onFocus
       onFocus={focusHandler}
       onBlur={() => setFocused(false)}
       onMouseEnter={() => {