Replace version text field with picker

This helps ensure that players don't accidentally type bad versions and
wonder why the version isn't supported or is considered invalid.

Author: alicerunsonfedora

CubiomesKit/Sources/CubiomesKit/MinecraftVersion.swift

@@ -33,7 +33,7 @@ public extension MinecraftVersion {
 
 extension MinecraftVersion: @retroactive CaseIterable {
     public static var allCases: [MinecraftVersion] {
-        return (MC_UNDEF.rawValue...MC_NEWEST.rawValue).map {
+        return (MC_B1_7.rawValue...MC_NEWEST.rawValue).map {
             MCVersion($0)
         }
     }

MCMaps/Resources/Localizable.xcstrings

@@ -214,9 +214,6 @@
     },
     "No view was defined for route %@" : {
 
-    },
-    "Not a valid Minecraft version." : {
-
     },
     "Ocean Ruin" : {
 
@@ -338,6 +335,9 @@
     },
     "Welcome to %@" : {
 
+    },
+    "World Generation" : {
+
     },
     "Write a description..." : {
 

MCMaps/Views/Forms/MapCreatorForm.swift

@@ -5,7 +5,6 @@
 //  Created by Marquis Kurt on 01-02-2025.
 //
 
-import CubiomesInternal
 import CubiomesKit
 import SwiftUI
 
@@ -25,10 +24,10 @@ struct MapCreatorForm: View {
     /// A binding to the seed used for world generation.
     @Binding var seed: Int64
 
-    @State private var versionString = ""
     @State private var seedString = ""
     @State private var invalidVersion = false
     @State private var autoconvert = false
+    @State private var version = MC_1_21_WD
 
     #if DEBUG
         internal var didAppear: ((Self) -> Void)?
@@ -38,45 +37,37 @@ struct MapCreatorForm: View {
         Form {
             TextField("Name", text: $worldName)
             Section {
-                TextField("Minecraft Version", text: $versionString)
-            } header: {
-                Text("Minecraft Version")
-            } footer: {
-                if invalidVersion {
-                    Label("Not a valid Minecraft version.", systemImage: "xmark.circle.fill")
-                        .foregroundStyle(Color.red)
-                }
-            }
-            .onSubmit {
-                let mcVersion = str2mc(versionString)
-                invalidVersion = mcVersion == MC_UNDEF.rawValue
-                if !invalidVersion {
-                    self.mcVersion = versionString
+                Picker("Minecraft Version", selection: $version) {
+                    ForEach(MinecraftVersion.allCases, id: \.rawValue) { ver in
+                        Text(String(ver) ?? "?").tag(ver)
+                    }
                 }
-            }
-
-            Section {
                 TextField("Seed", text: $seedString)
+                    .onSubmit {
+                        if let realNumber = Int64(seedString) {
+                            seed = realNumber
+                            autoconvert = false
+                        } else {
+                            seed = Int64(seedString.hashValue)
+                            autoconvert = true
+                        }
+                    }
             } header: {
-                Text("Seed")
+                Text("World Generation")
             } footer: {
                 if autoconvert {
                     let hashedString = String(seedString.hashValue)
                     Text("This seed will be converted to: `\(hashedString)`")
                 }
             }
-            .onSubmit {
-                if let realNumber = Int64(seedString) {
-                    seed = realNumber
-                    autoconvert = false
-                } else {
-                    seed = Int64(seedString.hashValue)
-                    autoconvert = true
+            .onChange(of: version) { _, newValue in
+                if let verString = String(newValue) {
+                    self.mcVersion = verString
                 }
             }
         }
         .onAppear {
-            versionString = mcVersion
+            version = MinecraftVersion(mcVersion)
             seedString = String(seed)
             #if DEBUG
             self.didAppear?(self)
@@ -106,7 +97,6 @@ struct MapCreatorForm: View {
                 self.target = target
             }
 
-            var versionString: String { target.versionString }
             var seedString: String { target.seedString }
             var invalidVersion: Bool { target.invalidVersion }
             var autoconvert: Bool { target.autoconvert }

MCMapsTests/Views/MapCreatorFormTests.swift

@@ -23,7 +23,6 @@ struct MapCreatorFormTests {
         let minecraftVersion: Binding<String> = .init(wrappedValue: "1.21")
         let minecraftSeed: Binding<Int64> = .init(wrappedValue: 123)
         let view = MapCreatorForm(worldName: worldName, mcVersion: minecraftVersion, seed: minecraftSeed) { newView in
-            #expect(newView.testHooks.versionString == "1.21")
             #expect(newView.testHooks.seedString == "123")
         }
         defer { ViewHosting.expel() }