From 04fd41dc2877777951c9a49a5c1ac9131683ea75 Mon Sep 17 00:00:00 2001
From: Alexander Onnikov <Alexander.Onnikov@xored.com>
Date: Wed, 13 Aug 2025 00:20:23 +0700
Subject: [PATCH 1/4] EQMS-1617 Fix document and category links

Signed-off-by: Alexander Onnikov <Alexander.Onnikov@xored.com>
---
 .../presenters/CategoryPresenter.svelte       | 27 +++++------
 .../presenters/DocumentPresenter.svelte       | 48 +++++++++++--------
 .../document/presenters/TitlePresenter.svelte | 17 ++++---
 .../right-panel/DocumentInfoTab.svelte        |  2 +-
 4 files changed, 50 insertions(+), 44 deletions(-)

diff --git a/plugins/controlled-documents-resources/src/components/category/presenters/CategoryPresenter.svelte b/plugins/controlled-documents-resources/src/components/category/presenters/CategoryPresenter.svelte
index bfcf1578a42..a8332cebd60 100644
--- a/plugins/controlled-documents-resources/src/components/category/presenters/CategoryPresenter.svelte
+++ b/plugins/controlled-documents-resources/src/components/category/presenters/CategoryPresenter.svelte
@@ -18,8 +18,8 @@
   import { Ref, WithLookup } from '@hcengineering/core'
   import { getEmbeddedLabel } from '@hcengineering/platform'
   import { getClient } from '@hcengineering/presentation'
-  import { getPanelURI, Icon, tooltip } from '@hcengineering/ui'
-  import view from '@hcengineering/view'
+  import { Icon, tooltip } from '@hcengineering/ui'
+  import { DocNavLink } from '@hcengineering/view-resources'
 
   import documents from '../../../plugin'
 
@@ -41,19 +41,18 @@
 </script>
 
 {#if category}
-  <a
-    class={!disableClick ? 'flex-presenter' : 'category-presenter'}
-    href={!disableClick
-      ? `#${getPanelURI(view.component.EditDoc, category._id, category._class, 'content')}`
-      : undefined}
-    class:inline-presenter={inline}
-    use:tooltip={{ label: getEmbeddedLabel(category.title) }}
-  >
-    <div class="icon category-icon">
-      <Icon icon={documents.icon.Document} size={'small'} />
+  <DocNavLink object={category} disabled={disableClick} accent {inline}>
+    <div
+      class={!disableClick ? 'flex-presenter' : 'category-presenter'}
+      class:inline-presenter={inline}
+      use:tooltip={{ label: getEmbeddedLabel(category.title) }}
+    >
+      <div class="icon category-icon">
+        <Icon icon={documents.icon.Document} size={'small'} />
+      </div>
+      <span class="label fs-bold whitespace-nowrap">{category.code}</span>
     </div>
-    <span class="label whitespace-nowrap">{category.code}</span>
-  </a>
+  </DocNavLink>
 {/if}
 
 <style lang="scss">
diff --git a/plugins/controlled-documents-resources/src/components/document/presenters/DocumentPresenter.svelte b/plugins/controlled-documents-resources/src/components/document/presenters/DocumentPresenter.svelte
index 619f17b2aae..f7d3d8a2d0d 100644
--- a/plugins/controlled-documents-resources/src/components/document/presenters/DocumentPresenter.svelte
+++ b/plugins/controlled-documents-resources/src/components/document/presenters/DocumentPresenter.svelte
@@ -17,7 +17,8 @@
   import { Document } from '@hcengineering/controlled-documents'
   import { WithLookup } from '@hcengineering/core'
   import { getEmbeddedLabel } from '@hcengineering/platform'
-  import { getPanelURI, tooltip } from '@hcengineering/ui'
+  import { tooltip } from '@hcengineering/ui'
+  import { DocNavLink } from '@hcengineering/view-resources'
 
   import DocumentIcon from '../../icons/DocumentIcon.svelte'
   import documents from '../../../plugin'
@@ -29,16 +30,17 @@
   export let withIcon = false
   export let withTitle = false
   export let disableLink = false
-  export let editable = false
+  export let canEdit = false
 
   $: documentCode = value !== undefined ? value.code : ''
-  $: noUnderline = disableLink && !editable
+  $: noUnderline = disableLink && !canEdit
+  $: disabled = disableLink && !canEdit
   $: title = withTitle ? `${documentCode} ${value?.title}` : documentCode
 
   const dispatch = createEventDispatcher()
 
   function handleClick (event: MouseEvent): void {
-    if (!editable) {
+    if (!canEdit) {
       return
     }
 
@@ -47,23 +49,29 @@
 </script>
 
 {#if value}
-  <a
-    class="flex-presenter"
-    href={!disableLink ? `#${getPanelURI(documents.component.EditDoc, value._id, value._class, 'content')}` : undefined}
-    class:inline-presenter={inline}
-    class:noBold={isRegular}
-    class:no-underline={noUnderline}
-    class:cursor-inherit={noUnderline}
-    use:tooltip={{ label: getEmbeddedLabel(title) }}
-    on:click={handleClick}
+  <DocNavLink
+    object={value}
+    component={documents.component.EditDoc}
+    onClick={canEdit ? handleClick : undefined}
+    accent={!isRegular}
+    {disabled}
+    {noUnderline}
+    {inline}
   >
-    {#if withIcon}
-      <div class="icon">
-        <DocumentIcon size="small" />
-      </div>
-    {/if}
-    <span class="label nowrap" class:no-underline={noUnderline} class:label-gray={isGray}>{title}</span>
-  </a>
+    <div class="flex-presenter" use:tooltip={{ label: getEmbeddedLabel(title) }}>
+      {#if withIcon}
+        <div class="icon">
+          <DocumentIcon size="small" />
+        </div>
+      {/if}
+      <span
+        class="label nowrap"
+        class:fs-bold={!isRegular}
+        class:no-underline={noUnderline || disableLink}
+        class:label-gray={isGray}>{title}</span
+      >
+    </div>
+  </DocNavLink>
 {/if}
 
 <style lang="scss">
diff --git a/plugins/controlled-documents-resources/src/components/document/presenters/TitlePresenter.svelte b/plugins/controlled-documents-resources/src/components/document/presenters/TitlePresenter.svelte
index d755c897441..4b97b32fcb1 100644
--- a/plugins/controlled-documents-resources/src/components/document/presenters/TitlePresenter.svelte
+++ b/plugins/controlled-documents-resources/src/components/document/presenters/TitlePresenter.svelte
@@ -15,19 +15,18 @@
 <script lang="ts">
   import { Document } from '@hcengineering/controlled-documents'
   import { WithLookup } from '@hcengineering/core'
-  import { getPanelURI } from '@hcengineering/ui'
-  import documents from '../../../plugin'
+  import { tooltip } from '@hcengineering/ui'
+  import { DocNavLink } from '@hcengineering/view-resources'
+  import { getEmbeddedLabel } from '@hcengineering/platform'
 
   export let value: WithLookup<Document>
   export let inline = false
 </script>
 
 {#if value}
-  <a
-    class="flex-presenter"
-    href="#{getPanelURI(documents.component.EditDoc, value._id, value._class, 'content')}"
-    class:inline-presenter={inline}
-  >
-    <span class="label nowrap">{value.title}</span>
-  </a>
+  <DocNavLink object={value} {inline}>
+    <div class="flex-presenter" class:inline-presenter={inline} use:tooltip={{ label: getEmbeddedLabel(value.title) }}>
+      <span class="label fs-bold nowrap">{value.title}</span>
+    </div>
+  </DocNavLink>
 {/if}
diff --git a/plugins/controlled-documents-resources/src/components/document/right-panel/DocumentInfoTab.svelte b/plugins/controlled-documents-resources/src/components/document/right-panel/DocumentInfoTab.svelte
index 6e215f655f6..34a3973f731 100644
--- a/plugins/controlled-documents-resources/src/components/document/right-panel/DocumentInfoTab.svelte
+++ b/plugins/controlled-documents-resources/src/components/document/right-panel/DocumentInfoTab.svelte
@@ -96,7 +96,7 @@
           value={$controlledDocument}
           isRegular
           disableLink
-          editable={isInitialEditableDraft}
+          canEdit={isInitialEditableDraft}
           on:edit={(e) => {
             handleCodeEdit(e.detail)
           }}

From e6ac2b6a0a994b4729f3d3ce9442049d20afe945 Mon Sep 17 00:00:00 2001
From: Alexander Onnikov <Alexander.Onnikov@xored.com>
Date: Wed, 13 Aug 2025 00:30:55 +0700
Subject: [PATCH 2/4] fix: update no-underline condition

Signed-off-by: Alexander Onnikov <Alexander.Onnikov@xored.com>
---
 .../src/components/document/presenters/DocumentPresenter.svelte | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/plugins/controlled-documents-resources/src/components/document/presenters/DocumentPresenter.svelte b/plugins/controlled-documents-resources/src/components/document/presenters/DocumentPresenter.svelte
index f7d3d8a2d0d..5b0ffb431ea 100644
--- a/plugins/controlled-documents-resources/src/components/document/presenters/DocumentPresenter.svelte
+++ b/plugins/controlled-documents-resources/src/components/document/presenters/DocumentPresenter.svelte
@@ -67,7 +67,7 @@
       <span
         class="label nowrap"
         class:fs-bold={!isRegular}
-        class:no-underline={noUnderline || disableLink}
+        class:no-underline={noUnderline}
         class:label-gray={isGray}>{title}</span
       >
     </div>

From 176448d5d0bc7f601ff37400630b6a6e6fefbac7 Mon Sep 17 00:00:00 2001
From: Alexander Onnikov <Alexander.Onnikov@xored.com>
Date: Wed, 13 Aug 2025 11:08:51 +0700
Subject: [PATCH 3/4] rush format

Signed-off-by: Alexander Onnikov <Alexander.Onnikov@xored.com>
---
 .../document/presenters/DocumentPresenter.svelte           | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/plugins/controlled-documents-resources/src/components/document/presenters/DocumentPresenter.svelte b/plugins/controlled-documents-resources/src/components/document/presenters/DocumentPresenter.svelte
index 5b0ffb431ea..d875c2e6a91 100644
--- a/plugins/controlled-documents-resources/src/components/document/presenters/DocumentPresenter.svelte
+++ b/plugins/controlled-documents-resources/src/components/document/presenters/DocumentPresenter.svelte
@@ -64,11 +64,8 @@
           <DocumentIcon size="small" />
         </div>
       {/if}
-      <span
-        class="label nowrap"
-        class:fs-bold={!isRegular}
-        class:no-underline={noUnderline}
-        class:label-gray={isGray}>{title}</span
+      <span class="label nowrap" class:fs-bold={!isRegular} class:no-underline={noUnderline} class:label-gray={isGray}
+        >{title}</span
       >
     </div>
   </DocNavLink>

From 3427969d48015f83565a923c1c2663dfdc291fba Mon Sep 17 00:00:00 2001
From: Alexander Onnikov <Alexander.Onnikov@xored.com>
Date: Wed, 13 Aug 2025 11:10:25 +0700
Subject: [PATCH 4/4] fix ui tests

Signed-off-by: Alexander Onnikov <Alexander.Onnikov@xored.com>
---
 .../sanity/tests/model/documents/document-reason-impact-page.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/qms-tests/sanity/tests/model/documents/document-reason-impact-page.ts b/qms-tests/sanity/tests/model/documents/document-reason-impact-page.ts
index 097ce4bf0fb..6f8ede8621d 100644
--- a/qms-tests/sanity/tests/model/documents/document-reason-impact-page.ts
+++ b/qms-tests/sanity/tests/model/documents/document-reason-impact-page.ts
@@ -25,7 +25,7 @@ export class DocumentReasonAndImpactPage extends DocumentCommonPage {
     this.textReason = page.locator('//div[contains(@class, "title") and text()="Reason"]/..')
     this.textImpactAnalysis = page.locator('//div[contains(@class, "title") and text()="Impact analysis"]/..')
     this.textImpactedDocuments = page.locator(
-      '//div[contains(@class, "title") and text()="Impacted documents"]/..//a/span'
+      '//div[contains(@class, "title") and text()="Impacted documents"]/..//div/span'
     )
   }