Revert "Move dialog outputs inside navigation context (#8511)" (#8531)

src/App.native.tsx

@@ -64,6 +64,7 @@ import {Shell} from '#/view/shell'
 import {ThemeProvider as Alf} from '#/alf'
 import {useColorModeTheme} from '#/alf/util/useColorModeTheme'
 import {Provider as ContextMenuProvider} from '#/components/ContextMenu'
+import {NuxDialogs} from '#/components/dialogs/nuxs'
 import {useStarterPackEntry} from '#/components/hooks/useStarterPackEntry'
 import {Provider as IntentDialogProvider} from '#/components/intents/IntentDialogs'
 import {Provider as PortalProvider} from '#/components/Portal'
@@ -156,6 +157,7 @@ function InnerApp() {
                                                     <IntentDialogProvider>
                                                       <TestCtrls />
                                                       <Shell />
+                                                      <NuxDialogs />
                                                     </IntentDialogProvider>
                                                   </GestureHandlerRootView>
                                                 </HideBottomBarBorderProvider>

src/App.web.tsx

@@ -54,6 +54,7 @@ import {Shell} from '#/view/shell/index'
 import {ThemeProvider as Alf} from '#/alf'
 import {useColorModeTheme} from '#/alf/util/useColorModeTheme'
 import {Provider as ContextMenuProvider} from '#/components/ContextMenu'
+import {NuxDialogs} from '#/components/dialogs/nuxs'
 import {useStarterPackEntry} from '#/components/hooks/useStarterPackEntry'
 import {Provider as IntentDialogProvider} from '#/components/intents/IntentDialogs'
 import {Provider as PortalProvider} from '#/components/Portal'
@@ -134,6 +135,7 @@ function InnerApp() {
                                                   <HideBottomBarBorderProvider>
                                                     <IntentDialogProvider>
                                                       <Shell />
+                                                      <NuxDialogs />
                                                     </IntentDialogProvider>
                                                   </HideBottomBarBorderProvider>
                                                 </ServiceConfigProvider>

src/view/shell/createNativeStackNavigatorWithAuth.tsx

@@ -40,14 +40,6 @@ import {Onboarding} from '#/screens/Onboarding'
 import {SignupQueued} from '#/screens/SignupQueued'
 import {Takendown} from '#/screens/Takendown'
 import {atoms as a, useLayoutBreakpoints} from '#/alf'
-import {EmailDialog} from '#/components/dialogs/EmailDialog'
-import {InAppBrowserConsentDialog} from '#/components/dialogs/InAppBrowserConsent'
-import {LinkWarningDialog} from '#/components/dialogs/LinkWarning'
-import {MutedWordsDialog} from '#/components/dialogs/MutedWords'
-import {NuxDialogs} from '#/components/dialogs/nuxs'
-import {SigninDialog} from '#/components/dialogs/Signin'
-import {Outlet as PortalOutlet} from '#/components/Portal'
-import {BottomSheetOutlet} from '#/../modules/bottom-sheet'
 import {BottomBarWeb} from './bottom-bar/BottomBarWeb'
 import {DesktopLeftNav} from './desktop/LeftNav'
 import {DesktopRightNav} from './desktop/RightNav'
@@ -175,20 +167,6 @@ function NativeStackNavigator({
           {!isMobile && <DesktopRightNav routeName={activeRoute.name} />}
         </>
       )}
-
-      {/* Start: individual dialogs and outlets */}
-      <MutedWordsDialog />
-      <SigninDialog />
-      <EmailDialog />
-      <LinkWarningDialog />
-      {!isWeb && <InAppBrowserConsentDialog />}
-      <PortalOutlet />
-      <BottomSheetOutlet />
-      {/* End: individual dialogs and outlets */}
-
-      {/* Start: dialog controllers */}
-      <NuxDialogs />
-      {/* End: dialog controllers */}
     </NavigationContent>
   )
 }

src/view/shell/index.tsx

@@ -25,7 +25,14 @@ import {ModalsContainer} from '#/view/com/modals/Modal'
 import {ErrorBoundary} from '#/view/com/util/ErrorBoundary'
 import {atoms as a, select, useTheme} from '#/alf'
 import {setSystemUITheme} from '#/alf/util/systemUI'
+import {EmailDialog} from '#/components/dialogs/EmailDialog'
+import {InAppBrowserConsentDialog} from '#/components/dialogs/InAppBrowserConsent'
+import {LinkWarningDialog} from '#/components/dialogs/LinkWarning'
+import {MutedWordsDialog} from '#/components/dialogs/MutedWords'
+import {SigninDialog} from '#/components/dialogs/Signin'
+import {Outlet as PortalOutlet} from '#/components/Portal'
 import {RoutesContainer, TabsNavigator} from '#/Navigation'
+import {BottomSheetOutlet} from '../../../modules/bottom-sheet'
 import {updateActiveViewAsync} from '../../../modules/expo-bluesky-swiss-army/src/VisibilityView'
 import {Composer} from './Composer'
 import {DrawerContent} from './Drawer'
@@ -145,7 +152,14 @@ function ShellInner() {
       </View>
       <Composer winHeight={winDim.height} />
       <ModalsContainer />
+      <MutedWordsDialog />
+      <SigninDialog />
+      <EmailDialog />
+      <InAppBrowserConsentDialog />
+      <LinkWarningDialog />
       <Lightbox />
+      <PortalOutlet />
+      <BottomSheetOutlet />
     </>
   )
 }

src/view/shell/index.web.tsx

@@ -17,6 +17,11 @@ import {Lightbox} from '#/view/com/lightbox/Lightbox'
 import {ModalsContainer} from '#/view/com/modals/Modal'
 import {ErrorBoundary} from '#/view/com/util/ErrorBoundary'
 import {atoms as a, select, useTheme} from '#/alf'
+import {EmailDialog} from '#/components/dialogs/EmailDialog'
+import {LinkWarningDialog} from '#/components/dialogs/LinkWarning'
+import {MutedWordsDialog} from '#/components/dialogs/MutedWords'
+import {SigninDialog} from '#/components/dialogs/Signin'
+import {Outlet as PortalOutlet} from '#/components/Portal'
 import {FlatNavigator, RoutesContainer} from '#/Navigation'
 import {Composer} from './Composer.web'
 import {DrawerContent} from './Drawer'
@@ -62,7 +67,12 @@ function ShellInner() {
       </ErrorBoundary>
       <Composer winHeight={0} />
       <ModalsContainer />
+      <MutedWordsDialog />
+      <SigninDialog />
+      <EmailDialog />
+      <LinkWarningDialog />
       <Lightbox />
+      <PortalOutlet />
 
       {showDrawerDelayedExit && (
         <>