From b5bbdd65e6a855c77d8bfa26894d431cc07a47f5 Mon Sep 17 00:00:00 2001
From: Alf Eaton <alf.eaton@overleaf.com>
Date: Mon, 22 Apr 2024 13:51:21 +0100
Subject: [PATCH] Allow read-only users to close the review panel (#18040)

GitOrigin-RevId: 93a470152da6dd215db07e3060621f32e5ae701d
---
 .../components/toolbar-header.jsx                        | 4 +---
 .../components/toolbar-header.test.jsx                   | 9 ---------
 2 files changed, 1 insertion(+), 12 deletions(-)

diff --git a/services/web/frontend/js/features/editor-navigation-toolbar/components/toolbar-header.jsx b/services/web/frontend/js/features/editor-navigation-toolbar/components/toolbar-header.jsx
index bed6fe78d6..5eb1517db2 100644
--- a/services/web/frontend/js/features/editor-navigation-toolbar/components/toolbar-header.jsx
+++ b/services/web/frontend/js/features/editor-navigation-toolbar/components/toolbar-header.jsx
@@ -39,8 +39,6 @@ const ToolbarHeader = React.memo(function ToolbarHeader({
 }) {
   const { t } = useTranslation()
   const shouldDisplayPublishButton = hasPublishPermissions && PublishButton
-  const shouldDisplayTrackChangesButton =
-    trackChangesVisible && !isRestrictedTokenMember
 
   return (
     <header
@@ -66,7 +64,7 @@ const ToolbarHeader = React.memo(function ToolbarHeader({
       <div className="toolbar-right">
         <OnlineUsersWidget onlineUsers={onlineUsers} goToUser={goToUser} />
 
-        {shouldDisplayTrackChangesButton && (
+        {trackChangesVisible && (
           <TrackChangesToggleButton
             onMouseDown={toggleReviewPanelOpen}
             disabled={historyIsOpen}
diff --git a/services/web/test/frontend/features/editor-navigation-toolbar/components/toolbar-header.test.jsx b/services/web/test/frontend/features/editor-navigation-toolbar/components/toolbar-header.test.jsx
index 5d73efd895..da8d601621 100644
--- a/services/web/test/frontend/features/editor-navigation-toolbar/components/toolbar-header.test.jsx
+++ b/services/web/test/frontend/features/editor-navigation-toolbar/components/toolbar-header.test.jsx
@@ -51,15 +51,6 @@ describe('<ToolbarHeader />', function () {
       screen.getByText('Review')
     })
 
-    it('is not displayed when "isRestrictedTokenMember" prop is set to true', function () {
-      const props = {
-        ...defaultProps,
-        isRestrictedTokenMember: true,
-      }
-      renderWithEditorContext(<ToolbarHeader {...props} />)
-      expect(screen.queryByText('Review')).to.not.exist
-    })
-
     it('is not displayed when "trackChangesVisible" prop is set to false', function () {
       const props = {
         ...defaultProps,