Do not load toobar only if autocapture enabled (#285)

PostHog · Sep 14, 2021 · a7991ce · a7991ce
1 parent aaed37f
commit a7991ce
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 7 deletions.
diff --git a/src/__tests__/autocapture.js b/src/__tests__/autocapture.js
@@ -897,12 +897,6 @@ describe('Autocapture system', () => {
 
             jest.spyOn(autocapture, '_addDomEventHandlers')
         })
-
-        it('should check whether to load the editor', () => {
-            given.subject()
-
-            expect(given.lib.toolbar.maybeLoadEditor).toHaveBeenCalled()
-        })
     })
 
     describe('afterDecideResponse()', () => {

diff --git a/src/autocapture.js b/src/autocapture.js
@@ -231,7 +231,6 @@ var autocapture = {
 
     _customProperties: {},
     init: function (instance) {
-        instance.toolbar.maybeLoadEditor()
         this.rageclicks = new RageClick(instance)
     },
 

diff --git a/src/posthog-core.js b/src/posthog-core.js
@@ -142,6 +142,7 @@ var create_mplib = function (token, config, name) {
     instance.feature_flags = instance.featureFlags
 
     instance.toolbar = new Toolbar(instance)
+    instance.toolbar.maybeLoadEditor()
 
     instance.sessionRecording = new SessionRecording(instance)
     instance.sessionRecording.startRecordingIfEnabled()