From 46b70a2e9c0b987c32425e6f23d1bdd3baa1021a Mon Sep 17 00:00:00 2001
From: Andrey Sobolev <haiodo@users.noreply.github.com>
Date: Fri, 25 Nov 2022 00:29:37 +0700
Subject: [PATCH] TSK-435: Fix create issue edit focus lost. (#2396)

Signed-off-by: Andrey Sobolev <haiodo@gmail.com>
---
 packages/platform/src/i18n.ts                 |  1 +
 .../components/CollaborationDiffViewer.svelte |  2 +-
 .../src/components/CollaboratorEditor.svelte  |  2 +-
 .../src/components/TextEditor.svelte          |  2 +-
 .../components/notifications/Notification.ts  |  4 +-
 .../src/components/CreateIssue.svelte         | 51 +++++++++++--------
 .../issues/IssueNotification.svelte           |  2 +-
 .../issues/edit/CreateSubIssue.svelte         |  1 +
 .../templates/EditIssueTemplate.svelte        | 31 +++++------
 plugins/tracker/src/index.ts                  | 15 +-----
 pods/account/src/index.ts                     |  8 ++-
 11 files changed, 59 insertions(+), 60 deletions(-)

diff --git a/packages/platform/src/i18n.ts b/packages/platform/src/i18n.ts
index 3d8c17feb5..93530b753f 100644
--- a/packages/platform/src/i18n.ts
+++ b/packages/platform/src/i18n.ts
@@ -67,6 +67,7 @@ async function loadTranslationsForComponent (plugin: Plugin, locale: string): Pr
   try {
     return (await loader(locale)) as Record<string, IntlString> | Status
   } catch (err) {
+    console.error(err)
     const status = unknownError(err)
     await setPlatformStatus(status)
     return status
diff --git a/packages/text-editor/src/components/CollaborationDiffViewer.svelte b/packages/text-editor/src/components/CollaborationDiffViewer.svelte
index eeacc16076..501acaf1d9 100644
--- a/packages/text-editor/src/components/CollaborationDiffViewer.svelte
+++ b/packages/text-editor/src/components/CollaborationDiffViewer.svelte
@@ -124,7 +124,7 @@
   .ProseMirror {
     flex-grow: 1;
     overflow: auto;
-    min-height: 3rem;
+    min-height: inherit !important;
     max-height: inherit !important;
     outline: none;
     line-height: 150%;
diff --git a/packages/text-editor/src/components/CollaboratorEditor.svelte b/packages/text-editor/src/components/CollaboratorEditor.svelte
index 994fcbb646..bc1f82f66c 100644
--- a/packages/text-editor/src/components/CollaboratorEditor.svelte
+++ b/packages/text-editor/src/components/CollaboratorEditor.svelte
@@ -587,7 +587,7 @@
 <style lang="scss" global>
   .ProseMirror {
     flex-grow: 1;
-    min-height: 3rem;
+    min-height: inherit !important;
     max-height: inherit !important;
     outline: none;
     line-height: 150%;
diff --git a/packages/text-editor/src/components/TextEditor.svelte b/packages/text-editor/src/components/TextEditor.svelte
index ebd0e126d6..8f5e8d6eed 100644
--- a/packages/text-editor/src/components/TextEditor.svelte
+++ b/packages/text-editor/src/components/TextEditor.svelte
@@ -216,7 +216,7 @@
 <style lang="scss" global>
   .ProseMirror {
     overflow-y: auto;
-    min-height: 3rem;
+    min-height: inherit !important;
     max-height: inherit !important;
     outline: none;
     line-height: 150%;
diff --git a/packages/ui/src/components/notifications/Notification.ts b/packages/ui/src/components/notifications/Notification.ts
index 6066c477f3..7526852a9d 100644
--- a/packages/ui/src/components/notifications/Notification.ts
+++ b/packages/ui/src/components/notifications/Notification.ts
@@ -1,11 +1,11 @@
 import { NotificationPosition } from './NotificationPosition'
 import { NotificationSeverity } from './NotificationSeverity'
-import { AnyComponent } from '../../types'
+import { AnyComponent, AnySvelteComponent } from '../../types'
 
 export interface Notification {
   id: string
   title: string
-  component: AnyComponent
+  component: AnyComponent | AnySvelteComponent
   subTitle?: string
   subTitlePostfix?: string
   position: NotificationPosition
diff --git a/plugins/tracker-resources/src/components/CreateIssue.svelte b/plugins/tracker-resources/src/components/CreateIssue.svelte
index 3786ce787c..e6a435a4f3 100644
--- a/plugins/tracker-resources/src/components/CreateIssue.svelte
+++ b/plugins/tracker-resources/src/components/CreateIssue.svelte
@@ -95,6 +95,14 @@
   let labels: TagReference[] = draft?.labels || []
   let objectId: Ref<Issue> = draft?.issueId || generateId()
 
+  function toIssue (initials: AttachedData<Issue>, draft: IssueDraft | null): AttachedData<Issue> {
+    if (draft == null) {
+      return { ...initials }
+    }
+    const { labels, subIssues, ...issue } = draft
+    return { ...initials, ...issue }
+  }
+
   let object: AttachedData<Issue> = originalIssue
     ? {
         ...originalIssue,
@@ -105,7 +113,8 @@
         reports: 0,
         childInfo: []
       }
-    : {
+    : toIssue(
+      {
         title: '',
         description: '',
         assignee,
@@ -122,9 +131,10 @@
         reportedTime: 0,
         estimation: 0,
         reports: 0,
-        childInfo: [],
-        ...(draft || {})
-      }
+        childInfo: []
+      },
+      draft
+    )
 
   function resetObject (): void {
     templateId = undefined
@@ -260,7 +270,7 @@
       return
     }
 
-    parentIssue = await client.findOne(tracker.class.Issue, { _id: draft.parentIssue })
+    parentIssue = await client.findOne(tracker.class.Issue, { _id: draft.parentIssue as Ref<Issue> })
   }
 
   $: originalIssue && setPropsFromOriginalIssue()
@@ -288,7 +298,7 @@
   }
 
   async function isDraftEmpty (draft: Data<IssueDraft>): Promise<boolean> {
-    const emptyDraft = {
+    const emptyDraft: Partial<IssueDraft> = {
       assignee: null,
       description: '',
       dueDate: null,
@@ -304,7 +314,7 @@
     }
 
     for (const key of Object.keys(emptyDraft)) {
-      if (!deepEqual(emptyDraft[key], draft[key])) {
+      if (!deepEqual((emptyDraft as any)[key], (draft as any)[key])) {
         return false
       }
     }
@@ -509,6 +519,7 @@
     }
 
     const notification: Notification = {
+      id: generateId(),
       title: tracker.string.IssueCreated,
       subTitle: getTitle(object.title),
       severity: NotificationSeverity.Success,
@@ -695,20 +706,18 @@
     <ParentIssue issue={parentIssue} on:close={clearParentIssue} />
   {/if}
   <EditBox bind:value={object.title} placeholder={tracker.string.IssueTitlePlaceholder} kind={'large-style'} focus />
-  {#key object.description}
-    <AttachmentStyledBox
-      bind:this={descriptionBox}
-      {objectId}
-      _class={tracker.class.Issue}
-      space={_space}
-      alwaysEdit
-      showButtons={false}
-      maxHeight={'20vh'}
-      bind:content={object.description}
-      placeholder={tracker.string.IssueDescriptionPlaceholder}
-      on:changeSize={() => dispatch('changeContent')}
-    />
-  {/key}
+  <AttachmentStyledBox
+    bind:this={descriptionBox}
+    {objectId}
+    _class={tracker.class.Issue}
+    space={_space}
+    alwaysEdit
+    showButtons={false}
+    maxHeight={'20vh'}
+    bind:content={object.description}
+    placeholder={tracker.string.IssueDescriptionPlaceholder}
+    on:changeSize={() => dispatch('changeContent')}
+  />
   <IssueTemplateChilds
     bind:children={subIssues}
     sprint={object.sprint}
diff --git a/plugins/tracker-resources/src/components/issues/IssueNotification.svelte b/plugins/tracker-resources/src/components/issues/IssueNotification.svelte
index 7c9eb271a7..5652aa8bdf 100644
--- a/plugins/tracker-resources/src/components/issues/IssueNotification.svelte
+++ b/plugins/tracker-resources/src/components/issues/IssueNotification.svelte
@@ -18,7 +18,7 @@
   import IssuePresenter from './IssuePresenter.svelte'
   import tracker from '../../plugin'
 
-  export let notification: Notification = {}
+  export let notification: Notification
   export let onRemove: () => void
 
   const issueQuery = createQuery()
diff --git a/plugins/tracker-resources/src/components/issues/edit/CreateSubIssue.svelte b/plugins/tracker-resources/src/components/issues/edit/CreateSubIssue.svelte
index 107ec1b793..e2dcdca16d 100644
--- a/plugins/tracker-resources/src/components/issues/edit/CreateSubIssue.svelte
+++ b/plugins/tracker-resources/src/components/issues/edit/CreateSubIssue.svelte
@@ -67,6 +67,7 @@
 
   function resetToDefaults () {
     newIssue = getIssueDefaults()
+    labels = []
     focusIssueTitle?.()
   }
 
diff --git a/plugins/tracker-resources/src/components/templates/EditIssueTemplate.svelte b/plugins/tracker-resources/src/components/templates/EditIssueTemplate.svelte
index b202c9454b..d62db4da04 100644
--- a/plugins/tracker-resources/src/components/templates/EditIssueTemplate.svelte
+++ b/plugins/tracker-resources/src/components/templates/EditIssueTemplate.svelte
@@ -212,21 +212,21 @@
             kind="large-style"
           />
           <div class="flex-between mt-6">
-            {#key description}
-              <div class="flex-grow">
-                <AttachmentStyledBox
-                  bind:this={descriptionBox}
-                  objectId={_id}
-                  _class={tracker.class.Issue}
-                  space={template.space}
-                  alwaysEdit
-                  showButtons
-                  maxHeight={'card'}
-                  bind:content={description}
-                  placeholder={tracker.string.IssueDescriptionPlaceholder}
-                />
-              </div>
-            {/key}
+            <div class="flex-grow">
+              <AttachmentStyledBox
+                bind:this={descriptionBox}
+                objectId={_id}
+                _class={tracker.class.Issue}
+                space={template.space}
+                alwaysEdit
+                showButtons
+                maxHeight={'card'}
+                bind:content={description}
+                placeholder={tracker.string.IssueDescriptionPlaceholder}
+              />
+            </div>
+
+            <!-- svelte-ignore a11y-click-events-have-key-events -->
             <div
               class="tool"
               on:click={() => {
@@ -242,6 +242,7 @@
       <span class="title select-text">{title}</span>
       <div class="mt-6 description-preview select-text">
         {#if isDescriptionEmpty}
+          <!-- svelte-ignore a11y-click-events-have-key-events -->
           <div class="placeholder" on:click={edit}>
             <Label label={tracker.string.IssueDescriptionPlaceholder} />
           </div>
diff --git a/plugins/tracker/src/index.ts b/plugins/tracker/src/index.ts
index 916a202ec5..ce0fc96a09 100644
--- a/plugins/tracker/src/index.ts
+++ b/plugins/tracker/src/index.ts
@@ -14,18 +14,7 @@
 //
 
 import { Employee } from '@hcengineering/contact'
-import type {
-  AttachedDoc,
-  Class,
-  Doc,
-  Markup,
-  Obj,
-  Ref,
-  RelatedDocument,
-  Space,
-  Timestamp,
-  Type
-} from '@hcengineering/core'
+import type { AttachedDoc, Class, Doc, Markup, Ref, RelatedDocument, Space, Timestamp, Type } from '@hcengineering/core'
 import type { Asset, IntlString, Plugin, Resource } from '@hcengineering/platform'
 import { plugin } from '@hcengineering/platform'
 import type { TagCategory, TagElement } from '@hcengineering/tags'
@@ -197,7 +186,7 @@ export interface Issue extends AttachedDoc {
 /**
  * @public
  */
-export interface IssueDraft extends Obj {
+export interface IssueDraft extends Doc {
   issueId: Ref<Issue>
   title: string
   description: Markup
diff --git a/pods/account/src/index.ts b/pods/account/src/index.ts
index 235f130d17..3dd648407c 100644
--- a/pods/account/src/index.ts
+++ b/pods/account/src/index.ts
@@ -15,7 +15,7 @@
 //
 
 import { AccountMethod, ACCOUNT_DB } from '@hcengineering/account'
-import platform, { Request, Response, serialize, setMetadata, Severity, Status } from '@hcengineering/platform'
+import platform, { Response, serialize, setMetadata, Severity, Status } from '@hcengineering/platform'
 import serverToken from '@hcengineering/server-token'
 import toolPlugin from '@hcengineering/server-tool'
 import cors from '@koa/cors'
@@ -23,7 +23,7 @@ import { IncomingHttpHeaders } from 'http'
 import Koa from 'koa'
 import bodyParser from 'koa-bodyparser'
 import Router from 'koa-router'
-import { Db, MongoClient } from 'mongodb'
+import { MongoClient } from 'mongodb'
 
 /**
  * @public
@@ -71,9 +71,7 @@ export function serveAccount (methods: Record<string, AccountMethod>, productId
     const token = extractToken(ctx.request.headers)
 
     const request = ctx.request.body as any
-    const method = (
-      methods as { [key: string]: (db: Db, productId: string, request: Request<any>, token?: string) => Response<any> }
-    )[request.method]
+    const method = (methods as { [key: string]: AccountMethod })[request.method]
     if (method === undefined) {
       const response: Response<void> = {
         id: request.id,