From 0843db0e048c575506ee83f980fbfde2b91b9ae6 Mon Sep 17 00:00:00 2001
From: Anna No <anna.no@xored.com>
Date: Thu, 21 Apr 2022 17:59:07 +0700
Subject: [PATCH] Board: Add attachment action (#1474)

Signed-off-by: Anna No <anna.no@xored.com>
---
 packages/theme/styles/_layouts.scss           |  1 +
 .../src/components/AddAttachment.svelte       |  4 ++
 plugins/board-assets/lang/en.json             |  3 ++
 plugins/board-assets/lang/ru.json             |  3 ++
 .../src/components/EditCard.svelte            |  6 +--
 .../components/editor/CardAttachments.svelte  | 20 ++++---
 .../src/components/editor/CardDetails.svelte  |  8 +--
 .../src/components/editor/CardLabels.svelte   |  4 +-
 .../components/popups/AttachmentPicker.svelte | 54 +++++++++++++++++++
 .../components/popups/CardLabelsEditor.svelte |  8 +--
 .../components/popups/CardLabelsPicker.svelte |  8 +--
 .../presenters/DatePresenter.svelte           |  2 +-
 .../presenters/MemberPresenter.svelte         |  5 +-
 plugins/board-resources/src/index.ts          | 14 +++--
 plugins/board-resources/src/plugin.ts         |  3 ++
 .../board-resources/src/utils/PopupUtils.ts   |  3 +-
 16 files changed, 113 insertions(+), 33 deletions(-)
 create mode 100644 plugins/board-resources/src/components/popups/AttachmentPicker.svelte

diff --git a/packages/theme/styles/_layouts.scss b/packages/theme/styles/_layouts.scss
index 4b8fc972f2c..2fc632919ca 100644
--- a/packages/theme/styles/_layouts.scss
+++ b/packages/theme/styles/_layouts.scss
@@ -420,6 +420,7 @@ p:last-child { margin-block-end: 0; }
 .w-9 { width: 2.25rem; }
 .w-14 { width: 3.5rem; }
 .w-16 { width: 4rem; }
+.w-60 { width: 15rem; }
 .w-85 { width: 21.25rem; }
 .w-165 { width: 41.25rem; }
 .min-w-0 { min-width: 0; }
diff --git a/plugins/attachment-resources/src/components/AddAttachment.svelte b/plugins/attachment-resources/src/components/AddAttachment.svelte
index 2fe19c89be3..e1394993013 100644
--- a/plugins/attachment-resources/src/components/AddAttachment.svelte
+++ b/plugins/attachment-resources/src/components/AddAttachment.svelte
@@ -16,6 +16,7 @@
   import { Class, Doc, Ref, Space } from '@anticrm/core'
   import { getClient } from '@anticrm/presentation'
   import { CircleButton, IconAdd } from '@anticrm/ui'
+  import { createEventDispatcher } from 'svelte'
   import { createAttachments } from '../utils'
 
   export let loading: number = 0
@@ -26,6 +27,7 @@
   export let space: Ref<Space>
 
   const client = getClient()
+  const dispatch = createEventDispatcher()
 
   async function fileSelected() {
     const list = inputFile.files
@@ -37,6 +39,8 @@
     } finally {
       loading--
     }
+
+    dispatch('attached')
   }
 
   function openFile() {
diff --git a/plugins/board-assets/lang/en.json b/plugins/board-assets/lang/en.json
index 34a9d7573c0..8052d7da257 100644
--- a/plugins/board-assets/lang/en.json
+++ b/plugins/board-assets/lang/en.json
@@ -41,6 +41,9 @@
     "Attachments": "Attachments",
     "AddAttachment": "Add an attachment",
     "DropFileToUpload": "Drop files to upload.",
+    "AttachFrom": "Attach from...",
+    "AttachmentTip": "Tip: You can drag and drop files onto cards to upload them.",
+    "Computer": "Computer",
     "CustomFields": "Custom Fields",
     "Automation": "Automation",
     "AddButton": "Add Button",
diff --git a/plugins/board-assets/lang/ru.json b/plugins/board-assets/lang/ru.json
index 2c54fc552e7..a32917bab4b 100644
--- a/plugins/board-assets/lang/ru.json
+++ b/plugins/board-assets/lang/ru.json
@@ -41,6 +41,9 @@
     "Attachments": "Прикрепленное",
     "AddAttachment": "Прикрепить",
     "DropFileToUpload": "Добавьте файлы.",
+    "AttachFrom": "Добавить из...",
+    "AttachmentTip": "Совет: Вы можете перетаскивать файлы на карточки, чтобы прикрепить их.",
+    "Computer": "Компьютер",
     "CustomFields": "Дополнительно",
     "Automation": "Автоматизация",
     "AddButton": "Добавить",
diff --git a/plugins/board-resources/src/components/EditCard.svelte b/plugins/board-resources/src/components/EditCard.svelte
index ec13ead43f2..7f3f20a4b99 100644
--- a/plugins/board-resources/src/components/EditCard.svelte
+++ b/plugins/board-resources/src/components/EditCard.svelte
@@ -41,7 +41,7 @@
 
   let object: Card | undefined
   let state: State | undefined
-  let handleMove: () => void
+  let handleMove: (e: Event) => void
 
   $: cardQuery.query(_class, { _id }, async (result) => {
     object = result[0]
@@ -55,9 +55,9 @@
   getCardActions(client, { _id: board.cardAction.Move }).then(async (result) => {
     if (result[0]?.handler) {
       const handler = await getResource(result[0].handler)
-      handleMove = () => {
+      handleMove = (e) => {
         if (object) {
-          handler(object, client)
+          handler(object, client, e)
         }
       }
     }
diff --git a/plugins/board-resources/src/components/editor/CardAttachments.svelte b/plugins/board-resources/src/components/editor/CardAttachments.svelte
index f7cdade4b7e..03e6cfded2f 100644
--- a/plugins/board-resources/src/components/editor/CardAttachments.svelte
+++ b/plugins/board-resources/src/components/editor/CardAttachments.svelte
@@ -15,8 +15,8 @@
 -->
 <script lang="ts">
   import attachment, { Attachment } from '@anticrm/attachment'
-  import { AddAttachment } from '@anticrm/attachment-resources'
   import type { Card } from '@anticrm/board'
+  import { getResource } from '@anticrm/platform'
   import { getClient } from '@anticrm/presentation'
   import { Button, Icon, IconAttachment, Label } from '@anticrm/ui'
   import AttachmentPresenter from '../presenters/AttachmentPresenter.svelte'
@@ -25,13 +25,21 @@
   export let value: Card
   const client = getClient()
   let attachments: Attachment[] = []
-
-  let inputFile: HTMLInputElement
+  let addAttachment: (e: Event) => void
 
   async function fetch () {
     attachments = await client.findAll(attachment.class.Attachment, { space: value.space, attachedTo: value._id })
   }
 
+  client.findOne(board.class.CardAction, { _id: board.cardAction.Attachments }).then(async (action) => {
+    if (action && action.handler) {
+      const handler = await getResource(action.handler)
+      addAttachment = (e) => {
+        handler(value, client, e)
+      }
+    }
+  })
+
   $: value?.attachments && value.attachments > 0 && fetch()
 </script>
 
@@ -52,11 +60,7 @@
           <AttachmentPresenter value={attach} />
         {/each}
         <div class="mt-2">
-          <AddAttachment bind:inputFile objectClass={value._class} objectId={value._id} space={value.space}>
-            <svelte:fragment slot="control" let:click>
-              <Button label={board.string.AddAttachment} kind="no-border" on:click={click} />
-            </svelte:fragment>
-          </AddAttachment>
+          <Button label={board.string.AddAttachment} kind="no-border" on:click={addAttachment} />
         </div>
       </div>
     </div>
diff --git a/plugins/board-resources/src/components/editor/CardDetails.svelte b/plugins/board-resources/src/components/editor/CardDetails.svelte
index 7ed67bb8485..516263e9b9f 100644
--- a/plugins/board-resources/src/components/editor/CardDetails.svelte
+++ b/plugins/board-resources/src/components/editor/CardDetails.svelte
@@ -31,8 +31,8 @@
   const query = createQuery()
   const client = getClient()
   let members: Employee[]
-  let membersHandler: () => void
-  let dateHandler: () => void
+  let membersHandler: (e: Event) => void
+  let dateHandler: (e: Event) => void
 
   $: membersIds = members?.map(m => m._id) ?? []
 
@@ -71,9 +71,9 @@
       if (action.handler) {
         const handler = await getResource(action.handler)
         if (action._id === board.cardAction.Dates) {
-          dateHandler = () => handler(value, client)
+          dateHandler = (e) => handler(value, client, e)
         } else if (action._id === board.cardAction.Members) {
-          membersHandler = () => handler(value, client)
+          membersHandler = (e) => handler(value, client, e)
         }
       }
     }
diff --git a/plugins/board-resources/src/components/editor/CardLabels.svelte b/plugins/board-resources/src/components/editor/CardLabels.svelte
index 2d2f7906e8e..bc50965a3f7 100644
--- a/plugins/board-resources/src/components/editor/CardLabels.svelte
+++ b/plugins/board-resources/src/components/editor/CardLabels.svelte
@@ -28,7 +28,7 @@
 
   const client = getClient()
   let labels: CardLabel[]
-  let labelsHandler: () => void
+  let labelsHandler: (e: Event) => void
   let isCompact: boolean = false
   let isHovered: boolean = false
 
@@ -46,7 +46,7 @@
     }).then(async (result) => {
       if (result?.[0]?.handler) {
         const handler = await getResource(result[0].handler)
-        labelsHandler = () => handler(value, client)
+        labelsHandler = (e: Event) => handler(value, client, e)
       }
     })
   }
diff --git a/plugins/board-resources/src/components/popups/AttachmentPicker.svelte b/plugins/board-resources/src/components/popups/AttachmentPicker.svelte
new file mode 100644
index 00000000000..7a37fd1843e
--- /dev/null
+++ b/plugins/board-resources/src/components/popups/AttachmentPicker.svelte
@@ -0,0 +1,54 @@
+<script lang="ts">
+    import { AddAttachment } from '@anticrm/attachment-resources'
+    import { Card } from '@anticrm/board'
+    import {
+      Button,
+      IconClose,
+      Label
+    } from '@anticrm/ui'
+    import { createEventDispatcher } from 'svelte'
+
+    import board from '../../plugin'
+
+    export let object: Card
+
+    let inputFile: HTMLInputElement
+    let loading: number = 0
+
+    const dispatch = createEventDispatcher()
+    function close () {
+      dispatch('close')
+    }
+    function onAttached () {
+      if (loading === 0) {
+        close()
+      }
+    }
+</script>
+
+  <div class="antiPopup w-60 pb-2">
+    <div class="relative flex-row-center w-full">
+        <div class="flex-center flex-grow fs-title mt-1 mb-1">
+        <Label label={board.string.AttachFrom} />
+    </div>
+      <div class="absolute mr-1 mt-1 mb-1" style:top="0" style:right="0">
+        <Button
+          icon={IconClose}
+          kind="transparent"
+          size="small"
+          on:click={close} />
+      </div>
+    </div>
+    <div class="ap-space bottom-divider" />
+    <div class="flex-col mt-2 w-full">
+        <AddAttachment bind:inputFile bind:loading objectClass={object._class} objectId={object._id} space={object.space} on:attached={onAttached} >
+            <svelte:fragment slot="control" let:click>
+        <Button label={board.string.Computer} kind="transparent" width="100%" justify="left" on:click={() => {
+                    click()
+                }} />
+            </svelte:fragment>
+          </AddAttachment>
+    </div>
+    <div class="ap-space bottom-divider mt-3" />
+    <div class="mt-1 ml-2 mr-2 text-md"><Label label={board.string.AttachmentTip} /></div>
+  </div>
\ No newline at end of file
diff --git a/plugins/board-resources/src/components/popups/CardLabelsEditor.svelte b/plugins/board-resources/src/components/popups/CardLabelsEditor.svelte
index 863ad8e2b1e..63394707759 100644
--- a/plugins/board-resources/src/components/popups/CardLabelsEditor.svelte
+++ b/plugins/board-resources/src/components/popups/CardLabelsEditor.svelte
@@ -82,15 +82,15 @@
 </script>
 
 <div class="antiPopup w-85">
-  <div class="relative fs-title flex-center h-9">
-    <div class="absolute flex-center ml-2 h-full" style:top="0" style:left="0">
+  <div class="relative flex-row-center w-full ">
+    <div class="absolute ml-1 mt-1 mb-1" style:top="0" style:left="0">
       <Button icon={IconBack} kind="transparent" size="small" on:click={onBack} />
     </div>
-    <div>
+    <div class="flex-center flex-grow fs-title mt-1 mb-1">
       <Label label={board.string.Labels} />
     </div>
 
-    <div class="absolute flex-center mr-2 h-full" style:top="0" style:right="0">
+    <div class="absolute mr-1 mt-1 mb-1" style:top="0" style:right="0">
       <Button
         icon={IconClose}
         kind="transparent"
diff --git a/plugins/board-resources/src/components/popups/CardLabelsPicker.svelte b/plugins/board-resources/src/components/popups/CardLabelsPicker.svelte
index 268674fa5ef..676e42db4cf 100644
--- a/plugins/board-resources/src/components/popups/CardLabelsPicker.svelte
+++ b/plugins/board-resources/src/components/popups/CardLabelsPicker.svelte
@@ -78,9 +78,11 @@
 </script>
 
 <div class="antiPopup w-85 pb-2">
-  <div class="relative fs-title flex-center h-9">
-    <Label label={board.string.Labels} />
-    <div class="absolute flex-center mr-2 h-full" style:top="0" style:right="0">
+  <div class="relative flex-row-center w-full">
+    <div class="flex-center flex-grow fs-title mt-1 mb-1">
+      <Label label={board.string.Labels} />
+    </div>
+    <div class="absolute mr-1 mt-1 mb-1" style:top="0" style:right="0">
       <Button
         icon={IconClose}
         kind="transparent"
diff --git a/plugins/board-resources/src/components/presenters/DatePresenter.svelte b/plugins/board-resources/src/components/presenters/DatePresenter.svelte
index 478ebb7d7fd..afd023e38d1 100644
--- a/plugins/board-resources/src/components/presenters/DatePresenter.svelte
+++ b/plugins/board-resources/src/components/presenters/DatePresenter.svelte
@@ -5,7 +5,7 @@
 
   export let value: CardDate
   export let isInline: boolean = false
-  export let size: 'x-small' | 'small' = 'small' 
+  export let size: 'x-small' | 'small' = 'small'
 
   let isChecked = value?.isChecked
   const dispatch = createEventDispatcher()
diff --git a/plugins/board-resources/src/components/presenters/MemberPresenter.svelte b/plugins/board-resources/src/components/presenters/MemberPresenter.svelte
index f1bd3bf40e7..97e32b89b90 100644
--- a/plugins/board-resources/src/components/presenters/MemberPresenter.svelte
+++ b/plugins/board-resources/src/components/presenters/MemberPresenter.svelte
@@ -4,15 +4,16 @@
   import { Button, showPopup } from '@anticrm/ui'
   import type { IntlString } from '@anticrm/platform'
   import EditMember from '../popups/EditMember.svelte'
+  import { getPopupAlignment } from '../../utils/PopupUtils'
 
   export let value: Employee
   export let size: 'large' | 'medium'
   export let menuItems: { title: IntlString; handler: () => void }[][]
 
-  const openPopup = () => {
+  const openPopup = (e: Event) => {
     const onClose = () => closePopup()
 
-    const closePopup = showPopup(EditMember, { member: value, menuItems, onClose })
+    const closePopup = showPopup(EditMember, { member: value, menuItems, onClose }, getPopupAlignment(e))
   }
 
   $: firstName = getFirstName(value.name)
diff --git a/plugins/board-resources/src/index.ts b/plugins/board-resources/src/index.ts
index ec7b7717a44..5648386c19c 100644
--- a/plugins/board-resources/src/index.ts
+++ b/plugins/board-resources/src/index.ts
@@ -22,7 +22,6 @@ import { showPopup } from '@anticrm/ui'
 import { Card } from '@anticrm/board'
 import type { TxOperations as Client } from '@anticrm/core'
 import { Resources } from '@anticrm/platform'
-import { TxOperations } from '@anticrm/core'
 import CardPresenter from './components/CardPresenter.svelte'
 import BoardPresenter from './components/BoardPresenter.svelte'
 import CreateBoard from './components/CreateBoard.svelte'
@@ -31,10 +30,10 @@ import EditCard from './components/EditCard.svelte'
 import KanbanCard from './components/KanbanCard.svelte'
 import TemplatesIcon from './components/TemplatesIcon.svelte'
 import KanbanView from './components/KanbanView.svelte'
+import AttachmentPicker from './components/popups/AttachmentPicker.svelte'
 import CardLabelsPopup from './components/popups/CardLabelsPopup.svelte'
 import MoveView from './components/popups/MoveCard.svelte'
 import DateRangePicker from './components/popups/DateRangePicker.svelte'
-import EditMembersView from './components/popups/EditMembers.svelte'
 import CardLabelPresenter from './components/presenters/LabelPresenter.svelte'
 import CardDatePresenter from './components/presenters/DatePresenter.svelte'
 import WatchCard from './components/WatchCard.svelte'
@@ -61,7 +60,7 @@ async function showCardLabelsPopup (object: Card, client: Client, e?: Event): Pr
   showPopup(CardLabelsPopup, { object }, getPopupAlignment(e))
 }
 
-async function showEditMembersPopup(object: Card, client: TxOperations): Promise<void> {
+async function showEditMembersPopup (object: Card, client: Client, e?: Event): Promise<void> {
   showPopup(
     UsersPopup,
     {
@@ -71,14 +70,18 @@ async function showEditMembersPopup(object: Card, client: TxOperations): Promise
       selectedUsers: object?.members ?? [],
       placeholder: board.string.SearchMembers
     },
-    undefined,
+    getPopupAlignment(e),
     () => {},
-    (result: Ref<Employee>[]) => {
+    (result: Array<Ref<Employee>>) => {
       client.update(object, { members: result })
     }
   )
 }
 
+async function showAttachmentsPopup (object: Card, client: Client, e?: Event): Promise<void> {
+  showPopup(AttachmentPicker, { object }, getPopupAlignment(e))
+}
+
 export default async (): Promise<Resources> => ({
   component: {
     CreateBoard,
@@ -98,6 +101,7 @@ export default async (): Promise<Resources> => ({
     Move: showMoveCardPopup,
     Dates: showDatePickerPopup,
     Labels: showCardLabelsPopup,
+    Attachments: showAttachmentsPopup,
     Archive: archiveCard,
     SendToBoard: unarchiveCard,
     Delete: deleteCard,
diff --git a/plugins/board-resources/src/plugin.ts b/plugins/board-resources/src/plugin.ts
index 326f9701021..12fd0eb1678 100644
--- a/plugins/board-resources/src/plugin.ts
+++ b/plugins/board-resources/src/plugin.ts
@@ -62,6 +62,9 @@ export default mergeIds(boardId, board, {
     Attachments: '' as IntlString,
     AddAttachment: '' as IntlString,
     DropFileToUpload: '' as IntlString,
+    AttachFrom: '' as IntlString,
+    AttachmentTip: '' as IntlString,
+    Computer: '' as IntlString,
     CustomFields: '' as IntlString,
     Automation: '' as IntlString,
     AddButton: '' as IntlString,
diff --git a/plugins/board-resources/src/utils/PopupUtils.ts b/plugins/board-resources/src/utils/PopupUtils.ts
index f3c6e60807e..e3fdeb950de 100644
--- a/plugins/board-resources/src/utils/PopupUtils.ts
+++ b/plugins/board-resources/src/utils/PopupUtils.ts
@@ -6,8 +6,9 @@ export function getPopupAlignment (e?: Event): PopupAlignment | undefined {
   }
   const target = e.target as HTMLElement
   if (target.getBoundingClientRect) {
+    const result = target.getBoundingClientRect()
     return {
-      getBoundingClientRect: () => target.getBoundingClientRect()
+      getBoundingClientRect: () => result
     }
   }