From 71a04f0bc28be88f15045e208baa3fff5f43d53a Mon Sep 17 00:00:00 2001
From: Alexander Platov <sas_lord@mail.ru>
Date: Fri, 26 Nov 2021 14:08:46 +0300
Subject: [PATCH] Remove bottom border in tables. Clean warinings. (#379)

Signed-off-by: Alexander Platov <sas_lord@mail.ru>
---
 .../src/components/ChannelsPopup.svelte       |  2 +-
 .../src/components/Attachments.svelte         | 41 ----------
 .../src/components/EditApplication.svelte     | 80 -------------------
 .../src/components/SocialEditor.svelte        |  2 +-
 .../src/components/BooleanEditor.svelte       | 30 +++----
 .../src/components/KanbanPanel.svelte         | 10 +--
 .../src/components/StringEditor.svelte        |  2 +-
 .../src/components/Table.svelte               |  1 +
 .../src/components/TableView.svelte           |  1 +
 9 files changed, 26 insertions(+), 143 deletions(-)

diff --git a/packages/presentation/src/components/ChannelsPopup.svelte b/packages/presentation/src/components/ChannelsPopup.svelte
index 62465c5e8f..ecff2913ed 100644
--- a/packages/presentation/src/components/ChannelsPopup.svelte
+++ b/packages/presentation/src/components/ChannelsPopup.svelte
@@ -43,7 +43,7 @@
 <div class="flex-row-center">
   <CircleButton icon={value.icon} size={'x-large'} />
   <div class="flex-col caption-color ml-3">
-    <div class="small-text">{value.label}</div>
+    <div class="label">{value.label}</div>
     <div class="overflow-label">{value.value}</div>
   </div>
   <div class="button" on:click|preventDefault={copyLink}>
diff --git a/plugins/recruit-resources/src/components/Attachments.svelte b/plugins/recruit-resources/src/components/Attachments.svelte
index 5f9387b2c1..bd1eeb5543 100644
--- a/plugins/recruit-resources/src/components/Attachments.svelte
+++ b/plugins/recruit-resources/src/components/Attachments.svelte
@@ -130,47 +130,6 @@
     }
   }
 
-  .table-body {
-    margin-top: .75rem;
-
-    th, td {
-      padding: .75rem 0;
-      text-align: left;
-    }
-    th {
-      font-weight: 500;
-      font-size: .75rem;
-      color: var(--theme-content-dark-color);
-    }
-    td {
-      color: var(--theme-caption-color);
-    }
-    .tr-body { border-top: 1px solid var(--theme-button-border-hovered); }
-  }
-
-  .item {
-    display: flex;
-    align-items: center;
-    padding: .75rem 1rem;
-  }
-  .file-icon {
-    margin-right: 1.25rem;
-    width: 2rem;
-    height: 2rem;
-    font-weight: 500;
-    font-size: 0.625rem;
-    line-height: 150%;
-    text-transform: uppercase;
-    color: #fff;
-    background-color: var(--primary-button-enabled);
-    border: 1px solid rgba(0, 0, 0, .1);
-    border-radius: .5rem;
-  }
-  .file-desc {
-    font-size: 0.75rem;
-    color: var(--theme-content-dark-color);
-  }
-
   .resume {
     cursor: pointer;
     padding: 1rem;
diff --git a/plugins/recruit-resources/src/components/EditApplication.svelte b/plugins/recruit-resources/src/components/EditApplication.svelte
index 5003d84f73..72f7f9811b 100644
--- a/plugins/recruit-resources/src/components/EditApplication.svelte
+++ b/plugins/recruit-resources/src/components/EditApplication.svelte
@@ -73,18 +73,6 @@
     </div>
   </svelte:fragment> -->
 
-  <!-- <div class="flex-row-center">
-    <div class="avatar">
-      <div class="border"/>
-      <Avatar />
-    </div>
-    <div class="flex-col">
-      <div class="name">{formatName(candidate.name)}</div>
-      <div class="title">For {getVacancyName()}</div>
-      <div class="city">at Cisco</div>
-    </div>
-  </div> -->
-
   <div class="grid-cards">
     <CandidateCard {candidate}/>
     <VacancyCard {vacancy}/>
@@ -98,57 +86,6 @@
 {/if}
 
 <style lang="scss">
-  @import '../../../../packages/theme/styles/mixins.scss';
-
-  .avatar {
-    flex-shrink: 0;
-    overflow: hidden;
-    position: relative;
-    display: flex;
-    justify-content: center;
-    align-items: center;
-    margin-right: 1.5rem;
-    width: 6rem;
-    height: 6rem;
-    border-radius: 50%;
-    filter: drop-shadow(0px 14px 44px rgba(28, 23, 22, .8));
-    cursor: pointer;
-
-    &::after {
-      content: '';
-      @include bg-layer(var(--theme-avatar-hover), .5);
-      z-index: -1;
-    }
-    &::before {
-      content: '';
-      @include bg-layer(var(--theme-avatar-bg), .1);
-      backdrop-filter: blur(25px);
-      z-index: -2;
-    }
-    .border {
-      @include bg-fullsize;
-      border: 2px solid var(--theme-avatar-border);
-      border-radius: 50%;
-    }
-  }
-
-  .name {
-    font-weight: 500;
-    font-size: 1.25rem;
-    color: var(--theme-caption-color);
-  }
-  .title, .city {
-    font-weight: 500;
-    font-size: .75rem;
-    color: var(--theme-content-color);
-  }
-  .title { margin-top: .75rem; }
-  .resume a {
-    font-size: .75rem;
-    color: var(--theme-content-dark-color);
-    &:hover { color: var(--theme-content-color); }
-  }
-
   .attachments {
     margin-top: 3.5rem;
   }
@@ -158,21 +95,4 @@
     grid-template-columns: 1fr 1fr;
     column-gap: 1.5rem;
   }
-
-  // .container {
-  //   display: flex;
-  //   flex-direction: column;
-  //   height: 100%;
-  //   background-color: var(--theme-bg-color);
-  //   border-radius: 1.25rem;
-  //   box-shadow: 0px 3.125rem 7.5rem rgba(0, 0, 0, .4);
-
-  //   .tabs-container {
-  //     flex-grow: 1;
-  //     display: flex;
-  //     flex-direction: column;
-  //     height: fit-content;
-  //     padding: 0 2.5rem 2.5rem;
-  //   }
-  // }
 </style>
diff --git a/plugins/recruit-resources/src/components/SocialEditor.svelte b/plugins/recruit-resources/src/components/SocialEditor.svelte
index e49c9df605..adc2cbbaed 100644
--- a/plugins/recruit-resources/src/components/SocialEditor.svelte
+++ b/plugins/recruit-resources/src/components/SocialEditor.svelte
@@ -106,6 +106,6 @@
     align-items: center;
     margin-top: 1rem;
     .btn { flex-grow: 1; }
-    .btn + .btn { margin-left: .75rem; }
+    // .btn + .btn { margin-left: .75rem; }
   }
 </style>
diff --git a/plugins/view-resources/src/components/BooleanEditor.svelte b/plugins/view-resources/src/components/BooleanEditor.svelte
index bc23dedb62..c07be7be13 100644
--- a/plugins/view-resources/src/components/BooleanEditor.svelte
+++ b/plugins/view-resources/src/components/BooleanEditor.svelte
@@ -15,15 +15,11 @@
 -->
 
 <script lang="ts">
-
-  import type { IntlString } from '@anticrm/platform'
+  // import type { IntlString } from '@anticrm/platform'
   import { Label } from '@anticrm/ui'
   
-  export let label: IntlString
-  export let placeholder: IntlString
   export let value: any
-  export let focus: boolean
-  export let maxWidth: string
+  export let maxWidth: string | undefined = undefined
   export let onChange: (value: any) => void
   
   function getLabel(value: boolean | undefined) {
@@ -31,15 +27,21 @@
     if (value === false) return 'No'
     return 'N/A'
   }
-  
-  </script>
+</script>
 
-<div class="flex-row-center yesno-container" class:yes={value === true} class:no={value === false} class:unknown={value === undefined} on:click={() => {
-  if (value === true) value = false
-  else if (value === false) value = undefined
-  else value = true
-  onChange(value)
-}}>
+<div
+  class="flex-row-center yesno-container"
+  class:yes={value === true}
+  class:no={value === false}
+  class:unknown={value === undefined}
+  style={(maxWidth) ? `max-width: ${maxWidth};` : ''}
+  on:click={() => {
+    if (value === true) value = false
+    else if (value === false) value = undefined
+    else value = true
+    onChange(value)
+  }}
+>
   <svg class="svg-small" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 16 16">
     <circle class:yes={value === true} class:no={value === false} cx="8" cy="8" r="6"/>
     {#if value === true}
diff --git a/plugins/view-resources/src/components/KanbanPanel.svelte b/plugins/view-resources/src/components/KanbanPanel.svelte
index a59880b30f..f997f73a66 100644
--- a/plugins/view-resources/src/components/KanbanPanel.svelte
+++ b/plugins/view-resources/src/components/KanbanPanel.svelte
@@ -19,7 +19,7 @@
 
   export let label: IntlString
   export let color: string
-  export let counter: number
+  // export let counter: number
   export let addAction: () => void | undefined
 </script>
 
@@ -71,10 +71,10 @@
         height: 100%;
         font-weight: 500;
         color: var(--theme-caption-color);
-        span {
-          font-weight: 400;
-          color: var(--theme-content-dark-color);
-        }
+        // span {
+        //   font-weight: 400;
+        //   color: var(--theme-content-dark-color);
+        // }
       }
       .tool {
         opacity: .4;
diff --git a/plugins/view-resources/src/components/StringEditor.svelte b/plugins/view-resources/src/components/StringEditor.svelte
index 19ab0b8709..73613a5d5e 100644
--- a/plugins/view-resources/src/components/StringEditor.svelte
+++ b/plugins/view-resources/src/components/StringEditor.svelte
@@ -19,7 +19,7 @@
 import type { IntlString } from '@anticrm/platform'
 import { EditBox } from '@anticrm/ui'
 
-export let label: IntlString
+// export let label: IntlString
 export let placeholder: IntlString
 export let value: any
 export let focus: boolean
diff --git a/plugins/view-resources/src/components/Table.svelte b/plugins/view-resources/src/components/Table.svelte
index 524b9bdddb..2335203651 100644
--- a/plugins/view-resources/src/components/Table.svelte
+++ b/plugins/view-resources/src/components/Table.svelte
@@ -158,6 +158,7 @@
     color: var(--theme-caption-color);
     border-bottom: 1px solid var(--theme-button-border-hovered);
     &:hover .firstCell .menuRow { visibility: visible; }
+    &:last-child { border-bottom: none; }
   }
   .fixed .menuRow { visibility: visible; }
 </style>
diff --git a/plugins/view-resources/src/components/TableView.svelte b/plugins/view-resources/src/components/TableView.svelte
index 688217c1de..dc903557a7 100644
--- a/plugins/view-resources/src/components/TableView.svelte
+++ b/plugins/view-resources/src/components/TableView.svelte
@@ -200,6 +200,7 @@
       .checkCell { visibility: visible; }
     }
     &:hover .firstCell .menuRow { visibility: visible; }
+    &:last-child { border-bottom: none; }
   }
 
   .fixed {