fix(annotations): Fix AnnotationControlsFSM reset when fullscreen ent…

…er (#1287)

* fix(annotations): Fix AnnotationControlsFSM reset when fullscreen enter

* fix(annotations): Address comments

src/lib/viewers/BaseViewer.js

@@ -575,7 +575,6 @@ class BaseViewer extends EventEmitter {
 
         if (this.annotator && this.areNewAnnotationsEnabled()) {
             this.annotator.emit(ANNOTATOR_EVENT.setVisibility, false);
-            this.annotator.toggleAnnotationMode(AnnotationMode.NONE);
             this.disableAnnotationControls();
         }
     }
@@ -919,9 +918,9 @@ class BaseViewer extends EventEmitter {
     disableAnnotationControls() {
         if (this.annotator && this.annotationControls && this.areNewAnnotationsEnabled()) {
             this.annotator.toggleAnnotationMode(AnnotationMode.NONE);
-            this.annotationControls.resetControls();
             this.annotationControls.toggle(false);
         }
+        this.processAnnotationModeChange(this.annotationControlsFSM.transition(AnnotationInput.RESET));
     }
 
     enableAnnotationControls() {

src/lib/viewers/__tests__/BaseViewer-test.js

@@ -527,6 +527,7 @@ describe('lib/viewers/BaseViewer', () => {
         test('should hide annotations and toggle annotations mode', () => {
             jest.spyOn(base, 'areNewAnnotationsEnabled').mockReturnValue(true);
             jest.spyOn(base, 'disableAnnotationControls');
+            jest.spyOn(base, 'processAnnotationModeChange');
 
             base.annotator = {
                 emit: jest.fn(),
@@ -535,6 +536,7 @@ describe('lib/viewers/BaseViewer', () => {
             base.annotationControls = {
                 destroy: jest.fn(),
                 resetControls: jest.fn(),
+                setMode: jest.fn(),
                 toggle: jest.fn(),
             };
 
@@ -543,6 +545,7 @@ describe('lib/viewers/BaseViewer', () => {
             expect(base.annotator.emit).toBeCalledWith(ANNOTATOR_EVENT.setVisibility, false);
             expect(base.annotator.toggleAnnotationMode).toBeCalledWith(AnnotationMode.NONE);
             expect(base.disableAnnotationControls).toBeCalled();
+            expect(base.processAnnotationModeChange).toBeCalledWith(AnnotationMode.NONE);
         });
     });
 
@@ -1042,20 +1045,22 @@ describe('lib/viewers/BaseViewer', () => {
     describe('disableAnnotationControls()', () => {
         test('should hide annotations and toggle annotations mode', () => {
             jest.spyOn(base, 'areNewAnnotationsEnabled').mockReturnValue(true);
+            jest.spyOn(base, 'processAnnotationModeChange');
 
             base.annotator = {
                 toggleAnnotationMode: jest.fn(),
             };
             base.annotationControls = {
                 destroy: jest.fn(),
                 resetControls: jest.fn(),
+                setMode: jest.fn(),
                 toggle: jest.fn(),
             };
 
             base.disableAnnotationControls();
 
-            expect(base.annotationControls.resetControls).toBeCalled();
             expect(base.annotationControls.toggle).toBeCalledWith(false);
+            expect(base.processAnnotationModeChange).toBeCalledWith(AnnotationMode.NONE);
         });
     });
 

src/lib/viewers/doc/DocBaseViewer.js

@@ -1283,7 +1283,6 @@ class DocBaseViewer extends BaseViewer {
 
         if (this.annotator && this.areNewAnnotationsEnabled() && this.options.enableAnnotationsDiscoverability) {
             this.annotator.toggleAnnotationMode(AnnotationMode.REGION);
-            this.processAnnotationModeChange(this.annotationControlsFSM.transition(AnnotationInput.RESET));
         }
     }
 

src/lib/viewers/doc/__tests__/DocBaseViewer-test.js

@@ -1977,7 +1977,6 @@ describe('src/lib/viewers/doc/DocBaseViewer', () => {
 
                 expect(docBase.annotator.emit).toBeCalledWith(ANNOTATOR_EVENT.setVisibility, true);
                 expect(docBase.annotator.toggleAnnotationMode).toBeCalledWith(AnnotationMode.REGION);
-                expect(docBase.processAnnotationModeChange).toBeCalledWith(AnnotationMode.NONE);
                 expect(docBase.enableAnnotationControls).toBeCalled();
             });
         });