fix: LEAP-640: Don't save empty draft on View All

web/libs/editor/src/components/NewTaxonomy/NewTaxonomy.tsx

@@ -191,7 +191,7 @@ const NewTaxonomy = ({
       }}
       treeCheckStrictly
       showCheckedStrategy={TreeSelect.SHOW_ALL}
-      treeExpandAction={false}
+      treeExpandAction="click"
       dropdownMatchSelectWidth={dropdownWidth}
       placeholder={options.placeholder || 'Click to add...'}
       style={style}

web/libs/editor/src/components/TopBar/Actions.js

@@ -15,7 +15,6 @@ export const Actions = ({ store }) => {
   const isViewAll = annotationStore.viewingAll;
 
   const onToggleVisibility = useCallback(() => {
-    !isViewAll && entity.saveDraftImmediatelyWithResults({ useToast: true });
     annotationStore.toggleViewingAllAnnotations();
   }, [annotationStore]);
 

web/libs/editor/src/components/TopBar/TopBar.js

@@ -1,15 +1,17 @@
 import { observer } from 'mobx-react';
+
+import { IconViewAll, LsPlus } from '../../assets/icons';
+import { Button } from '../../common/Button/Button';
+import { Tooltip } from '../../common/Tooltip/Tooltip';
 import { Block, Elem } from '../../utils/bem';
+import { FF_DEV_3873, isFF } from '../../utils/feature-flags';
+import { AnnotationsCarousel } from '../AnnotationsCarousel/AnnotationsCarousel';
 import { DynamicPreannotationsToggle } from '../AnnotationTab/DynamicPreannotationsToggle';
 import { Actions } from './Actions';
 import { Annotations } from './Annotations';
 import { Controls } from './Controls';
 import { CurrentTask } from './CurrentTask';
-import { FF_DEV_3873, isFF } from '../../utils/feature-flags';
-import { Button } from '../../common/Button/Button';
-import { Tooltip } from '../../common/Tooltip/Tooltip';
-import { IconViewAll, LsPlus } from '../../assets/icons';
-import { AnnotationsCarousel } from '../AnnotationsCarousel/AnnotationsCarousel';
+
 import './TopBar.styl';
 
 export const TopBar = observer(({ store }) => {
@@ -19,11 +21,6 @@ export const TopBar = observer(({ store }) => {
 
   const isViewAll = annotationStore?.viewingAll === true;
 
-  const toggleViewAll = () => {
-    !isViewAll && entity.saveDraftImmediatelyWithResults({ useToast: true });
-    annotationStore.toggleViewingAllAnnotations();
-  };
-
   return store ? (
     <Block name="topbar" mod={{ newLabelingUI: isFF(FF_DEV_3873) }}>
       {isFF(FF_DEV_3873) ? (
@@ -36,7 +33,7 @@ export const TopBar = observer(({ store }) => {
                 icon={<IconViewAll />}
                 type="text"
                 aria-label="View All"
-                onClick={toggleViewAll}
+                onClick={annotationStore.toggleViewingAllAnnotations}
                 primary={ isViewAll }
                 style={{
                   height: 36,

web/libs/editor/src/stores/Annotation/store.js

@@ -50,6 +50,19 @@ const AnnotationStoreModel = types
     function toggleViewingAll() {
       if (self.viewingAllAnnotations || self.viewingAllPredictions) {
         if (self.selected) {
+          // const comments = self.store.commentStore;
+
+          // @todo `currentComment` is an object and saving them was not a part of original fix
+          // @todo so I leave it for next fix coming soon
+          // if (comments.currentComment) {
+          //   // comment will save draft automatically
+          //   comments.commentFormSubmit();
+          // } else
+          if (self.selected.type === 'annotation') {
+            // save draft if there are changes waiting to be saved — it's handled inside
+            self.selected.saveDraftImmediately();
+          }
+
           self.selected.unselectAll();
           self.selected.selected = false;
         }