Merge remote-tracking branch 'upstream/main' into update-german-local…

…ization
XcodesOrg · Mar 11, 2024 · fd6baa8 · fd6baa8
2 parents dc691ad + 6f432ee
commit fd6baa8
Show file tree

Hide file tree

Showing 16 changed files with 1,520 additions and 155 deletions.
diff --git a/.github/workflows/appcast.yml b/.github/workflows/appcast.yml
@@ -15,15 +15,15 @@ jobs:
         persist-credentials: false
 
     - name: Cache 📦
-      uses: actions/cache@v4.0.0
+      uses: actions/cache@v4.0.1
       with:
         path: AppCast/vendor/bundle
         key: ${{ runner.os }}-gems-v1.0-${{ hashFiles('AppCast/Gemfile') }}
         restore-keys: |
           ${{ runner.os }}-gems-
           
     - name: Setup Ruby, JRuby and TruffleRuby
-      uses: ruby/setup-ruby@v1.171.0
+      uses: ruby/setup-ruby@v1.172.0
       with:
         ruby-version: '3.0'
 

diff --git a/Xcodes/Backend/AppState+Runtimes.swift b/Xcodes/Backend/AppState+Runtimes.swift
@@ -15,10 +15,10 @@ extension AppState {
                     var updatedRuntime = runtime
 
                     // This loops through and matches up the simulatorVersion to the mappings
-                    let simulatorBuildUpdate = downloadableRuntimes.sdkToSimulatorMappings.first { SDKToSimulatorMapping in
+                    let simulatorBuildUpdate = downloadableRuntimes.sdkToSimulatorMappings.filter { SDKToSimulatorMapping in
                         SDKToSimulatorMapping.simulatorBuildUpdate == runtime.simulatorVersion.buildUpdate
                     }
-                    updatedRuntime.sdkBuildUpdate = simulatorBuildUpdate?.sdkBuildUpdate
+                    updatedRuntime.sdkBuildUpdate = simulatorBuildUpdate.map { $0.sdkBuildUpdate }
                     return updatedRuntime
                 }
 

diff --git a/Xcodes/Frontend/About/AcknowledgementsView.swift b/Xcodes/Frontend/About/AcknowledgementsView.swift
@@ -10,7 +10,7 @@ struct AcknowledgmentsView: View {
             )!
             .addingAttribute(.foregroundColor, value: NSColor.labelColor)
         )
-        .frame(minWidth: 500, minHeight: 500)
+        .frame(minWidth: 600, minHeight: 500)
     }
 }
 

diff --git a/Xcodes/Frontend/InfoPane/CompilersView.swift b/Xcodes/Frontend/InfoPane/CompilersView.swift
@@ -16,7 +16,9 @@ struct CompilersView: View {
         if let compilers = compilers {
             VStack(alignment: .leading) {
                 Text("Compilers").font(.headline)
-                Text(Self.content(from: compilers)).font(.subheadline)
+                Text(Self.content(from: compilers))
+                    .font(.subheadline)
+                    .textSelection(.enabled)
             }
         } else {
             EmptyView()

diff --git a/Xcodes/Frontend/InfoPane/InfoPane.swift b/Xcodes/Frontend/InfoPane/InfoPane.swift
@@ -19,6 +19,7 @@ struct InfoPane: View {
                             Text(verbatim: "Xcode \(xcode.description) \(xcode.version.buildMetadataIdentifiersDisplay)")
                                 .font(.title)
                                 .frame(maxWidth: .infinity, alignment: .leading)
+                                .textSelection(.enabled)
                         }
                         InfoPaneControls(xcode: xcode)
                     }
@@ -146,12 +147,12 @@ var xcodeDict: [XcodePreviewName: Xcode] = [
 var downloadableRuntimes: [DownloadableRuntime] = {
     var runtimes = try! JSONDecoder().decode([DownloadableRuntime].self, from: Current.files.contents(atPath: Path.runtimeCacheFile.string)!)
     // set iOS to installed
-    let iOSIndex = runtimes.firstIndex { $0.sdkBuildUpdate == "19E239" }!
+    let iOSIndex = 0//runtimes.firstIndex { $0.sdkBuildUpdate.contains == "19E239" }!
     var iOSRuntime = runtimes[iOSIndex]
     iOSRuntime.installState = .installed
     runtimes[iOSIndex] = iOSRuntime
 
-    let watchOSIndex = runtimes.firstIndex { $0.sdkBuildUpdate == "20R362" }!
+    let watchOSIndex = 0//runtimes.firstIndex { $0.sdkBuildUpdate.first == "20R362" }!
     var runtime = runtimes[watchOSIndex]
     runtime.installState = .installing(
         RuntimeInstallationStep.downloading(

diff --git a/Xcodes/Frontend/InfoPane/PlatformsView.swift b/Xcodes/Frontend/InfoPane/PlatformsView.swift
@@ -19,7 +19,7 @@ struct PlatformsView: View {
         let builds = xcode.sdks?.allBuilds()
         let runtimes = builds?.flatMap { sdkBuild in
             appState.downloadableRuntimes.filter {
-                $0.sdkBuildUpdate == sdkBuild
+                $0.sdkBuildUpdate?.contains(sdkBuild) ?? false
             }
         }
 

diff --git a/Xcodes/Frontend/InfoPane/SDKsView.swift b/Xcodes/Frontend/InfoPane/SDKsView.swift
@@ -18,7 +18,9 @@ struct SDKsView: View {
         } else {
             VStack(alignment: .leading) {
                 Text("SDKs").font(.headline)
-                Text(content).font(.subheadline)
+                Text(content)
+                    .font(.subheadline)
+                    .textSelection(.enabled)
             }
         }
     }

diff --git a/Xcodes/Frontend/Preferences/AdvancedPreferencePane.swift b/Xcodes/Frontend/Preferences/AdvancedPreferencePane.swift
@@ -38,6 +38,7 @@ struct AdvancedPreferencePane: View {
                     }
                     Text("InstallPathDescription")
                         .font(.footnote)
+                        .foregroundStyle(.secondary)
                         .fixedSize(horizontal: false, vertical: true)
                 }
             }
@@ -73,6 +74,7 @@ struct AdvancedPreferencePane: View {
                     }
                     Text("LocalCachePathDescription")
                         .font(.footnote)
+                        .foregroundStyle(.secondary)
                         .fixedSize(horizontal: false, vertical: true)
                 }
             }
@@ -94,6 +96,7 @@ struct AdvancedPreferencePane: View {
 
                     Text(appState.onSelectActionType.detailedDescription)
                         .font(.footnote)
+                        .foregroundStyle(.secondary)
                         .fixedSize(horizontal: false, vertical: true)
                     Spacer()
                         .frame(height: 20)
@@ -102,6 +105,7 @@ struct AdvancedPreferencePane: View {
                         .disabled(appState.createSymLinkOnSelectDisabled)
                     Text("AutomaticallyCreateSymbolicLinkDescription")
                         .font(.footnote)
+                        .foregroundStyle(.secondary)
                         .fixedSize(horizontal: false, vertical: true)
                 }
                 .fixedSize(horizontal: false, vertical: true)
@@ -114,6 +118,7 @@ struct AdvancedPreferencePane: View {
                         .disabled(appState.createSymLinkOnSelectDisabled)
                     Text("ShowOpenInRosettaDescription")
                         .font(.footnote)
+                        .foregroundStyle(.secondary)
                         .fixedSize(horizontal: false, vertical: true)
                 }
                 .groupBoxStyle(PreferencesGroupBoxStyle())
@@ -128,16 +133,18 @@ struct AdvancedPreferencePane: View {
                     case .installed:
                         Text("HelperInstalled")
                     case .notInstalled:
-                        HStack {
-                            Text("HelperNotInstalled")
+                        VStack(alignment: .leading) {
                             Button("InstallHelper") {
                                 appState.installHelperIfNecessary()
                             }
+                            Text("HelperNotInstalled")
+                                .font(.footnote)
                         }
                     }
 
                     Text("PrivilegedHelperDescription")
                         .font(.footnote)
+                        .foregroundStyle(.secondary)
                         .fixedSize(horizontal: false, vertical: true)
 
                     Spacer()
@@ -153,21 +160,18 @@ struct AdvancedPreferencePane_Previews: PreviewProvider {
         Group {
             AdvancedPreferencePane()
                 .environmentObject(AppState())
-                .frame(maxWidth: 500)
+                .frame(maxWidth: 600)
         }
-        .frame(width: 500, height: 700, alignment: .center)
+        .frame(width: 600, height: 700, alignment: .center)
     }
 }
 
 // A group style for the preferences
 struct PreferencesGroupBoxStyle: GroupBoxStyle {
     func makeBody(configuration: Configuration) -> some View {
         HStack(alignment: .top, spacing: 20) {
-            HStack {
-                Spacer()
-                configuration.label
-            }
-            .frame(width: 120)
+            configuration.label
+                .frame(width: 180, alignment: .trailing)
 
             VStack(alignment: .leading) {
                 configuration.content

diff --git a/Xcodes/Frontend/Preferences/DownloadPreferencePane.swift b/Xcodes/Frontend/Preferences/DownloadPreferencePane.swift
@@ -18,6 +18,7 @@ struct DownloadPreferencePane: View {
                         }
                     }
                     .labelsHidden()
+                    .fixedSize()
 
                     AttributedText(dataSourceFootnote)
                 }
@@ -34,6 +35,7 @@ struct DownloadPreferencePane: View {
                         }
                     }
                     .labelsHidden()
+                    .fixedSize()
 
                     AttributedText(downloaderFootnote)
                 }
@@ -50,7 +52,7 @@ struct DownloadPreferencePane: View {
             string: string,
             attributes: [
                 .font: NSFont.preferredFont(forTextStyle: .footnote, options: [:]),
-                .foregroundColor: NSColor.labelColor
+                .foregroundColor: NSColor.secondaryLabelColor
             ]
         )
         attributedString.addAttribute(.link, value: URL(string: "https://xcodereleases.com")!, range: NSRange(string.range(of: "Xcode Releases")!, in: string))
@@ -63,7 +65,7 @@ struct DownloadPreferencePane: View {
             string: string,
             attributes: [
                 .font: NSFont.preferredFont(forTextStyle: .footnote, options: [:]),
-                .foregroundColor: NSColor.labelColor
+                .foregroundColor: NSColor.secondaryLabelColor
             ]
         )
         attributedString.addAttribute(.link, value: URL(string: "https://github.com/aria2/aria2")!, range: NSRange(string.range(of: "aria2")!, in: string))
@@ -74,9 +76,10 @@ struct DownloadPreferencePane: View {
 struct DownloadPreferencePane_Previews: PreviewProvider {
     static var previews: some View {
         Group {
-            GeneralPreferencePane()
+            DownloadPreferencePane()
                 .environmentObject(AppState())
-                .frame(maxWidth: 500)
+                .frame(maxWidth: 600)
+                .frame(minHeight: 300)
         }
     }
 }
diff --git a/Xcodes/Frontend/Preferences/ExperiementsPreferencePane.swift b/Xcodes/Frontend/Preferences/ExperiementsPreferencePane.swift
@@ -18,8 +18,6 @@ struct ExperimentsPreferencePane: View {
                 .fixedSize(horizontal: false, vertical: true)
             }
             .groupBoxStyle(PreferencesGroupBoxStyle())
-
-            Divider()
         }
     }
 
@@ -29,7 +27,7 @@ struct ExperimentsPreferencePane: View {
             string: string,
             attributes: [
                 .font: NSFont.preferredFont(forTextStyle: .footnote, options: [:]),
-                .foregroundColor: NSColor.labelColor
+                .foregroundColor: NSColor.secondaryLabelColor
             ]
         )
         attributedString.addAttribute(.link, value: URL(string: "https://twitter.com/_saagarjha")!, range: NSRange(string.range(of: "@_saagarjha")!, in: string))
@@ -43,7 +41,7 @@ struct ExperimentsPreferencePane_Previews: PreviewProvider {
         Group {
             ExperimentsPreferencePane()
                 .environmentObject(AppState())
-                .frame(maxWidth: 500)
+                .frame(maxWidth: 600)
         }
     }
 }
diff --git a/Xcodes/Frontend/Preferences/GeneralPreferencePane.swift b/Xcodes/Frontend/Preferences/GeneralPreferencePane.swift
@@ -29,7 +29,7 @@ struct GeneralPreferencePane_Previews: PreviewProvider {
         Group {
             GeneralPreferencePane()
                 .environmentObject(AppState())
-                .frame(maxWidth: 500)
+                .frame(maxWidth: 600)
         }
     }
 }
diff --git a/Xcodes/Frontend/Preferences/PreferencesView.swift b/Xcodes/Frontend/Preferences/PreferencesView.swift
@@ -39,6 +39,6 @@ struct PreferencesView: View {
                 .tag(Tabs.experiment)
         }
         .padding(20)
-        .frame(width: 500)
+        .frame(width: 600)
     }
 }
diff --git a/Xcodes/Frontend/Preferences/UpdatesPreferencePane.swift b/Xcodes/Frontend/Preferences/UpdatesPreferencePane.swift
@@ -43,9 +43,11 @@ struct UpdatesPreferencePane: View {
                     Button("CheckNow") {
                         updater.checkForUpdates()
                     }
+                    .padding(.top)
 
                     Text(String(format: localizeString("LastChecked"), lastUpdatedString))
                         .font(.footnote)
+                        .foregroundStyle(.secondary)
                 }
                 .frame(maxWidth: .infinity, alignment: .leading)
             }
@@ -140,7 +142,9 @@ struct UpdatesPreferencePane_Previews: PreviewProvider {
         Group {
             UpdatesPreferencePane()
                 .environmentObject(AppState())
-                .frame(maxWidth: 500)
+                .environmentObject(ObservableUpdater())
+                .frame(maxWidth: 600)
+                .frame(minHeight: 300)
         }
     }
 }
diff --git a/Xcodes/Frontend/SignIn/SignInCredentialsView.swift b/Xcodes/Frontend/SignIn/SignInCredentialsView.swift
@@ -1,9 +1,14 @@
 import SwiftUI
 
 struct SignInCredentialsView: View {
+    private enum FocusedField {
+        case username, password
+    }
+
     @EnvironmentObject var appState: AppState
     @State private var username: String = ""
     @State private var password: String = ""
+    @FocusState private var focusedField: FocusedField?
 
     var body: some View {
         VStack(alignment: .leading) {
@@ -16,11 +21,13 @@ struct SignInCredentialsView: View {
                 TextField(text: $username) {
                     Text(verbatim: "example@icloud.com")
                 }
+                .focused($focusedField, equals: .username)
             }
             HStack {
                 Text("Password")
                     .frame(minWidth: 100, alignment: .trailing)
                 SecureField("Required", text: $password)
+                    .focused($focusedField, equals: .password)
             }
             if appState.authError != nil {
                 HStack {