Apply non-pure-black dark mode to preferences screen

Tusker/Screens/Preferences/About/AboutView.swift

@@ -42,6 +42,7 @@ struct AboutView: View {
                     }
                 }
             }
+            .listRowBackground(Color.appGroupedCellBackground)
             
             Section {
                 Link("Website", destination: URL(string: "https://vaccor.space/tusker")!)
@@ -67,7 +68,10 @@ struct AboutView: View {
                 Link("Source Code", destination: URL(string: "https://git.shadowfacts.net/shadowfacts/Tusker")!)
                 Link("Issue Tracker", destination: URL(string: "https://git.shadowfacts.net/shadowfacts/Tusker/issues")!)
             }
+            .listRowBackground(Color.appGroupedCellBackground)
         }
+        .listStyle(.insetGrouped)
+        .appGroupedScrollBackgroundIfAvailable()
         .sheet(isPresented: $isShowingMailSheet) {
             MailSheet(logData: logData)
         }

Tusker/Screens/Preferences/AcknowledgementsView.swift

@@ -14,6 +14,7 @@ struct AcknowledgementsView: View {
             Text(text)
                 .padding(.horizontal, 16)
         }
+        .appGroupedScrollBackgroundIfAvailable()
         .navigationTitle("Acknowledgements")
     }
     

Tusker/Screens/Preferences/AdvancedPrefsView.swift

@@ -23,7 +23,8 @@ struct AdvancedPrefsView : View {
             errorReportingSection
             cachingSection
         }
-        .listStyle(InsetGroupedListStyle())
+        .listStyle(.insetGrouped)
+        .appGroupedScrollBackgroundIfAvailable()
         .navigationBarTitle(Text("Advanced"))
     }
     
@@ -50,6 +51,7 @@ struct AdvancedPrefsView : View {
                 // see FB6838291
             }
         }
+        .listRowBackground(Color.appGroupedCellBackground)
     }
     
     var cloudKitSection: some View {
@@ -74,7 +76,9 @@ struct AdvancedPrefsView : View {
                     Text(String(describing: cloudKitStatus!))
                 }
             }
-        }.task {
+        }
+        .listRowBackground(Color.appGroupedCellBackground)
+        .task {
             CKContainer.default().accountStatus { status, error in
                 if let error {
                     Logging.general.error("Unable to get CloudKit status: \(String(describing: error))")
@@ -99,6 +103,7 @@ struct AdvancedPrefsView : View {
                     .lineLimit(nil)
             }
         }
+        .listRowBackground(Color.appGroupedCellBackground)
     }
     
     var cachingSection: some View {
@@ -120,7 +125,9 @@ struct AdvancedPrefsView : View {
                 s += AttributedString("\nMastodon cache size: \(ByteCountFormatter().string(fromByteCount: mastodonCacheSize))")
             }
             return Text(s)
-        }.task {
+        }
+        .listRowBackground(Color.appGroupedCellBackground)
+        .task {
             imageCacheSize = [
                 ImageCache.avatars,
                 .headers,

Tusker/Screens/Preferences/AppearancePrefsView.swift

@@ -33,7 +33,8 @@ struct AppearancePrefsView : View {
             accountsSection
             postsSection
         }
-        .listStyle(InsetGroupedListStyle())
+        .listStyle(.insetGrouped)
+        .appGroupedScrollBackgroundIfAvailable()
         .navigationBarTitle(Text("Appearance"))
     }
     
@@ -65,6 +66,7 @@ struct AppearancePrefsView : View {
         .onReceive(appearanceChangePublisher) { _ in
             NotificationCenter.default.post(name: .themePreferenceChanged, object: nil)
         }
+        .listRowBackground(Color.appGroupedCellBackground)
     }
     
     private var accountsSection: some View {
@@ -76,6 +78,7 @@ struct AppearancePrefsView : View {
                 Text("Hide Custom Emoji in Usernames")
             }
         }
+        .listRowBackground(Color.appGroupedCellBackground)
     }
     
     private var postsSection: some View {
@@ -103,6 +106,7 @@ struct AppearancePrefsView : View {
                     .navigationTitle("Trailing Swipe Actions")
             }
         }
+        .listRowBackground(Color.appGroupedCellBackground)
     }
 }
 

Tusker/Screens/Preferences/BehaviorPrefsView.swift

@@ -18,7 +18,8 @@ struct BehaviorPrefsView: View {
             linksSection
             contentWarningsSection
         }
-        .listStyle(InsetGroupedListStyle())
+        .listStyle(.insetGrouped)
+        .appGroupedScrollBackgroundIfAvailable()
         .navigationBarTitle(Text("Behavior"))
     }
     
@@ -28,6 +29,7 @@ struct BehaviorPrefsView: View {
                 Text("Require Confirmation Before Reblogging")
             }
         }
+        .listRowBackground(Color.appGroupedCellBackground)
     }
     
     private var timelineSection: some View {
@@ -38,6 +40,7 @@ struct BehaviorPrefsView: View {
         } header: {
             Text("Timeline")
         }
+        .listRowBackground(Color.appGroupedCellBackground)
     }
 
     private var linksSection: some View {
@@ -52,6 +55,7 @@ struct BehaviorPrefsView: View {
                 Text("Always Use Reader Mode in In-App Safari")
             }.disabled(!preferences.useInAppSafari)
         }
+        .listRowBackground(Color.appGroupedCellBackground)
     }
     
     private var contentWarningsSection: some View {
@@ -68,6 +72,7 @@ struct BehaviorPrefsView: View {
                 Text(preferences.expandAllContentWarnings ? "Collapse Posts with Keywords in CWs" : "Expand Posts with Keywords in CWs")
             }
         }
+        .listRowBackground(Color.appGroupedCellBackground)
     }
 }
 

Tusker/Screens/Preferences/ComposingPrefsView.swift

@@ -19,7 +19,8 @@ struct ComposingPrefsView: View {
             replyingSection
             writingSection
         }
-        .listStyle(InsetGroupedListStyle())
+        .listStyle(.insetGrouped)
+        .appGroupedScrollBackgroundIfAvailable()
         .navigationBarTitle("Composing")
     }
     
@@ -51,6 +52,7 @@ struct ComposingPrefsView: View {
         } footer: {
             Text("When starting a reply, Tusker will use your preferred visibility or the visibility of the post to which you're replying, whichever is narrower.")
         }
+        .listRowBackground(Color.appGroupedCellBackground)
     }
     
     var composingSection: some View {
@@ -62,6 +64,7 @@ struct ComposingPrefsView: View {
                 Text("Require Attachment Descriptions")
             }
         }
+        .listRowBackground(Color.appGroupedCellBackground)
     }
     
     var replyingSection: some View {
@@ -75,6 +78,7 @@ struct ComposingPrefsView: View {
                 Text("Mention Reblogger")
             }
         }
+        .listRowBackground(Color.appGroupedCellBackground)
     }
     
     var writingSection: some View {
@@ -83,6 +87,7 @@ struct ComposingPrefsView: View {
                 Text("Show @ and # on Keyboard")
             }
         }
+        .listRowBackground(Color.appGroupedCellBackground)
     }
 
 }

Tusker/Screens/Preferences/MediaPrefsView.swift

@@ -15,7 +15,8 @@ struct MediaPrefsView: View {
         List {
             viewingSection
         }
-        .listStyle(InsetGroupedListStyle())
+        .listStyle(.insetGrouped)
+        .appGroupedScrollBackgroundIfAvailable()
         .navigationBarTitle("Media")
     }
     
@@ -42,6 +43,7 @@ struct MediaPrefsView: View {
                 Text("Show Uncropped Media Inline")
             }
         }
+        .listRowBackground(Color.appGroupedCellBackground)
     }
 }
 

Tusker/Screens/Preferences/OppositeCollapseKeywordsView.swift

@@ -37,9 +37,11 @@ struct OppositeCollapseKeywordsView: View {
                     
                     FocusableTextField(placeholder: "Add Keyword", text: $valueToAdd, becomeFirstResponder: $makeAddFieldFirstResponder, onCommit: self.addKeyword)
                 }
+                .listRowBackground(Color.appGroupedCellBackground)
             }
             .animation(.default, value: keywords.map(\.id))
-            .listStyle(GroupedListStyle())
+            .listStyle(.grouped)
+            .appGroupedScrollBackgroundIfAvailable()
         }
         .onAppear(perform: updateAppearance)
         .navigationBarTitle(preferences.expandAllContentWarnings ? "Collapse Post CW Keywords" : "Expand Post CW Keywords")

Tusker/Screens/Preferences/PreferencesView.swift

@@ -6,11 +6,11 @@
 //
 
 import SwiftUI
-import TTTKit
 
 struct PreferencesView: View {
     let mastodonController: MastodonController
-    @ObservedObject var localData = LocalData.shared
+    
+    @ObservedObject private var localData = LocalData.shared
     @State private var showingLogoutConfirmation = false
     
     init(mastodonController: MastodonController) {
@@ -18,10 +18,24 @@ struct PreferencesView: View {
     }
 
     var body: some View {
-        // workaround: the navigation view is provided by MyProfileTableViewController so that it can inject the Done button
-//        NavigationView {
         List {
-                Section(header: Text("Accounts")) {
+            accountsSection
+            preferencesSection
+            aboutSection
+        }
+        .listStyle(.insetGrouped)
+        .appGroupedScrollBackgroundIfAvailable()
+        .navigationBarTitle("Preferences")
+        .navigationBarTitleDisplayMode(.inline)
+        .onAppear {
+            if #unavailable(iOS 16.0) {
+                UITableView.appearance(whenContainedInInstancesOf: [PreferencesNavigationController.self]).backgroundColor = .appGroupedBackground
+            }
+        }
+    }
+    
+    private var accountsSection: some View {
+        Section {
             ForEach(localData.accounts, id: \.accessToken) { (account) in
                 Button(action: {
                     NotificationCenter.default.post(name: .activateAccount, object: nil, userInfo: ["account": account])
@@ -66,7 +80,6 @@ struct PreferencesView: View {
             }) {
                 Text("Add Account...")
             }
-                    if localData.getMostRecentAccount() != nil {
             Button(action: {
                 self.showingLogoutConfirmation = true
             }) {
@@ -74,9 +87,13 @@ struct PreferencesView: View {
             }.alert(isPresented: $showingLogoutConfirmation) {
                 Alert(title: Text("Are you sure you want to logout?"), message: nil, primaryButton: .destructive(Text("Logout"), action: self.logoutPressed), secondaryButton: .cancel())
             }
+        } header: {
+            Text("Accounts")
         }
+        .listRowBackground(Color.appGroupedCellBackground)
     }
     
+    private var preferencesSection: some View {
         Section {
             NavigationLink(destination: AppearancePrefsView()) {
                 Text("Appearance")
@@ -97,7 +114,10 @@ struct PreferencesView: View {
                 Text("Advanced")
             }
         }
+        .listRowBackground(Color.appGroupedCellBackground)
+    }
     
+    private var aboutSection: some View {
         Section {
             NavigationLink("About") {
                 AboutView()
@@ -109,10 +129,7 @@ struct PreferencesView: View {
                 AcknowledgementsView()
             }
         }
-            }
-            .listStyle(InsetGroupedListStyle())
-            .navigationBarTitle(Text("Preferences"), displayMode: .inline)
-//        }
+        .listRowBackground(Color.appGroupedCellBackground)
     }
     
     func logoutPressed() {
@@ -120,6 +137,20 @@ struct PreferencesView: View {
     }
 }
 
+extension View {
+    @available(iOS, obsoleted: 16.0)
+    @ViewBuilder
+    func appGroupedScrollBackgroundIfAvailable() -> some View {
+        if #available(iOS 16.0, *) {
+            self
+                .scrollContentBackground(.hidden)
+                .background(Color.appGroupedBackground)
+        } else {
+            self
+        }
+    }
+}
+
 //#if DEBUG
 //struct PreferencesView_Previews : PreviewProvider {
 //    static var previews: some View {

Tusker/Screens/Preferences/SwipeActionsPrefsView.swift

@@ -42,6 +42,7 @@ class SwipeActionsPrefsViewController: UIViewController, UICollectionViewDelegat
     override func loadView() {
         let layout = UICollectionViewCompositionalLayout { [unowned self] sectionIndex, environment in
             var config = UICollectionLayoutListConfiguration(appearance: .insetGrouped)
+            config.backgroundColor = .appGroupedBackground
             if dataSource.sectionIdentifier(for: sectionIndex) == .selected {
                 config.headerMode = .supplementary
             }
@@ -59,6 +60,15 @@ class SwipeActionsPrefsViewController: UIViewController, UICollectionViewDelegat
             config.image = UIImage(systemName: item.systemImageName)
             cell.contentConfiguration = config
             cell.accessories = [.reorder(displayed: .always)]
+            cell.configurationUpdateHandler = { cell, state in
+                var config = UIBackgroundConfiguration.listGroupedCell().updated(for: state)
+                if state.isHighlighted || state.isSelected {
+                    config.backgroundColor = .appSelectedCellBackground
+                } else {
+                    config.backgroundColor = .appGroupedCellBackground
+                }
+                cell.backgroundConfiguration = config
+            }
         }
         let dataSource = UICollectionViewDiffableDataSource<Section, Item>(collectionView: collectionView) { collectionView, indexPath, itemIdentifier in
             return collectionView.dequeueConfiguredReusableCell(using: listCell, for: indexPath, item: itemIdentifier)

Tusker/Screens/Preferences/Tip Jar/TipJarView.swift

@@ -26,8 +26,12 @@ struct TipJarView: View {
     @StateObject private var observer = UbiquitousKeyValueStoreObserver()
     
     var body: some View {
+        ZStack {
+            Color.appGroupedBackground
+                .edgesIgnoringSafeArea(.all)
+            
             productsView
-        .overlay {
+            
             if showConfetti {
                 ConfettiView()
                     .transition(.opacity.animation(.default))

Tusker/Screens/Preferences/WellnessPrefsView.swift

@@ -19,7 +19,8 @@ struct WellnessPrefsView: View {
             disableInfiniteScrolling
             hideDiscover
         }
-        .listStyle(InsetGroupedListStyle())
+        .listStyle(.insetGrouped)
+        .appGroupedScrollBackgroundIfAvailable()
         .navigationBarTitle(Text("Digital Wellness"))
     }
     
@@ -29,6 +30,7 @@ struct WellnessPrefsView: View {
                 Text("Favorite and Reblog Counts in Conversations")
             }
         }
+        .listRowBackground(Color.appGroupedCellBackground)
     }
     
     private var notificationsMode: some View {
@@ -39,6 +41,7 @@ struct WellnessPrefsView: View {
                 }
             }
         }
+        .listRowBackground(Color.appGroupedCellBackground)
     }
     
     private var grayscaleImages: some View {
@@ -47,6 +50,7 @@ struct WellnessPrefsView: View {
                 Text("Grayscale Images")
             }
         }
+        .listRowBackground(Color.appGroupedCellBackground)
     }
     
     private var disableInfiniteScrolling: some View {
@@ -55,6 +59,7 @@ struct WellnessPrefsView: View {
                 Text("Disable Infinite Scrolling")
             }
         }
+        .listRowBackground(Color.appGroupedCellBackground)
     }
     
     private var hideDiscover: some View {
@@ -63,6 +68,7 @@ struct WellnessPrefsView: View {
                 Text("Hide Discover Section")
             }
         }
+        .listRowBackground(Color.appGroupedCellBackground)
     }
 }
 

Tusker/Screens/Profile/MyProfileViewController.swift

@@ -8,6 +8,7 @@
 
 import UIKit
 import Pachyderm
+import SwiftUI
 
 class MyProfileViewController: ProfileViewController {