From 9d101263a0ec7bb41fd9daa9e656cedbca064a97 Mon Sep 17 00:00:00 2001
From: John Knox <jknox@fb.com>
Date: Thu, 12 Nov 2020 10:01:28 -0800
Subject: [PATCH] Put PopoverProvider inside scope of Store Provider

Summary:
Many UI components (such as Button) rely on the redux store being present in the context.

So to use these components freely in popovers, the popover provider has to be within the store provider, not outside of it.

This moves it, along with the context menu and tooltip providers for the same reason.

Has the effect of fixing the ITSR dialog, which had a button inside a popover.

Reviewed By: mweststrate

Differential Revision: D24922493

fbshipit-source-id: e15540ac49795875b1cf86d6c66ec0591dc374ef
---
 desktop/app/src/init.tsx | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/desktop/app/src/init.tsx b/desktop/app/src/init.tsx
index a17622421..9a883d797 100644
--- a/desktop/app/src/init.tsx
+++ b/desktop/app/src/init.tsx
@@ -52,19 +52,19 @@ enableMapSet();
 GK.init();
 
 const AppFrame = () => (
-  <TooltipProvider>
-    <PopoverProvider>
-      <ContextMenuProvider>
-        <Provider store={store}>
-          <CacheProvider value={cache}>
+  <Provider store={store}>
+    <CacheProvider value={cache}>
+      <TooltipProvider>
+        <PopoverProvider>
+          <ContextMenuProvider>
             <NuxManagerContext.Provider value={createNuxManager()}>
               <App logger={logger} />
             </NuxManagerContext.Provider>
-          </CacheProvider>
-        </Provider>
-      </ContextMenuProvider>
-    </PopoverProvider>
-  </TooltipProvider>
+          </ContextMenuProvider>
+        </PopoverProvider>
+      </TooltipProvider>
+    </CacheProvider>
+  </Provider>
 );
 
 function setProcessState(store: Store) {