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

@@ -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) {