Fix lifecycle of environment objects (#790)

#779 was happening because environment objects were set on contentView,
which is not the _outmost_ root view. This clarifies why the Theme
object was not being found in ThemeLockScreenModifier.

Also, do not hardcode LogoView as lock view.

Passepartout/App/PassepartoutApp.swift

@@ -86,6 +86,5 @@ extension PassepartoutApp {
                 break
             }
         }
-        .withEnvironment(from: context, theme: theme)
     }
 }

Passepartout/App/Platforms/App+iOS.swift

@@ -42,7 +42,8 @@ extension PassepartoutApp {
                 .onOpenURL { url in
                     ImporterPipe.shared.send([url])
                 }
-                .themeLockScreen(theme)
+                .themeLockScreen()
+                .withEnvironment(from: context, theme: theme)
         }
     }
 }

Passepartout/App/Platforms/App+macOS.swift

@@ -73,10 +73,12 @@ extension PassepartoutApp {
     var body: some Scene {
         Window(appName, id: appName, content: contentView)
             .defaultSize(width: 600, height: 400)
+            .withEnvironment(from: context, theme: theme)
 
         Settings {
             SettingsView(profileManager: context.profileManager)
                 .frame(minWidth: 300, minHeight: 200)
+                .withEnvironment(from: context, theme: theme)
         }
         MenuBarExtra {
             AppMenu()

Passepartout/App/Platforms/App+tvOS.swift

@@ -37,7 +37,10 @@ extension AppDelegate: UIApplicationDelegate {
 
 extension PassepartoutApp {
     var body: some Scene {
-        WindowGroup(content: contentView)
+        WindowGroup {
+            contentView()
+                .withEnvironment(from: context, theme: theme)
+        }
     }
 }
 

Passepartout/Library/Sources/AppUI/Theme/Theme+UI.swift

@@ -68,7 +68,7 @@ struct ThemeBooleanModalModifier<Modal>: ViewModifier where Modal: View {
                 modal()
                     .frame(minWidth: modalSize?.width, minHeight: modalSize?.height)
                     .interactiveDismissDisabled(!isInteractive)
-                    .themeLockScreen(theme)
+                    .themeLockScreen()
             }
     }
 
@@ -97,7 +97,7 @@ struct ThemeItemModalModifier<Modal, T>: ViewModifier where Modal: View, T: Iden
                 modal($0)
                     .frame(minWidth: modalSize?.width, minHeight: modalSize?.height)
                     .interactiveDismissDisabled(!isInteractive)
-                    .themeLockScreen(theme)
+                    .themeLockScreen()
             }
     }
 
@@ -129,13 +129,13 @@ struct ThemeBooleanPopoverModifier<Popover>: ViewModifier, SizeClassProviding wh
                 .popover(isPresented: $isPresented) {
                     popover
                         .frame(minWidth: theme.popoverSize?.width, minHeight: theme.popoverSize?.height)
-                        .themeLockScreen(theme)
+                        .themeLockScreen()
                 }
         } else {
             content
                 .sheet(isPresented: $isPresented) {
                     popover
-                        .themeLockScreen(theme)
+                        .themeLockScreen()
                 }
         }
     }
@@ -319,13 +319,16 @@ struct ThemeHoverListRowModifier: ViewModifier {
     }
 }
 
-struct ThemeLockScreenModifier: ViewModifier {
+struct ThemeLockScreenModifier<LockedContent>: ViewModifier where LockedContent: View {
 
     @AppStorage(AppPreference.locksInBackground.key)
     private var locksInBackground = false
 
-    @ObservedObject
+    @EnvironmentObject
-    var theme: Theme
+    private var theme: Theme
+
+    @ViewBuilder
+    let lockedContent: () -> LockedContent
 
     func body(content: Content) -> some View {
         LockableView(
@@ -333,10 +336,9 @@ struct ThemeLockScreenModifier: ViewModifier {
             content: {
                 content
             },
-            lockedContent: LogoView.init,
+            lockedContent: lockedContent,
             unlockBlock: Self.unlockScreenBlock
         )
-        .environmentObject(theme)
     }
 
     private static func unlockScreenBlock() async -> Bool {

Passepartout/Library/Sources/AppUI/Theme/Theme.swift

@@ -209,8 +209,8 @@ extension View {
         modifier(ThemeHoverListRowModifier())
     }
 
-    public func themeLockScreen(_ theme: Theme) -> some View {
+    public func themeLockScreen() -> some View {
-        modifier(ThemeLockScreenModifier(theme: theme))
+        modifier(ThemeLockScreenModifier(lockedContent: LogoView.init))
     }
 
     public func themeTip(_ text: String, edge: Edge) -> some View {