feat(web): prefer hide-ui over hide-ui=true

docat-org · Oct 25, 2024 · 91548d3 · 91548d3
1 parent 9f3bc63
commit 91548d3
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 10 deletions.
diff --git a/web/src/components/DocumentControlButtons.tsx b/web/src/components/DocumentControlButtons.tsx
@@ -41,12 +41,10 @@ export default function DocumentControlButtons(props: Props): JSX.Element {
       url = url.replace(props.version, 'latest')
     }
 
-    if (shareModalHideUi && !url.includes('?hide-ui=true')) {
-      const urlObject = URL.parse(url)
-      if (urlObject !== null) {
-        urlObject.searchParams.set('hide-ui', 'true')
-        url = urlObject.toString()
-      }
+    if (shareModalHideUi) {
+      const urlObject = new URL(url)
+      urlObject.search = 'hide-ui'
+      url = urlObject.toString()
     }
 
     return url

diff --git a/web/src/pages/Docs.tsx b/web/src/pages/Docs.tsx
@@ -24,7 +24,7 @@ export default function Docs(): JSX.Element {
   const hash = useRef(location.hash)
 
   const [version, setVersion] = useState<string>(params.version ?? 'latest')
-  const [hideUi, setHideUi] = useState<boolean>(searchParams.get('hide-ui') === 'true')
+  const [hideUi, setHideUi] = useState<boolean>(searchParams.get('hide-ui') === '' || searchParams.get('hide-ui') === 'true')
   const [iframeUpdateTrigger, setIframeUpdateTrigger] = useState<number>(0)
 
   // This provides the url for the iframe.
@@ -99,7 +99,7 @@ export default function Docs(): JSX.Element {
     hash: string,
     hideUi: boolean
   ): string => {
-    return `/${project}/${version}/${encodeURI(page)}${hash}${hideUi ? '?hide-ui=true' : ''}`
+    return `/${project}/${version}/${encodeURI(page)}${hash}${hideUi ? '?hide-ui' : ''}`
   }
 
   const updateUrl = (newVersion: string, hideUi: boolean): void => {
@@ -159,7 +159,7 @@ export default function Docs(): JSX.Element {
     const urlVersion = params.version ?? 'latest'
     const urlPage = params['*'] ?? ''
     const urlHash = location.hash
-    const urlHideUi = searchParams.get('hide-ui') === 'true'
+    const urlHideUi = searchParams.get('hide-ui') === '' || searchParams.get('hide-ui') === 'true'
 
     // update the state to the url params on first loadon
     if (urlProject !== project.current) {

diff --git a/web/src/pages/IframePageNotFound.tsx b/web/src/pages/IframePageNotFound.tsx
@@ -11,7 +11,7 @@ interface Props {
 }
 
 export default function IframeNotFound(props: Props): JSX.Element {
-  const link = `/${props.project}/${props.version}${props.hideUi ? '?hide-ui=true' : ''}`
+  const link = `/${props.project}/${props.version}${props.hideUi ? '?hide-ui' : ''}`
 
   return (
     <div className={styles['iframe-page-not-found']}>