Advanced settings fixes (#1857)

* vector-im/element-meta/issues/2042 - Fix advanced options ordering and capitalisation

* Bump swiftlint's type_body_length warning to 1000, make it match error.

* Fix unit tests

.swiftlint.yml

@@ -34,7 +34,7 @@ type_name:
     error: 1000
 
 type_body_length:
-  warning: 700
+  warning: 1000
   error: 1000
 
 function_body_length:

ElementX/Resources/Localizations/en.lproj/Localizable.strings

@@ -54,7 +54,7 @@
 "action_reply" = "Reply";
 "action_reply_in_thread" = "Reply in thread";
 "action_report_bug" = "Report bug";
-"action_report_content" = "Report Content";
+"action_report_content" = "Report content";
 "action_retry" = "Retry";
 "action_retry_decryption" = "Retry decryption";
 "action_save" = "Save";
@@ -69,7 +69,7 @@
 "action_start_verification" = "Start verification";
 "action_static_map_load" = "Tap to load map";
 "action_take_photo" = "Take photo";
-"action_view_source" = "View Source";
+"action_view_source" = "View source";
 "action_yes" = "Yes";
 "action.edit_poll" = "Edit poll";
 "common_about" = "About";
@@ -122,6 +122,7 @@
 "common_room_name_placeholder" = "e.g. your project name";
 "common_search_for_someone" = "Search for someone";
 "common_search_results" = "Search results";
+"common_secure_backup" = "Secure backup";
 "common_security" = "Security";
 "common_select_your_server" = "Select your server";
 "common_sending" = "Sending…";

ElementX/Sources/Generated/Strings.swift

@@ -124,7 +124,7 @@ public enum L10n {
   public static var actionReplyInThread: String { return L10n.tr("Localizable", "action_reply_in_thread") }
   /// Report bug
   public static var actionReportBug: String { return L10n.tr("Localizable", "action_report_bug") }
-  /// Report Content
+  /// Report content
   public static var actionReportContent: String { return L10n.tr("Localizable", "action_report_content") }
   /// Retry
   public static var actionRetry: String { return L10n.tr("Localizable", "action_retry") }
@@ -154,7 +154,7 @@ public enum L10n {
   public static var actionStaticMapLoad: String { return L10n.tr("Localizable", "action_static_map_load") }
   /// Take photo
   public static var actionTakePhoto: String { return L10n.tr("Localizable", "action_take_photo") }
-  /// View Source
+  /// View source
   public static var actionViewSource: String { return L10n.tr("Localizable", "action_view_source") }
   /// Yes
   public static var actionYes: String { return L10n.tr("Localizable", "action_yes") }
@@ -280,6 +280,8 @@ public enum L10n {
   public static var commonSearchForSomeone: String { return L10n.tr("Localizable", "common_search_for_someone") }
   /// Search results
   public static var commonSearchResults: String { return L10n.tr("Localizable", "common_search_results") }
+  /// Secure backup
+  public static var commonSecureBackup: String { return L10n.tr("Localizable", "common_secure_backup") }
   /// Security
   public static var commonSecurity: String { return L10n.tr("Localizable", "common_security") }
   /// Select your server

ElementX/Sources/Screens/Settings/AvancedOptionsScreen/View/AdvancedSettingsScreen.swift

@@ -28,11 +28,11 @@ struct AdvancedSettingsScreen: View {
                         kind: .picker(selection: $context.timelineStyle,
                                       items: TimelineStyle.allCases.map { (title: $0.name, tag: $0) }))
                 
-                ListRow(label: .default(title: L10n.actionViewSource, systemIcon: .mailAndTextMagnifyingglass),
-                        kind: .toggle($context.viewSourceEnabled))
-                
                 ListRow(label: .default(title: L10n.commonRichTextEditor, icon: Image(asset: Asset.Images.textFormat)),
                         kind: .toggle($context.richTextEditorEnabled))
+                
+                ListRow(label: .default(title: L10n.actionViewSource, systemIcon: .mailAndTextMagnifyingglass),
+                        kind: .toggle($context.viewSourceEnabled))
             }
         }
         .compoundList()