Remove the multipleAttachmentUploadEnabled feature flag.

This commit is contained in:
Stefan Ceriu
2025-08-19 17:19:39 +03:00
parent beb11fd7fd
commit 15fdc2c334
10 changed files with 8 additions and 24 deletions
@@ -22,7 +22,6 @@ protocol CommonSettingsProtocol {
var enableKeyShareOnInvite: Bool { get }
var threadsEnabled: Bool { get }
var hideQuietNotificationAlerts: Bool { get }
var multipleAttachmentUploadEnabled: Bool { get }
}
/// Store Element specific app settings.
@@ -61,7 +60,6 @@ final class AppSettings {
case knockingEnabled
case threadsEnabled
case developerOptionsEnabled
case multipleAttachmentUploadEnabled
// Doug's tweaks 🔧
case hideUnreadMessagesBadge
@@ -380,9 +378,6 @@ final class AppSettings {
@UserPreference(key: UserDefaultsKeys.hideQuietNotificationAlerts, defaultValue: false, storageType: .userDefaults(store))
var hideQuietNotificationAlerts
@UserPreference(key: UserDefaultsKeys.multipleAttachmentUploadEnabled, defaultValue: isDevelopmentBuild, storageType: .userDefaults(store))
var multipleAttachmentUploadEnabled
}
extension AppSettings: CommonSettingsProtocol { }
@@ -963,7 +963,6 @@ class RoomFlowCoordinator: FlowCoordinatorProtocol {
let stackCoordinator = NavigationStackCoordinator()
let mediaPickerCoordinator = MediaPickerScreenCoordinator(mode: mode,
appSettings: appSettings,
userIndicatorController: userIndicatorController,
orientationManager: appMediator.windowManager) { [weak self] action in
guard let self else {
@@ -41,6 +41,8 @@ struct DocumentPicker: UIViewControllerRepresentable {
}
func updateUIViewController(_ uiViewController: UIDocumentPickerViewController, context: Context) {
// Override the app wide tint color (currently set to `.compound.texActionPrimary
// as it's not legible enough in dark mode
uiViewController.view.tintColor = .compound.textActionAccent
}
@@ -29,23 +29,19 @@ enum MediaPickerScreenCoordinatorAction {
}
class MediaPickerScreenCoordinator: CoordinatorProtocol {
private let orientationManager: OrientationManagerProtocol
private let userIndicatorController: UserIndicatorControllerProtocol
private let mode: MediaPickerScreenMode
private let userIndicatorController: UserIndicatorControllerProtocol
private let orientationManager: OrientationManagerProtocol
private let callback: (MediaPickerScreenCoordinatorAction) -> Void
init(mode: MediaPickerScreenMode,
appSettings: AppSettings,
userIndicatorController: UserIndicatorControllerProtocol,
orientationManager: OrientationManagerProtocol,
callback: @escaping (MediaPickerScreenCoordinatorAction) -> Void) {
self.mode = mode
self.userIndicatorController = userIndicatorController
self.orientationManager = orientationManager
self.callback = callback
// The users of the media picker chose their ideal selection type leaving
// the feature flag to only be checked and enforced on this level.
self.mode = appSettings.multipleAttachmentUploadEnabled ? mode : .init(source: mode.source, selectionType: .single)
}
func toPresentable() -> AnyView {
@@ -49,6 +49,8 @@ struct PhotoLibraryPicker: UIViewControllerRepresentable {
}
func updateUIViewController(_ uiViewController: PHPickerViewController, context: Context) {
// Override the app wide tint color (currently set to `.compound.texActionPrimary
// as it's not legible enough in dark mode
uiViewController.view.tintColor = .compound.textActionAccent
}
@@ -68,7 +68,6 @@ final class RoomDetailsEditScreenCoordinator: CoordinatorProtocol {
let stackCoordinator = NavigationStackCoordinator()
let mediaPickerCoordinator = MediaPickerScreenCoordinator(mode: mode,
appSettings: parameters.appSettings,
userIndicatorController: parameters.userIndicatorController,
orientationManager: parameters.orientationManager) { [weak self] action in
guard let self else { return }
@@ -43,7 +43,6 @@ protocol DeveloperOptionsProtocol: AnyObject {
var enableKeyShareOnInvite: Bool { get set }
var threadsEnabled: Bool { get set }
var hideQuietNotificationAlerts: Bool { get set }
var multipleAttachmentUploadEnabled: Bool { get set }
var hideUnreadMessagesBadge: Bool { get set }
var elementCallBaseURLOverride: URL? { get set }
@@ -55,13 +55,7 @@ struct DeveloperOptionsScreen: View {
Text("Low priority filter")
}
}
Section("Timeline") {
Toggle(isOn: $context.multipleAttachmentUploadEnabled) {
Text("Allow selecting multiple attachments for upload")
}
}
Section("Join rules") {
Toggle(isOn: $context.knockingEnabled) {
Text("Knocking")
@@ -57,7 +57,6 @@ final class UserDetailsEditScreenCoordinator: CoordinatorProtocol {
let stackCoordinator = NavigationStackCoordinator()
let mediaPickerCoordinator = MediaPickerScreenCoordinator(mode: mode,
appSettings: parameters.appSettings,
userIndicatorController: parameters.userIndicatorController,
orientationManager: parameters.orientationManager) { [weak self] action in
guard let self else { return }
@@ -144,7 +144,6 @@ final class StartChatScreenCoordinator: CoordinatorProtocol {
let stackCoordinator = NavigationStackCoordinator()
let mediaPickerCoordinator = MediaPickerScreenCoordinator(mode: mode,
appSettings: parameters.appSettings,
userIndicatorController: parameters.userIndicatorController,
orientationManager: parameters.orientationManager) { [weak self] action in
guard let self else { return }