sp2 adding rooms to spaces

Riot/Assets/en.lproj/Vector.strings

@@ -69,7 +69,11 @@
 "private" = "Private";
 "public" = "Public";
 "stop" = "Stop";
+"new_word" = "New";
+"existing" = "Existing";
+"add" = "Add";
 "ok" = "OK";
+"error" = "Error";
 
 // Call Bar
 "callbar_only_single_active" = "Tap to return to the call (%@)";
@@ -1692,20 +1696,27 @@ Tap the + to start adding people.";
 // MARK: - Create Room
 
 "create_room_title" = "New Room";
-"create_room_section_header_name" = "Room name";
+"create_room_section_header_name" = "NAME";
 "create_room_placeholder_name" = "Name";
-"create_room_section_header_topic" = "Room topic (optional)";
-"create_room_placeholder_topic" = "Topic";
-"create_room_section_header_encryption" = "Room encryption";
+"create_room_section_header_topic" = "TOPIC (OPTIONAL)";
+"create_room_placeholder_topic" = "What is this room about?";
+"create_room_section_header_encryption" = "ENCRYPTION";
 "create_room_enable_encryption" = "Enable Encryption";
 "create_room_section_footer_encryption" = "Encryption can’t be disabled afterwards.";
-"create_room_section_header_type" = "Room type";
-"create_room_type_private" = "Private Room";
-"create_room_type_public" = "Public Room";
-"create_room_section_footer_type" = "People join a private room only with the room invitation.";
-"create_room_show_in_directory" = "Show the room in the directory";
-"create_room_section_header_address" = "Room address";
+"create_room_section_header_type" = "WHO CAN ACCESS";
+"create_room_type_private" = "Private Room (invite only)";
+"create_room_type_restricted" = "Space members";
+"create_room_type_public" = "Public Room (anyone)";
+"create_room_section_footer_type_private" = "Only people invited can find and join.";
+"create_room_section_footer_type_restricted" = "Anyone in Space name can find and join.";
+"create_room_section_footer_type_public" = "Only people invited can find and join, not just people in Space name.";
+"create_room_promotion_header" = "PROMOTION";
+"create_room_show_in_directory" = "Show in room directory";
+"create_room_show_in_directory_footer" = "This will help people find and join.";
+"create_room_section_header_address" = "ADDRESS";
 "create_room_placeholder_address" = "#testroom:matrix.org";
+"create_room_suggest_room" = "Suggest to space members";
+"create_room_suggest_room_footer" = "Suggested rooms are promoted to space members as good to join.";
 
 // MARK: - Room Info
 
@@ -1888,6 +1899,11 @@ Tap the + to start adding people.";
 "spaces_creation_in_many_spaces" = "in %@ spaces";
 "spaces_creation_in_one_space" = "in 1 space";
 
+"spaces_invite_people" = "Invite people";
+"spaces_add_room" = "Add room";
+"spaces_add_room_missing_permission_message" = "You do not have permissions to add rooms to this space.";
+"spaces_add_space" = "Add space";
+
 // Mark: Avatar
 
 "space_avatar_view_accessibility_label" = "avatar";

Riot/Categories/UIViewController.swift

@@ -44,28 +44,50 @@ extension UIViewController {
     /// - Parameters:
     ///   - viewController: The child view controller to add.
     ///   - view: The view on which to add the child view controller view.
-    func vc_addChildViewController(viewController: UIViewController, onView view: UIView) {
+    ///   - animated: true to add a fade in animation
+    func vc_addChildViewController(viewController: UIViewController, onView view: UIView, animated: Bool = false) {
         self.addChild(viewController)
 
         viewController.view.frame = view.bounds
+        if animated {
+            viewController.view.alpha = 0
+        }
         view.vc_addSubViewMatchingParent(viewController.view)
+        if animated {
+            UIView.animate(withDuration: 0.2) {
+                viewController.view.alpha = 1
+            }
+        }
         viewController.didMove(toParent: self)
     }
 
 
     /// Remove a child view controller from current view controller.
     ///
-    /// - Parameter viewController: The child view controller to remove.
-    func vc_removeChildViewController(viewController: UIViewController) {
+    /// - Parameters:
+    ///     - viewController: The child view controller to remove.
+    ///     - animated: true to add a fade out animation
+    func vc_removeChildViewController(viewController: UIViewController, animated: Bool = false) {
         viewController.willMove(toParent: nil)
-        viewController.view.removeFromSuperview()
+        if animated {
+            UIView.animate(withDuration: 0.2) {
+                viewController.view.alpha = 0
+            } completion: { finished in
+                viewController.view.removeFromSuperview()
+                viewController.view.alpha = 1
+            }
+        } else {
+            viewController.view.removeFromSuperview()
+        }
         viewController.removeFromParent()
     }
 
 
     /// Remove current view controller from parent.
-    func vc_removeFromParent() {
-        self.vc_removeChildViewController(viewController: self)
+    ///
+    /// - Parameter animated: true to add a fade out animation
+    func vc_removeFromParent(animated: Bool = false) {
+        self.vc_removeChildViewController(viewController: self, animated: animated)
     }
 
     /// Adds a floating action button to the bottom-right of the page.

Riot/Generated/Strings.swift

@@ -35,6 +35,10 @@ public class VectorL10n: NSObject {
   public static func activeCallDetails(_ p1: String) -> String {
     return VectorL10n.tr("Vector", "active_call_details", p1)
   }
+  /// Add
+  public static var add: String { 
+    return VectorL10n.tr("Vector", "add") 
+  }
   /// Help us identify issues and improve %@ by sharing anonymous usage data. To understand how people use multiple devices, we’ll generate a random identifier, shared by your devices.
   public static func analyticsPromptMessageNewUser(_ p1: String) -> String {
     return VectorL10n.tr("Vector", "analytics_prompt_message_new_user", p1)
@@ -659,54 +663,82 @@ public class VectorL10n: NSObject {
   public static var createRoomPlaceholderName: String { 
     return VectorL10n.tr("Vector", "create_room_placeholder_name") 
   }
-  /// Topic
+  /// What is this room about?
   public static var createRoomPlaceholderTopic: String { 
     return VectorL10n.tr("Vector", "create_room_placeholder_topic") 
   }
+  /// PROMOTION
+  public static var createRoomPromotionHeader: String { 
+    return VectorL10n.tr("Vector", "create_room_promotion_header") 
+  }
   /// Encryption can’t be disabled afterwards.
   public static var createRoomSectionFooterEncryption: String { 
     return VectorL10n.tr("Vector", "create_room_section_footer_encryption") 
   }
-  /// People join a private room only with the room invitation.
-  public static var createRoomSectionFooterType: String { 
-    return VectorL10n.tr("Vector", "create_room_section_footer_type") 
+  /// Only people invited can find and join.
+  public static var createRoomSectionFooterTypePrivate: String { 
+    return VectorL10n.tr("Vector", "create_room_section_footer_type_private") 
   }
-  /// Room address
+  /// Only people invited can find and join, not just people in Space name.
+  public static var createRoomSectionFooterTypePublic: String { 
+    return VectorL10n.tr("Vector", "create_room_section_footer_type_public") 
+  }
+  /// Anyone in Space name can find and join.
+  public static var createRoomSectionFooterTypeRestricted: String { 
+    return VectorL10n.tr("Vector", "create_room_section_footer_type_restricted") 
+  }
+  /// ADDRESS
   public static var createRoomSectionHeaderAddress: String { 
     return VectorL10n.tr("Vector", "create_room_section_header_address") 
   }
-  /// Room encryption
+  /// ENCRYPTION
   public static var createRoomSectionHeaderEncryption: String { 
     return VectorL10n.tr("Vector", "create_room_section_header_encryption") 
   }
-  /// Room name
+  /// NAME
   public static var createRoomSectionHeaderName: String { 
     return VectorL10n.tr("Vector", "create_room_section_header_name") 
   }
-  /// Room topic (optional)
+  /// TOPIC (OPTIONAL)
   public static var createRoomSectionHeaderTopic: String { 
     return VectorL10n.tr("Vector", "create_room_section_header_topic") 
   }
-  /// Room type
+  /// WHO CAN ACCESS
   public static var createRoomSectionHeaderType: String { 
     return VectorL10n.tr("Vector", "create_room_section_header_type") 
   }
-  /// Show the room in the directory
+  /// Show in room directory
   public static var createRoomShowInDirectory: String { 
     return VectorL10n.tr("Vector", "create_room_show_in_directory") 
   }
+  /// This will help people find and join.
+  public static var createRoomShowInDirectoryFooter: String { 
+    return VectorL10n.tr("Vector", "create_room_show_in_directory_footer") 
+  }
+  /// Suggest to space members
+  public static var createRoomSuggestRoom: String { 
+    return VectorL10n.tr("Vector", "create_room_suggest_room") 
+  }
+  /// Suggested rooms are promoted to space members as good to join.
+  public static var createRoomSuggestRoomFooter: String { 
+    return VectorL10n.tr("Vector", "create_room_suggest_room_footer") 
+  }
   /// New Room
   public static var createRoomTitle: String { 
     return VectorL10n.tr("Vector", "create_room_title") 
   }
-  /// Private Room
+  /// Private Room (invite only)
   public static var createRoomTypePrivate: String { 
     return VectorL10n.tr("Vector", "create_room_type_private") 
   }
-  /// Public Room
+  /// Public Room (anyone)
   public static var createRoomTypePublic: String { 
     return VectorL10n.tr("Vector", "create_room_type_public") 
   }
+  /// Space members
+  public static var createRoomTypeRestricted: String { 
+    return VectorL10n.tr("Vector", "create_room_type_restricted") 
+  }
   /// Verify your other devices easier
   public static var crossSigningSetupBannerSubtitle: String { 
     return VectorL10n.tr("Vector", "cross_signing_setup_banner_subtitle") 
@@ -1303,6 +1335,10 @@ public class VectorL10n: NSObject {
   public static var encryptedRoomMessageReplyToPlaceholder: String { 
     return VectorL10n.tr("Vector", "encrypted_room_message_reply_to_placeholder") 
   }
+  /// Error
+  public static var error: String { 
+    return VectorL10n.tr("Vector", "error") 
+  }
   /// Add an identity server in your settings to invite by email.
   public static var errorInvite3pidWithNoIdentityServer: String { 
     return VectorL10n.tr("Vector", "error_invite_3pid_with_no_identity_server") 
@@ -1451,6 +1487,10 @@ public class VectorL10n: NSObject {
   public static func eventFormatterWidgetRemovedByYou(_ p1: String) -> String {
     return VectorL10n.tr("Vector", "event_formatter_widget_removed_by_you", p1)
   }
+  /// Existing
+  public static var existing: String { 
+    return VectorL10n.tr("Vector", "existing") 
+  }
   /// The link %@ is taking you to another site: %@\n\nAre you sure you want to continue?
   public static func externalLinkConfirmationMessage(_ p1: String, _ p2: String) -> String {
     return VectorL10n.tr("Vector", "external_link_confirmation_message", p1, p2)
@@ -2379,6 +2419,10 @@ public class VectorL10n: NSObject {
   public static var networkOfflinePrompt: String { 
     return VectorL10n.tr("Vector", "network_offline_prompt") 
   }
+  /// New
+  public static var newWord: String { 
+    return VectorL10n.tr("Vector", "new_word") 
+  }
   /// Next
   public static var next: String { 
     return VectorL10n.tr("Vector", "next") 
@@ -5375,6 +5419,10 @@ public class VectorL10n: NSObject {
   public static var spacesAddRoom: String { 
     return VectorL10n.tr("Vector", "spaces_add_room") 
   }
+  /// You do not have permissions to add rooms to this space.
+  public static var spacesAddRoomMissingPermissionMessage: String { 
+    return VectorL10n.tr("Vector", "spaces_add_room_missing_permission_message") 
+  }
   /// Adding rooms coming soon
   public static var spacesAddRoomsComingSoonTitle: String { 
     return VectorL10n.tr("Vector", "spaces_add_rooms_coming_soon_title") 

Riot/Modules/Common/Recents/RecentsViewController.m

@@ -1989,7 +1989,8 @@ - (void)createNewRoom
     // Sanity check
     if (self.mainSession)
     {
-        self.createRoomCoordinatorBridgePresenter = [[CreateRoomCoordinatorBridgePresenter alloc] initWithSession:self.mainSession];
+        CreateRoomCoordinatorParameter *parameters = [[CreateRoomCoordinatorParameter alloc] initWithSession:self.mainSession parentSpace: self.dataSource.currentSpace];
+        self.createRoomCoordinatorBridgePresenter = [[CreateRoomCoordinatorBridgePresenter alloc] initWithParameters:parameters];
         self.createRoomCoordinatorBridgePresenter.delegate = self;
         [self.createRoomCoordinatorBridgePresenter presentFrom:self animated:YES];
     }
@@ -2197,6 +2198,12 @@ - (void)createRoomCoordinatorBridgePresenterDelegateDidCancel:(CreateRoomCoordin
     coordinatorBridgePresenter = nil;
 }
 
+- (void)createRoomCoordinatorBridgePresenterDelegate:(CreateRoomCoordinatorBridgePresenter *)coordinatorBridgePresenter didAddRoomsWithIds:(NSArray<NSString *> *)roomIds
+{
+    [coordinatorBridgePresenter dismissWithAnimated:YES completion:nil];
+    coordinatorBridgePresenter = nil;
+}
+
 #pragma mark - Empty view management
 
 - (void)showEmptyViewIfNeeded