From ef90950cd3e9940b27391112e427fef6c1236f7a Mon Sep 17 00:00:00 2001
From: David <33458145+davidmcpowell@users.noreply.github.com>
Date: Mon, 28 Oct 2024 15:39:02 +0000
Subject: [PATCH] Merge pull request #21439 from
 overleaf/dk-accept-reject-all-tooltip

Added tooltips to "Accept all" and "Reject all" options

GitOrigin-RevId: 62887e4e26964119cbb2c2aa140565c46aba94fe
---
 .../components/review-tooltip-menu.tsx        | 30 +++++++++++--------
 .../app/editor/review-panel-new.less          |  3 +-
 .../pages/editor/review-panel-new.scss        |  3 +-
 3 files changed, 22 insertions(+), 14 deletions(-)

diff --git a/services/web/frontend/js/features/review-panel-new/components/review-tooltip-menu.tsx b/services/web/frontend/js/features/review-panel-new/components/review-tooltip-menu.tsx
index 50c154dcc9..d750ac7f78 100644
--- a/services/web/frontend/js/features/review-panel-new/components/review-tooltip-menu.tsx
+++ b/services/web/frontend/js/features/review-panel-new/components/review-tooltip-menu.tsx
@@ -29,6 +29,7 @@ import {
 } from '../context/ranges-context'
 import { isInsertOperation } from '@/utils/operations'
 import { isCursorNearViewportEdge } from '@/features/source-editor/utils/is-cursor-near-edge'
+import OLTooltip from '@/features/ui/components/ol/ol-tooltip'
 
 const ReviewTooltipMenu: FC = () => {
   const state = useCodeMirrorStateContext()
@@ -129,18 +130,23 @@ const ReviewTooltipMenuContent: FC<{
       {showChangesButtons && (
         <>
           <div className="review-tooltip-menu-divider" />
-          <button
-            className="review-tooltip-menu-button"
-            onClick={acceptChangesHandler}
-          >
-            <MaterialIcon type="check" />
-          </button>
-          <button
-            className="review-tooltip-menu-button"
-            onClick={rejectChangesHandler}
-          >
-            <MaterialIcon type="clear" />
-          </button>
+          <OLTooltip id="accept-all-changes" description={t('accept_all')}>
+            <button
+              className="review-tooltip-menu-button"
+              onClick={acceptChangesHandler}
+            >
+              <MaterialIcon type="check" />
+            </button>
+          </OLTooltip>
+
+          <OLTooltip id="reject-all-changes" description={t('reject_all')}>
+            <button
+              className="review-tooltip-menu-button"
+              onClick={rejectChangesHandler}
+            >
+              <MaterialIcon type="clear" />
+            </button>
+          </OLTooltip>
         </>
       )}
     </div>
diff --git a/services/web/frontend/stylesheets/app/editor/review-panel-new.less b/services/web/frontend/stylesheets/app/editor/review-panel-new.less
index b455bf6d1b..e45ea94225 100644
--- a/services/web/frontend/stylesheets/app/editor/review-panel-new.less
+++ b/services/web/frontend/stylesheets/app/editor/review-panel-new.less
@@ -621,10 +621,11 @@
 
     .review-panel-entry-hover {
       .review-panel-entry-content {
-        display: initial;
+        display: flex;
         position: absolute;
         left: -200px;
         top: 0;
+        padding: @spacing-04;
       }
     }
 
diff --git a/services/web/frontend/stylesheets/bootstrap-5/pages/editor/review-panel-new.scss b/services/web/frontend/stylesheets/bootstrap-5/pages/editor/review-panel-new.scss
index 1cb90a476d..71f5f77722 100644
--- a/services/web/frontend/stylesheets/bootstrap-5/pages/editor/review-panel-new.scss
+++ b/services/web/frontend/stylesheets/bootstrap-5/pages/editor/review-panel-new.scss
@@ -612,10 +612,11 @@
 
     .review-panel-entry-hover {
       .review-panel-entry-content {
-        display: initial;
+        display: flex;
         position: absolute;
         left: -200px;
         top: 0;
+        padding: var(--spacing-04);
       }
     }