Merge pull request #786 from docat-org/fix/hide-copy-button-http

Fix: Remove copy button when copying is not supported
docat-org · Feb 1, 2024 · 31f5e24 · 31f5e24
2 parents 1b759dc + d023b93
commit 31f5e24
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 12 deletions.
diff --git a/web/src/components/DocumentControlButtons.tsx b/web/src/components/DocumentControlButtons.tsx
@@ -28,6 +28,9 @@ export default function DocumentControlButtons(props: Props): JSX.Element {
   const [shareModalUseLatest, setShareModalUseLatest] = useState<boolean>(false)
   const [shareModalHideUi, setShareModalHideUi] = useState<boolean>(false)
 
+  // Cannot copy when page is served over HTTP
+  const canCopy = navigator.clipboard !== undefined
+
   const getShareUrl = (): string => {
     // adapt the current URL so we can leave Docs.tsx's state as refs
     // (which means if the page was passed down as a prop it wouldn't update correctly)
@@ -97,18 +100,20 @@ export default function DocumentControlButtons(props: Props): JSX.Element {
         <div className={styles['share-modal']}>
           <div className={styles['share-modal-link-container']}>
             <p className={styles['share-modal-link']}>{getShareUrl()}</p>
-            <div className={styles['share-modal-copy-container']}>
-              <button
-                className={styles['share-modal-copy']}
-                onClick={() => {
-                  void (async () => {
-                    await navigator.clipboard.writeText(getShareUrl())
-                  })()
-                }}
-              >
-                Copy
-              </button>
-            </div>
+            {canCopy && (
+              <div className={styles['share-modal-copy-container']}>
+                <button
+                  className={styles['share-modal-copy']}
+                  onClick={() => {
+                    void (async () => {
+                      await navigator.clipboard.writeText(getShareUrl())
+                    })()
+                  }}
+                >
+                  Copy
+                </button>
+              </div>
+            )}
           </div>
 
           <FormGroup>

diff --git a/web/src/style/components/DocumentControlButtons.module.css b/web/src/style/components/DocumentControlButtons.module.css
@@ -86,6 +86,7 @@
   -moz-user-select: all;
   -webkit-user-select: all;
   font-size: small;
+  width: 100%;
 }
 
 .share-modal-copy-container {
@@ -131,6 +132,10 @@
     align-items: center;
   }
 
+  .share-modal-link {
+    width: auto;
+  }
+
   .share-modal-copy-container {
     margin-left: 0;
     margin-top: 1rem;