[iOS] Media Item Menu - Edit Item Images

Shared/Coordinators/ItemEditorCoordinator.swift

@@ -26,6 +26,11 @@ final class ItemEditorCoordinator: ObservableObject, NavigationCoordinatable {
     @Route(.modal)
     var editMetadata = makeEditMetadata
 
+    // MARK: - Route to Images
+
+    @Route(.modal)
+    var imageEditor = makeImageEditor
+
     // MARK: - Route to Genres
 
     @Route(.push)
@@ -73,6 +78,12 @@ final class ItemEditorCoordinator: ObservableObject, NavigationCoordinatable {
         }
     }
 
+    // MARK: - Item Images
+
+    func makeImageEditor(viewModel: ItemImagesViewModel) -> NavigationViewCoordinator<ItemImagesCoordinator> {
+        NavigationViewCoordinator(ItemImagesCoordinator(viewModel: viewModel))
+    }
+
     // MARK: - Item Genres
 
     @ViewBuilder

Shared/Coordinators/ItemImagesCoordinator.swift

@@ -0,0 +1,69 @@
+//
+// Swiftfin is subject to the terms of the Mozilla Public
+// License, v2.0. If a copy of the MPL was not distributed with this
+// file, you can obtain one at https://mozilla.org/MPL/2.0/.
+//
+// Copyright (c) 2025 Jellyfin & Jellyfin Contributors
+//
+
+import JellyfinAPI
+import Stinsen
+import SwiftUI
+
+final class ItemImagesCoordinator: ObservableObject, NavigationCoordinatable {
+
+    let stack = NavigationStack(initial: \ItemImagesCoordinator.start)
+
+    @Root
+    var start = makeStart
+
+    @ObservedObject
+    private var viewModel: ItemImagesViewModel
+
+    // MARK: - Route to Views
+
+    @Route(.push)
+    var addImage = makeAddImage
+    @Route(.modal)
+    var deleteImage = makeDeleteImage
+    @Route(.modal)
+    var selectImage = makeSelectImage
+    @Route(.modal)
+    var photoPicker = makePhotoPicker
+
+    // MARK: - Initializer
+
+    init(viewModel: ItemImagesViewModel) {
+        self._viewModel = ObservedObject(wrappedValue: viewModel)
+    }
+
+    // MARK: - Item Images
+
+    @ViewBuilder
+    func makeAddImage(imageType: ImageType) -> some View {
+        AddItemImageView(viewModel: viewModel, imageType: imageType)
+    }
+
+    func makeDeleteImage(imageInfo: (key: ImageInfo, value: UIImage)) -> NavigationViewCoordinator<BasicNavigationViewCoordinator> {
+        NavigationViewCoordinator {
+            ItemImageDetailsView(viewModel: self.viewModel, localImageInfo: imageInfo)
+        }
+    }
+
+    func makeSelectImage(remoteImageInfo: RemoteImageInfo) -> NavigationViewCoordinator<BasicNavigationViewCoordinator> {
+        NavigationViewCoordinator {
+            ItemImageDetailsView(viewModel: self.viewModel, remoteImageInfo: remoteImageInfo)
+        }
+    }
+
+    func makePhotoPicker(type: ImageType) -> NavigationViewCoordinator<ItemPhotoCoordinator> {
+        NavigationViewCoordinator(ItemPhotoCoordinator(viewModel: self.viewModel, type: type))
+    }
+
+    // MARK: - Start
+
+    @ViewBuilder
+    func makeStart() -> some View {
+        ItemImagesView(viewModel: self.viewModel)
+    }
+}

Shared/Coordinators/ItemPhotoCoordinator.swift

@@ -0,0 +1,59 @@
+//
+// Swiftfin is subject to the terms of the Mozilla Public
+// License, v2.0. If a copy of the MPL was not distributed with this
+// file, you can obtain one at https://mozilla.org/MPL/2.0/.
+//
+// Copyright (c) 2025 Jellyfin & Jellyfin Contributors
+//
+
+import JellyfinAPI
+import Stinsen
+import SwiftUI
+
+final class ItemPhotoCoordinator: NavigationCoordinatable {
+
+    // MARK: - Navigation Components
+
+    let stack = Stinsen.NavigationStack(initial: \ItemPhotoCoordinator.start)
+
+    @Root
+    var start = makeStart
+
+    // MARK: - Routes
+
+    @Route(.push)
+    var cropImage = makeCropImage
+
+    // MARK: - Observed Object
+
+    @ObservedObject
+    var viewModel: ItemImagesViewModel
+
+    let type: ImageType
+
+    // MARK: - Initializer
+
+    init(viewModel: ItemImagesViewModel, type: ImageType) {
+        self.viewModel = viewModel
+        self.type = type
+    }
+
+    // MARK: - Views
+
+    func makeCropImage(image: UIImage) -> some View {
+        #if os(iOS)
+        ItemImagePicker.ImageCropView(viewModel: viewModel, image: image, type: type)
+        #else
+        AssertionFailureView("not implemented")
+        #endif
+    }
+
+    @ViewBuilder
+    func makeStart() -> some View {
+        #if os(iOS)
+        ItemImagePicker()
+        #else
+        AssertionFailureView("not implemented")
+        #endif
+    }
+}

Shared/Extensions/Hashable.swift → Shared/Extensions/JellyfinAPI/ImageInfo.swift

@@ -7,10 +7,11 @@
 //
 
 import Foundation
+import JellyfinAPI
 
-extension Hashable {
+extension ImageInfo: @retroactive Identifiable {
 
-    var hashString: String {
-        "\(hashValue)"
+    public var id: Int {
+        hashValue
     }
 }

Shared/Extensions/JellyfinAPI/RemoteImageInfo.swift

@@ -0,0 +1,34 @@
+//
+// Swiftfin is subject to the terms of the Mozilla Public
+// License, v2.0. If a copy of the MPL was not distributed with this
+// file, you can obtain one at https://mozilla.org/MPL/2.0/.
+//
+// Copyright (c) 2025 Jellyfin & Jellyfin Contributors
+//
+
+import Foundation
+import JellyfinAPI
+import SwiftUI
+
+extension RemoteImageInfo: @retroactive Identifiable, Poster {
+
+    var displayTitle: String {
+        self.providerName ?? L10n.unknown
+    }
+
+    var unwrappedIDHashOrZero: Int {
+        self.id
+    }
+
+    var subtitle: String? {
+        self.language
+    }
+
+    var systemImage: String {
+        "circle"
+    }
+
+    public var id: Int {
+        self.hashValue
+    }
+}

Shared/Extensions/JellyfinAPI/Request.swift

@@ -0,0 +1,66 @@
+//
+// Swiftfin is subject to the terms of the Mozilla Public
+// License, v2.0. If a copy of the MPL was not distributed with this
+// file, you can obtain one at https://mozilla.org/MPL/2.0/.
+//
+// Copyright (c) 2025 Jellyfin & Jellyfin Contributors
+//
+
+import Foundation
+import Get
+
+public extension Request {
+    /// Size of the request HTTP method in bytes
+    var methodSize: Int {
+        method.rawValue.count
+    }
+
+    /// Size of the request URL in bytes
+    var urlSize: Int {
+        url?.absoluteString.count ?? 0
+    }
+
+    /// Size of the request query parameters in bytes
+    var querySize: Int {
+        guard let query = query else { return 0 }
+        return query.reduce(0) { $0 + $1.0.count + ($1.1?.count ?? 0) + 2 }
+    }
+
+    /// Size of the request headers in bytes
+    var headersSize: Int {
+        guard let headers = headers else { return 0 }
+        return headers.reduce(0) { $0 + $1.key.count + $1.value.count + 4 }
+    }
+
+    /// Size of the request body in bytes
+    var bodySize: Int {
+        var size = 0
+        if let body = body {
+            do {
+                let bodyData = try JSONEncoder().encode(AnyEncodable(body))
+                size += bodyData.count
+            } catch {
+                size += 0
+            }
+        }
+        return size
+    }
+
+    /// Total size of the total request in bytes
+    var requestSize: Int {
+        methodSize + urlSize + querySize + headersSize + bodySize
+    }
+}
+
+/// A type-erased `Encodable` to encode any value conforming to `Encodable`
+private struct AnyEncodable: Encodable {
+    private let _encode: (Encoder) throws -> Void
+
+    init<T: Encodable>(_ value: T) {
+        _encode = value.encode
+    }
+
+    func encode(to encoder: Encoder) throws {
+        try _encode(encoder)
+    }
+}

Shared/Strings/Strings.swift

@@ -398,6 +398,8 @@ internal enum L10n {
   }
   /// Are you sure you wish to delete this device? This session will be logged out.
   internal static let deleteDeviceWarning = L10n.tr("Localizable", "deleteDeviceWarning", fallback: "Are you sure you wish to delete this device? This session will be logged out.")
+  /// Delete image
+  internal static let deleteImage = L10n.tr("Localizable", "deleteImage", fallback: "Delete image")
   /// Are you sure you want to delete this item?
   internal static let deleteItemConfirmation = L10n.tr("Localizable", "deleteItemConfirmation", fallback: "Are you sure you want to delete this item?")
   /// Are you sure you want to delete this item? This action cannot be undone.
@@ -624,6 +626,8 @@ internal enum L10n {
   internal static let idle = L10n.tr("Localizable", "idle", fallback: "Idle")
   /// Illustrator
   internal static let illustrator = L10n.tr("Localizable", "illustrator", fallback: "Illustrator")
+  /// Images
+  internal static let images = L10n.tr("Localizable", "images", fallback: "Images")
   /// Indicators
   internal static let indicators = L10n.tr("Localizable", "indicators", fallback: "Indicators")
   /// Inker
@@ -1320,6 +1324,10 @@ internal enum L10n {
   internal static let unreleased = L10n.tr("Localizable", "unreleased", fallback: "Unreleased")
   /// You have unsaved changes. Are you sure you want to discard them?
   internal static let unsavedChangesMessage = L10n.tr("Localizable", "unsavedChangesMessage", fallback: "You have unsaved changes. Are you sure you want to discard them?")
+  /// Upload file
+  internal static let uploadFile = L10n.tr("Localizable", "uploadFile", fallback: "Upload file")
+  /// Upload photo
+  internal static let uploadPhoto = L10n.tr("Localizable", "uploadPhoto", fallback: "Upload photo")
   /// URL
   internal static let url = L10n.tr("Localizable", "url", fallback: "URL")
   /// Use as Transcoding Profile