Merge pull request #7317 from vector-im/aringenbach/7316_indentation_…

…support Labs: Rich text editor: enable list items indentation
element-hq · Feb 1, 2023 · 2aaef07 · 2aaef07
2 parents b1a1cb1 + 4e0a127
commit 2aaef07
Show file tree

Hide file tree

Showing 17 changed files with 81 additions and 132 deletions.
diff --git a/Podfile b/Podfile
@@ -53,8 +53,6 @@ end
 
 def import_MatrixKit_pods
   pod 'libPhoneNumber-iOS', '~> 0.9.13'  
-  pod 'DTCoreText', '~> 1.6.25'
-  #pod 'DTCoreText/Extension', '~> 1.6.25'
   pod 'Down', '~> 0.11.0'
 end
 

diff --git a/Riot.xcworkspace/xcshareddata/swiftpm/Package.resolved b/Riot.xcworkspace/xcshareddata/swiftpm/Package.resolved
@@ -9,6 +9,24 @@
         "version" : "4.7.0"
       }
     },
+    {
+      "identity" : "dtcoretext",
+      "kind" : "remoteSourceControl",
+      "location" : "https://github.com/Cocoanetics/DTCoreText",
+      "state" : {
+        "revision" : "9d2d4d2296e5d2d852a7d3c592b817d913a5d020",
+        "version" : "1.6.27"
+      }
+    },
+    {
+      "identity" : "dtfoundation",
+      "kind" : "remoteSourceControl",
+      "location" : "https://github.com/Cocoanetics/DTFoundation.git",
+      "state" : {
+        "revision" : "76062513434421cb6c8a1ae1d4f8368a7ebc2da3",
+        "version" : "1.7.18"
+      }
+    },
     {
       "identity" : "maplibre-gl-native-distribution",
       "kind" : "remoteSourceControl",
@@ -23,8 +41,8 @@
       "kind" : "remoteSourceControl",
       "location" : "https://github.com/matrix-org/matrix-wysiwyg-composer-swift",
       "state" : {
-        "revision" : "6927cb878376136c4a03d919b689af8dfbdad080",
-        "version" : "0.19.0"
+        "revision" : "3f72aeab7d7e04b52ff3f735ab79a75993f97ef2",
+        "version" : "0.22.0"
       }
     },
     {

diff --git a/Riot/Assets/en.lproj/Vector.strings b/Riot/Assets/en.lproj/Vector.strings
@@ -2582,8 +2582,8 @@ To enable access, tap Settings> Location and select Always";
 "wysiwyg_composer_format_action_ordered_list" = "Toggle numbered list";
 "wysiwyg_composer_format_action_code_block" = "Toggle code block";
 "wysiwyg_composer_format_action_quote" = "Toggle quote";
-
-
+"wysiwyg_composer_format_action_indent" = "Increase indentation";
+"wysiwyg_composer_format_action_un_indent" = "Decrease indentation";
 
 // Links
 "wysiwyg_composer_link_action_text" = "Text";

diff --git a/Riot/Generated/Strings.swift b/Riot/Generated/Strings.swift
@@ -9415,6 +9415,10 @@ public class VectorL10n: NSObject {
   public static var wysiwygComposerFormatActionCodeBlock: String { 
     return VectorL10n.tr("Vector", "wysiwyg_composer_format_action_code_block") 
   }
+  /// Increase indentation
+  public static var wysiwygComposerFormatActionIndent: String { 
+    return VectorL10n.tr("Vector", "wysiwyg_composer_format_action_indent") 
+  }
   /// Apply inline code format
   public static var wysiwygComposerFormatActionInlineCode: String { 
     return VectorL10n.tr("Vector", "wysiwyg_composer_format_action_inline_code") 
@@ -9439,6 +9443,10 @@ public class VectorL10n: NSObject {
   public static var wysiwygComposerFormatActionStrikethrough: String { 
     return VectorL10n.tr("Vector", "wysiwyg_composer_format_action_strikethrough") 
   }
+  /// Decrease indentation
+  public static var wysiwygComposerFormatActionUnIndent: String { 
+    return VectorL10n.tr("Vector", "wysiwyg_composer_format_action_un_indent") 
+  }
   /// Apply strikethrough format
   public static var wysiwygComposerFormatActionUnderline: String { 
     return VectorL10n.tr("Vector", "wysiwyg_composer_format_action_underline") 

diff --git a/Riot/Modules/MatrixKit/Utils/EventFormatter/HTMLFormatter.swift b/Riot/Modules/MatrixKit/Utils/EventFormatter/HTMLFormatter.swift
@@ -47,9 +47,7 @@ class HTMLFormatter: NSObject {
 
         var options: [AnyHashable: Any] = [
             DTUseiOS6Attributes: true,
-            DTDefaultFontFamily: font.familyName,
-            DTDefaultFontName: font.fontName,
-            DTDefaultFontSize: font.pointSize,
+            DTDefaultFontDescriptor: font.fontDescriptor,
             DTDefaultLinkDecoration: false,
             DTDefaultLinkColor: ThemeService.shared().theme.colors.links,
             DTWillFlushBlockCallBack: sanitizeCallback

diff --git a/Riot/Modules/Room/Views/WYSIWYGInputToolbar/WysiwygInputToolbarView.swift b/Riot/Modules/Room/Views/WYSIWYGInputToolbar/WysiwygInputToolbarView.swift
@@ -17,6 +17,7 @@
 import Foundation
 import Reusable
 import WysiwygComposer
+import HTMLParser
 import SwiftUI
 import Combine
 import UIKit
@@ -43,9 +44,14 @@ class WysiwygInputToolbarView: MXKRoomInputToolbarView, NibLoadable, HtmlRoomInp
     private var voiceMessageBottomConstraint: NSLayoutConstraint?
     private var hostingViewController: VectorHostingController!
     private var wysiwygViewModel = WysiwygComposerViewModel(
-        textColor: ThemeService.shared().theme.colors.primaryContent,
-        linkColor: ThemeService.shared().theme.colors.links,
-        codeBackgroundColor: ThemeService.shared().theme.selectedBackgroundColor
+        parserStyle: HTMLParserStyle(textColor: ThemeService.shared().theme.colors.primaryContent,
+                                     linkColor: ThemeService.shared().theme.colors.links,
+                                     codeBackgroundColor: ThemeService.shared().theme.selectedBackgroundColor,
+                                     codeBorderColor: ThemeService.shared().theme.textQuinaryColor,
+                                     quoteBackgroundColor: ThemeService.shared().theme.selectedBackgroundColor,
+                                     quoteBorderColor: ThemeService.shared().theme.textQuinaryColor,
+                                     borderWidth: 1.0,
+                                     cornerRadius: 4.0)
     )
     private var viewModel: ComposerViewModelProtocol!
 
@@ -298,9 +304,14 @@ class WysiwygInputToolbarView: MXKRoomInputToolbarView, NibLoadable, HtmlRoomInp
 
     private func update(theme: Theme) {
         hostingViewController.view.backgroundColor = theme.colors.background
-        wysiwygViewModel.textColor = theme.colors.primaryContent
-        wysiwygViewModel.linkColor = theme.colors.links
-        wysiwygViewModel.codeBackgroundColor = theme.selectedBackgroundColor
+        wysiwygViewModel.parserStyle = HTMLParserStyle(textColor: ThemeService.shared().theme.colors.primaryContent,
+                                                       linkColor: ThemeService.shared().theme.colors.links,
+                                                       codeBackgroundColor: ThemeService.shared().theme.selectedBackgroundColor,
+                                                       codeBorderColor: ThemeService.shared().theme.textQuinaryColor,
+                                                       quoteBackgroundColor: ThemeService.shared().theme.selectedBackgroundColor,
+                                                       quoteBorderColor: ThemeService.shared().theme.textQuinaryColor,
+                                                       borderWidth: 1.0,
+                                                       cornerRadius: 4.0)
     }
 
     private func updateTextViewHeight() {

diff --git a/Riot/Utils/EventFormatter+DTCoreTextFix.h b/Riot/Utils/EventFormatter+DTCoreTextFix.h
diff --git a/Riot/Utils/EventFormatter+DTCoreTextFix.m b/Riot/Utils/EventFormatter+DTCoreTextFix.m
diff --git a/Riot/Utils/EventFormatter.m b/Riot/Utils/EventFormatter.m
@@ -25,7 +25,6 @@
 #import "MXDecryptionResult.h"
 #import "DecryptionFailureTracker.h"
 
-#import "EventFormatter+DTCoreTextFix.h"
 #import <MatrixSDK/MatrixSDK.h>
 
 #pragma mark - Constants definitions
@@ -50,11 +49,6 @@ @interface EventFormatter ()
 
 @implementation EventFormatter
 
-+ (void)load
-{
-    [self fixDTCoreTextFont];
-}
-
 - (void)initDateTimeFormatters
 {
     [super initDateTimeFormatters];

diff --git a/Riot/target.yml b/Riot/target.yml
@@ -44,6 +44,7 @@ targets:
     - package: SwiftOGG
     - package: WysiwygComposer
     - package: DeviceKit
+    - package: DTCoreText
 
     configFiles:
       Debug: Debug.xcconfig

diff --git a/RiotNSE/target.yml b/RiotNSE/target.yml
@@ -33,6 +33,7 @@ targets:
 
     dependencies:
     - package: DeviceKit
+    - package: DTCoreText
 
     configFiles:
       Debug: Debug.xcconfig

diff --git a/RiotShareExtension/target.yml b/RiotShareExtension/target.yml
@@ -33,6 +33,7 @@ targets:
 
     dependencies:
     - package: DeviceKit
+    - package: DTCoreText
 
     configFiles:
       Debug: Debug.xcconfig

diff --git a/RiotSwiftUI/Modules/Room/Composer/Model/ComposerModels.swift b/RiotSwiftUI/Modules/Room/Composer/Model/ComposerModels.swift
@@ -36,6 +36,8 @@ enum FormatType {
     case strikethrough
     case unorderedList
     case orderedList
+    case indent
+    case unIndent
     case inlineCode
     case codeBlock
     case quote
@@ -66,6 +68,10 @@ extension FormatItem {
             return Asset.Images.bulletList.name
         case .orderedList:
             return Asset.Images.numberedList.name
+        case .indent:
+            return Asset.Images.indentIncrease.name
+        case .unIndent:
+            return Asset.Images.indentDecrease.name
         case .inlineCode:
             return Asset.Images.code.name
         case .codeBlock:
@@ -91,6 +97,10 @@ extension FormatItem {
             return "unorderedListButton"
         case .orderedList:
             return "orderedListButton"
+        case .indent:
+            return "indentListButton"
+        case .unIndent:
+            return "unIndentButton"
         case .inlineCode:
             return "inlineCodeButton"
         case .codeBlock:
@@ -116,6 +126,10 @@ extension FormatItem {
             return VectorL10n.wysiwygComposerFormatActionUnorderedList
         case .orderedList:
             return VectorL10n.wysiwygComposerFormatActionOrderedList
+        case .indent:
+            return VectorL10n.wysiwygComposerFormatActionIndent
+        case .unIndent:
+            return VectorL10n.wysiwygComposerFormatActionUnIndent
         case .inlineCode:
             return VectorL10n.wysiwygComposerFormatActionInlineCode
         case .codeBlock:
@@ -144,6 +158,10 @@ extension FormatType {
             return .unorderedList
         case .orderedList:
             return .orderedList
+        case .indent:
+            return .indent
+        case .unIndent:
+            return .unIndent
         case .inlineCode:
             return .inlineCode
         case .codeBlock:
@@ -171,6 +189,10 @@ extension FormatType {
             return .unorderedList
         case .orderedList:
             return .orderedList
+        case .indent:
+            return .indent
+        case .unIndent:
+            return .unIndent
         case .inlineCode:
             return .inlineCode
         case .codeBlock:

diff --git a/SiriIntents/target.yml b/SiriIntents/target.yml
@@ -34,6 +34,7 @@ targets:
     dependencies:
     - sdk: Intents.framework
     - package: DeviceKit
+    - package: DTCoreText
 
     configFiles:
       Debug: Debug.xcconfig

diff --git a/changelog.d/7316.change b/changelog.d/7316.change
@@ -0,0 +1 @@
+Labs: Rich text editor: enable list items indentation
diff --git a/fastlane/Fastfile b/fastlane/Fastfile
@@ -259,6 +259,8 @@ platform :ios do
     # Generate xcodebuild additional arguments
     xcargs_hash = {
       "GCC_PREPROCESSOR_DEFINITIONS" => "$(GCC_PREPROCESSOR_DEFINITIONS) #{additional_preprocessor_definitions}",
+      # Fix XCode 14 code signing issues for Swift packages containing resources bundles.
+      "CODE_SIGN_STYLE" => "Manual",
     }
 
     xcargs = xcargs_hash.map { |k, v| "#{k}=#{v.shellescape}" }.join(" ")

diff --git a/project.yml b/project.yml
@@ -53,7 +53,10 @@ packages:
     branch: main
   WysiwygComposer:
     url: https://github.com/matrix-org/matrix-wysiwyg-composer-swift
-    version: 0.19.0
+    version: 0.22.0
   DeviceKit:
     url: https://github.com/devicekit/DeviceKit
     majorVersion: 4.7.0
+  DTCoreText:
+    url: https://github.com/Cocoanetics/DTCoreText
+    version: 1.6.27