Fix minor issues (#2426)

Signed-off-by: Andrey Sobolev <haiodo@gmail.com>

packages/panel/src/components/Panel.svelte

@@ -37,6 +37,7 @@
   export let floatAside = false
   export let allowClose = true
   export let useMaxWidth = false
+  export let isFullSize = false
 </script>
 
 <Panel
@@ -48,7 +49,8 @@
   on:close
   {allowClose}
   {floatAside}
-  {useMaxWidth}
+  bind:useMaxWidth
+  {isFullSize}
 >
   <svelte:fragment slot="navigator">
     {#if $$slots.navigator}

packages/theme/styles/_layouts.scss

@@ -231,6 +231,11 @@ input.search {
 .flex-gap-2 { gap: .5rem; }
 .flex-gap-1 { gap: .25rem; }
 
+.flex-break {  
+  flex-basis: 100%;
+  height: 0;
+}
+
 .flex-presenter, .inline-presenter {
   flex-wrap: nowrap;
   cursor: pointer;

packages/theme/styles/dialogs.scss

@@ -84,6 +84,7 @@
     flex-direction: column;
     margin: 0 1rem .75rem;
     color: var(--theme-caption-color);
+    flex-wrap: wrap;
 
     &__separator {
       margin: 1rem 0;

packages/ui/src/components/Panel.svelte

@@ -36,21 +36,41 @@
   const dispatch = createEventDispatcher()
 
   let asideFloat: boolean = false
-  let asideShown: boolean = false
+  let asideShown: boolean = true
-  let fullSize: boolean = false
   $: twoRows = $deviceInfo.minWidth
 
+  let oldWidth = ''
+  let hideTimer: number | undefined
+
   const checkPanel = (): void => {
+    const k = `${panelWidth}-${asideFloat}`
+    if (oldWidth === k) {
+      return
+    }
+    oldWidth = k
     if (floatAside) {
       asideFloat = true
-    } else if (panelWidth <= 900 && !asideFloat) asideFloat = true
+    } else if (panelWidth <= 900 && !asideFloat) {
-    else if (panelWidth > 900 && asideFloat) {
+      asideFloat = true
-      asideFloat = false
+      if (asideShown) {
         asideShown = false
       }
+    } else if (panelWidth > 900) {
+      if (asideFloat) {
+        asideFloat = false
+      }
+      if (!asideShown) {
+        asideShown = true
+      }
+    }
   }
   afterUpdate(() => {
+    if (hideTimer) {
+      clearTimeout(hideTimer)
+    }
+    hideTimer = setTimeout(() => {
       checkPanel()
+    }, 500)
   })
 </script>
 
@@ -80,7 +100,7 @@
       <div class="buttons-group xsmall-gap">
         <slot name="utils" />
         {#if isFullSize || (asideFloat && $$slots.aside && isAside)}<div class="buttons-divider" />{/if}
-        {#if asideFloat && $$slots.aside && isAside}
+        {#if $$slots.aside && isAside}
           <Button
             icon={IconDetails}
             kind={'transparent'}
@@ -93,12 +113,12 @@
         {/if}
         {#if isFullSize}
           <Button
-            icon={fullSize ? IconScale : IconScaleFull}
+            icon={useMaxWidth ? IconScale : IconScaleFull}
             kind={'transparent'}
             size={'medium'}
-            selected={fullSize}
+            selected={useMaxWidth}
             on:click={() => {
-              fullSize = !fullSize
+              useMaxWidth = !useMaxWidth
               dispatch('fullsize')
             }}
           />
@@ -141,7 +161,7 @@
         <slot />
       </div>
     {/if}
-    {#if $$slots.aside && isAside}
+    {#if $$slots.aside && isAside && asideShown}
       <div class="popupPanel-body__aside" class:float={asideFloat} class:shown={asideShown}>
         <slot name="aside" />
       </div>

packages/ui/src/components/PopupInstance.svelte

@@ -135,6 +135,7 @@
     on:close={(ev) => _close(ev?.detail)}
     on:fullsize={() => {
       fullSize = !fullSize
+      fitPopup()
     }}
     on:changeContent={() => {
       fitPopup()

plugins/document-resources/src/components/EditDoc.svelte

@@ -380,7 +380,9 @@
     isAside={true}
     isSub={false}
     bind:innerWidth
-    floatAside={true}
+    floatAside={false}
+    useMaxWidth={true}
+    isFullSize={true}
     on:close={() => dispatch('close')}
   >
     <svelte:fragment slot="navigator">

plugins/recruit-resources/src/components/CreateCandidate.svelte

@@ -512,6 +512,8 @@
       }}
     />
     {#if skills.length > 0}
+      <div class="flex-break" />
+      <div class="antiComponent antiEmphasized flex-grow mt-2">
         <Component
           is={tags.component.TagsEditor}
           props={{
@@ -536,6 +538,7 @@
             skills = skills
           }}
         />
+      </div>
     {/if}
   </svelte:fragment>
 

plugins/tags-resources/src/components/TagsEditor.svelte

@@ -105,8 +105,14 @@
     </div>
   {/if}
   <ShowMore ignore={!showTitle}>
-    <div class:tags-container={showTitle} class:mt-3={showTitle} class:empty={items.length === 0}>
+    <div
-      <div class="tag-items-container" class:tag-items-scroll={!showTitle}>
+      class:tags-container={showTitle}
+      class:antiComponent={showTitle}
+      class:antiEmphasized={showTitle}
+      class:mt-3={showTitle}
+      class:empty={items.length === 0}
+    >
+      <div class="flex flex-grow flex-col" class:tag-items-scroll={!showTitle}>
         {#if items.length === 0}
           {#if keyLabel}
             <div class="text-sm dark-color w-full flex-center">
@@ -120,7 +126,7 @@
               <Label label={cat.label} />
             </div>
           {/if}
-          <div class="tag-items">
+          <div class="flex flex-grow flex-wrap">
             {#each cat.items as tag}
               <TagItem
                 {tag}
@@ -154,26 +160,10 @@
 
 <style lang="scss">
   .tags-container {
-    padding: 0.5rem;
-    color: var(--accent-color);
-    background-color: var(--board-bg-color);
-    border: 1px solid var(--divider-color);
-    border-radius: 0.75rem;
-
     &.empty {
       background-color: transparent;
     }
   }
-  .tag-items-container {
-    flex-grow: 1;
-    display: flex;
-    flex-direction: column;
-  }
-  .tag-items {
-    flex-grow: 1;
-    display: flex;
-    flex-wrap: wrap;
-  }
   .tag-items-scroll {
     overflow-y: scroll;
     max-height: 20rem;

plugins/tracker-resources/src/components/CreateIssue.svelte

@@ -692,6 +692,7 @@
     <ParentIssue issue={parentIssue} on:close={clearParentIssue} />
   {/if}
   <EditBox bind:value={object.title} placeholder={tracker.string.IssueTitlePlaceholder} kind={'large-style'} focus />
+  {#key objectId}
     <AttachmentStyledBox
       bind:this={descriptionBox}
       {objectId}
@@ -704,6 +705,7 @@
       placeholder={tracker.string.IssueDescriptionPlaceholder}
       on:changeSize={() => dispatch('changeContent')}
     />
+  {/key}
   <IssueTemplateChilds bind:children={subIssues} sprint={object.sprint} project={object.project} isScrollable />
   <svelte:fragment slot="pool">
     {#if issueStatuses}

plugins/tracker-resources/src/components/issues/edit/CreateSubIssue.svelte

@@ -41,7 +41,7 @@
   let labels: TagReference[] = []
   let descriptionBox: AttachmentStyledBox
 
-  const objectId: Ref<Issue> = generateId()
+  let objectId: Ref<Issue> = generateId()
   const key: KeyedAttribute = {
     key: 'labels',
     attr: client.getHierarchy().getAttribute(tracker.class.Issue, 'labels')
@@ -73,6 +73,7 @@
     newIssue = getIssueDefaults()
     labels = []
     focusIssueTitle?.()
+    objectId = generateId()
   }
 
   function getTitle (value: string) {
@@ -191,6 +192,7 @@
         focus
       />
       <div class="mt-4">
+        {#key objectId}
           <AttachmentStyledBox
             bind:this={descriptionBox}
             {objectId}
@@ -204,6 +206,7 @@
             placeholder={tracker.string.IssueDescriptionPlaceholder}
             on:changeSize={() => dispatch('changeContent')}
           />
+        {/key}
       </div>
     </div>
   </div>

plugins/tracker-resources/src/components/workflow/Statuses.svelte

@@ -220,7 +220,7 @@
   $: updateStatusCategories()
 </script>
 
-<Panel isHeader={false} isAside={false} isFullSize on:fullsize on:close={() => dispatch('close')}>
+<Panel isHeader={false} isAside={false} on:fullsize on:close={() => dispatch('close')}>
   <svelte:fragment slot="title">
     <div class="antiTitle icon-wrapper">
       <div class="wrapped-icon">

plugins/view-resources/src/components/EditDoc.svelte

@@ -94,9 +94,7 @@
         m.kind === ClassifierKind.MIXIN &&
         !ignoreMixins.has(m._id) &&
         (hierarchy.hasMixin(object, m._id) ||
-          (showAllMixins &&
+          (showAllMixins && hierarchy.isDerived(realObjectClass, hierarchy.getBaseClass(m._id))))
-            // hierarchy.isDerived(hierarchy.getBaseClass(m._id), realObjectClass)
-            hierarchy.isDerived(realObjectClass, hierarchy.getBaseClass(m._id))))
     )
   }