fix: improve error handling around vwo-script anti-flicker (#5593)

Author: wdoconnell

src/App.tsx

@@ -83,7 +83,27 @@ const App: FC = () => {
     }
 
     if (CLOUD && isFlagEnabled('vwoAbTesting')) {
-      executeVWO()
+      const removeAntiFlickerStyle = () => {
+        // This id must correspond to the id of the <style> tag set in the VWO script.
+        const antiFlickerStyle = document.getElementById('_vis_opt_path_hides')
+        if (antiFlickerStyle) {
+          antiFlickerStyle.remove()
+        }
+      }
+
+      try {
+        setTimeout(() => {
+          removeAntiFlickerStyle()
+        }, 2000)
+
+        executeVWO()
+      } catch (err) {
+        removeAntiFlickerStyle()
+
+        reportErrorThroughHoneyBadger(err, {
+          name: 'VWO script failed to execute successfully',
+        })
+      }
     }
 
     setAutoFreeze(false)

src/utils/vwo.ts

@@ -1,4 +1,6 @@
 // this code is related to implementing the A/B testing tool VMO: https://vwo.com/
+// Whenever this script is updated, ensure that any changes are reflected in the
+// error handling in App.tsx, especially for any elements that change opacity of the page.
 /* eslint-disable */
 // @ts-nocheck
 export const executeVWO = () => {
@@ -10,7 +12,7 @@ export const executeVWO = () => {
         library_tolerance = 2500,
         use_existing_jquery = false,
         is_spa = 1,
-        hide_element = 'body',
+        hide_element = '',
         /* DO NOT EDIT BELOW THIS LINE */
         f = false,
         d = document,