From ff44afb74cfaec7c81578578debe4e9251ed54e9 Mon Sep 17 00:00:00 2001
From: Alf Eaton <alf.eaton@overleaf.com>
Date: Wed, 27 Apr 2022 11:29:40 +0100
Subject: [PATCH] Only show compile shortcut when editor is visible (#7773)

GitOrigin-RevId: 0b6d25d7012e82a676e3a782730fb6ca481cd1e8
---
 .../pdf-preview/components/pdf-compile-button-inner.js   | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/services/web/frontend/js/features/pdf-preview/components/pdf-compile-button-inner.js b/services/web/frontend/js/features/pdf-preview/components/pdf-compile-button-inner.js
index 2ba54481a7..165cf38049 100644
--- a/services/web/frontend/js/features/pdf-preview/components/pdf-compile-button-inner.js
+++ b/services/web/frontend/js/features/pdf-preview/components/pdf-compile-button-inner.js
@@ -8,12 +8,17 @@ import { useLayoutContext } from '../../../shared/context/layout-context'
 const modifierKey = /Mac/i.test(navigator.platform) ? 'Cmd' : 'Ctrl'
 
 function PdfCompileButtonInner({ startCompile, compiling }) {
-  const { detachRole } = useLayoutContext()
+  const { detachRole, view, pdfLayout } = useLayoutContext()
 
   const { t } = useTranslation()
 
   const compileButtonLabel = compiling ? t('compiling') + '…' : t('recompile')
 
+  // show the compile shortcut when the editor is visible
+  const showCompileShortcut =
+    detachRole !== 'detached' &&
+    (view === 'editor' || pdfLayout === 'sideBySide')
+
   return (
     <OverlayTrigger
       placement="bottom"
@@ -21,7 +26,7 @@ function PdfCompileButtonInner({ startCompile, compiling }) {
       overlay={
         <Tooltip id="tooltip-logs-toggle" className="keyboard-tooltip">
           {t('recompile_pdf')}{' '}
-          {detachRole !== 'detached' && (
+          {showCompileShortcut && (
             <span className="keyboard-shortcut">({modifierKey} + Enter)</span>
           )}
         </Tooltip>