From 9b6b71a0c05ae76e3ead1e278e5ec4e5dbc47d2e Mon Sep 17 00:00:00 2001
From: manuroe <manu@matrix.org>
Date: Fri, 9 Mar 2018 17:28:45 +0100
Subject: [PATCH 1/9] BF: Cannot join from a room preview for room with a long
 topic (#1645)

Turn the room topic label into a text view to be able to scroll.
---
 Riot/Views/RoomTitle/PreviewRoomTitleView.h   |  2 +-
 Riot/Views/RoomTitle/PreviewRoomTitleView.m   |  1 -
 Riot/Views/RoomTitle/PreviewRoomTitleView.xib | 24 +++++++++++--------
 3 files changed, 15 insertions(+), 12 deletions(-)

diff --git a/Riot/Views/RoomTitle/PreviewRoomTitleView.h b/Riot/Views/RoomTitle/PreviewRoomTitleView.h
index 619fc90b70..79fdde1bb1 100644
--- a/Riot/Views/RoomTitle/PreviewRoomTitleView.h
+++ b/Riot/Views/RoomTitle/PreviewRoomTitleView.h
@@ -24,7 +24,7 @@
 @property (weak, nonatomic) IBOutlet UIView *mainHeaderContainer;
 
 @property (weak, nonatomic) IBOutlet MXKImageView *roomAvatar;
-@property (weak, nonatomic) IBOutlet UILabel *roomTopic;
+@property (weak, nonatomic) IBOutlet UITextView *roomTopic;
 @property (weak, nonatomic) IBOutlet UILabel *roomMembers;
 @property (weak, nonatomic) IBOutlet UIView *roomMembersDetailsIcon;
 
diff --git a/Riot/Views/RoomTitle/PreviewRoomTitleView.m b/Riot/Views/RoomTitle/PreviewRoomTitleView.m
index 1e8888d460..7b723562b3 100644
--- a/Riot/Views/RoomTitle/PreviewRoomTitleView.m
+++ b/Riot/Views/RoomTitle/PreviewRoomTitleView.m
@@ -67,7 +67,6 @@ -(void)customizeViewRendering
     self.displayNameTextField.textColor = (self.mxRoom.summary.displayname.length ? kRiotPrimaryTextColor : kRiotSecondaryTextColor);
     
     self.roomTopic.textColor = kRiotTopicTextColor;
-    self.roomTopic.numberOfLines = 0;
     
     self.roomMembers.textColor = kRiotColorGreen;
     
diff --git a/Riot/Views/RoomTitle/PreviewRoomTitleView.xib b/Riot/Views/RoomTitle/PreviewRoomTitleView.xib
index 2cb75c1551..802efbe335 100644
--- a/Riot/Views/RoomTitle/PreviewRoomTitleView.xib
+++ b/Riot/Views/RoomTitle/PreviewRoomTitleView.xib
@@ -1,11 +1,11 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<document type="com.apple.InterfaceBuilder3.CocoaTouch.XIB" version="3.0" toolsVersion="13196" targetRuntime="iOS.CocoaTouch" propertyAccessControl="none" useAutolayout="YES" useTraitCollections="YES" colorMatched="YES">
+<document type="com.apple.InterfaceBuilder3.CocoaTouch.XIB" version="3.0" toolsVersion="13771" targetRuntime="iOS.CocoaTouch" propertyAccessControl="none" useAutolayout="YES" useTraitCollections="YES" colorMatched="YES">
     <device id="retina4_7" orientation="portrait">
         <adaptation id="fullscreen"/>
     </device>
     <dependencies>
         <deployment identifier="iOS"/>
-        <plugIn identifier="com.apple.InterfaceBuilder.IBCocoaTouchPlugin" version="13173"/>
+        <plugIn identifier="com.apple.InterfaceBuilder.IBCocoaTouchPlugin" version="13772"/>
         <capability name="Aspect ratio constraints" minToolsVersion="5.1"/>
         <capability name="Constraints with non-1.0 multipliers" minToolsVersion="5.1"/>
         <capability name="documents saved in the Xcode 8 format" minToolsVersion="8.0"/>
@@ -47,22 +47,25 @@
                                 <outlet property="delegate" destination="Cpa-Xt-cwB" id="BIq-nD-Pb9"/>
                             </connections>
                         </textField>
-                        <label opaque="NO" userInteractionEnabled="NO" contentMode="left" horizontalHuggingPriority="251" verticalHuggingPriority="251" text="Label" textAlignment="center" lineBreakMode="tailTruncation" numberOfLines="2" baselineAdjustment="alignBaselines" adjustsFontSizeToFit="NO" translatesAutoresizingMaskIntoConstraints="NO" id="Tk0-pA-9a0">
-                            <rect key="frame" x="282" y="153" width="36" height="17"/>
+                        <textView opaque="NO" contentMode="left" horizontalHuggingPriority="251" verticalHuggingPriority="251" text="Room Topic" textAlignment="center" translatesAutoresizingMaskIntoConstraints="NO" id="Tk0-pA-9a0">
+                            <rect key="frame" x="262" y="153" width="76" height="17"/>
                             <accessibility key="accessibilityConfiguration" identifier="RoomTopic"/>
-                            <fontDescription key="fontDescription" type="system" pointSize="14"/>
+                            <constraints>
+                                <constraint firstAttribute="width" constant="76" id="BEk-1P-gqy"/>
+                            </constraints>
                             <color key="textColor" cocoaTouchSystemColor="darkTextColor"/>
-                            <nil key="highlightedColor"/>
-                        </label>
-                        <label opaque="NO" userInteractionEnabled="NO" contentMode="left" horizontalHuggingPriority="251" verticalHuggingPriority="251" text="Label" textAlignment="center" lineBreakMode="tailTruncation" baselineAdjustment="alignBaselines" adjustsFontSizeToFit="NO" translatesAutoresizingMaskIntoConstraints="NO" id="ou0-3Z-weL">
-                            <rect key="frame" x="282" y="184" width="36" height="17"/>
+                            <fontDescription key="fontDescription" type="system" pointSize="14"/>
+                            <textInputTraits key="textInputTraits"/>
+                        </textView>
+                        <label opaque="NO" userInteractionEnabled="NO" contentMode="left" horizontalHuggingPriority="251" verticalHuggingPriority="251" text="X members" textAlignment="center" lineBreakMode="tailTruncation" baselineAdjustment="alignBaselines" adjustsFontSizeToFit="NO" translatesAutoresizingMaskIntoConstraints="NO" id="ou0-3Z-weL">
+                            <rect key="frame" x="262" y="184" width="76" height="17"/>
                             <accessibility key="accessibilityConfiguration" identifier="RoomMembers"/>
                             <fontDescription key="fontDescription" type="system" weight="medium" pointSize="14"/>
                             <color key="textColor" cocoaTouchSystemColor="darkTextColor"/>
                             <nil key="highlightedColor"/>
                         </label>
                         <imageView hidden="YES" userInteractionEnabled="NO" contentMode="scaleToFill" horizontalHuggingPriority="251" verticalHuggingPriority="251" image="details_icon" translatesAutoresizingMaskIntoConstraints="NO" id="S3Y-wJ-HOe">
-                            <rect key="frame" x="325" y="187" width="6" height="12"/>
+                            <rect key="frame" x="345" y="187" width="6" height="12"/>
                             <accessibility key="accessibilityConfiguration" identifier="RoomMembersDetailsIcon"/>
                             <constraints>
                                 <constraint firstAttribute="width" constant="6" id="XTx-6p-2wB"/>
@@ -166,6 +169,7 @@
                 <constraint firstItem="BkF-x3-7fX" firstAttribute="leading" secondItem="Cpa-Xt-cwB" secondAttribute="leading" id="Uvb-cK-dQf"/>
                 <constraint firstItem="BkF-x3-7fX" firstAttribute="top" secondItem="Cpa-Xt-cwB" secondAttribute="top" id="Zru-9A-ifB"/>
                 <constraint firstAttribute="trailing" secondItem="y5s-FK-My4" secondAttribute="trailing" id="d4g-19-C58"/>
+                <constraint firstItem="BkF-x3-7fX" firstAttribute="bottom" secondItem="y5s-FK-My4" secondAttribute="bottom" id="gzp-8c-3iR"/>
                 <constraint firstAttribute="trailing" relation="greaterThanOrEqual" secondItem="IwA-0X-IYb" secondAttribute="trailing" constant="31" id="iGR-eh-0e9"/>
                 <constraint firstItem="WmW-5h-jL1" firstAttribute="top" secondItem="gIX-nY-f6M" secondAttribute="bottom" constant="17" id="ioD-jo-XQe"/>
                 <constraint firstItem="IwA-0X-IYb" firstAttribute="leading" relation="greaterThanOrEqual" secondItem="Cpa-Xt-cwB" secondAttribute="leading" constant="31" id="lpD-yd-qgt"/>

From 8c9c07531209a364c414228c335669860f9c363c Mon Sep 17 00:00:00 2001
From: manuroe <manu@matrix.org>
Date: Fri, 9 Mar 2018 17:54:35 +0100
Subject: [PATCH 2/9] BF: Cannot join from a room preview for room with a long
 topic (#1645)

Make sure the topic use all container width but keep it centered if the topic string is short
---
 Riot/Views/RoomTitle/PreviewRoomTitleView.xib | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/Riot/Views/RoomTitle/PreviewRoomTitleView.xib b/Riot/Views/RoomTitle/PreviewRoomTitleView.xib
index 802efbe335..5f75dc10bc 100644
--- a/Riot/Views/RoomTitle/PreviewRoomTitleView.xib
+++ b/Riot/Views/RoomTitle/PreviewRoomTitleView.xib
@@ -48,11 +48,8 @@
                             </connections>
                         </textField>
                         <textView opaque="NO" contentMode="left" horizontalHuggingPriority="251" verticalHuggingPriority="251" text="Room Topic" textAlignment="center" translatesAutoresizingMaskIntoConstraints="NO" id="Tk0-pA-9a0">
-                            <rect key="frame" x="262" y="153" width="76" height="17"/>
+                            <rect key="frame" x="31" y="153" width="538" height="17"/>
                             <accessibility key="accessibilityConfiguration" identifier="RoomTopic"/>
-                            <constraints>
-                                <constraint firstAttribute="width" constant="76" id="BEk-1P-gqy"/>
-                            </constraints>
                             <color key="textColor" cocoaTouchSystemColor="darkTextColor"/>
                             <fontDescription key="fontDescription" type="system" pointSize="14"/>
                             <textInputTraits key="textInputTraits"/>
@@ -90,6 +87,7 @@
                         <constraint firstItem="Tk0-pA-9a0" firstAttribute="top" secondItem="6uH-I3-RQg" secondAttribute="bottom" constant="5" id="TUS-xE-O6O"/>
                         <constraint firstItem="6uH-I3-RQg" firstAttribute="top" secondItem="4yt-FK-V2Z" secondAttribute="bottom" constant="8" id="Wsd-KT-hxy"/>
                         <constraint firstAttribute="trailing" relation="greaterThanOrEqual" secondItem="6uH-I3-RQg" secondAttribute="trailing" constant="31" id="aK3-vQ-EVu"/>
+                        <constraint firstItem="Tk0-pA-9a0" firstAttribute="width" secondItem="BkF-x3-7fX" secondAttribute="width" priority="750" id="bWP-2y-KTg"/>
                         <constraint firstItem="ou0-3Z-weL" firstAttribute="leading" relation="greaterThanOrEqual" secondItem="BkF-x3-7fX" secondAttribute="leading" constant="31" id="c9h-h2-VEs"/>
                         <constraint firstItem="6uH-I3-RQg" firstAttribute="leading" relation="greaterThanOrEqual" secondItem="BkF-x3-7fX" secondAttribute="leading" constant="31" id="gnq-cO-l4Y"/>
                         <constraint firstAttribute="trailing" relation="greaterThanOrEqual" secondItem="ou0-3Z-weL" secondAttribute="trailing" constant="31" id="sPZ-Hp-JeH"/>

From c5aab876ad1c0b1942dc496de58fd08be3e9e995 Mon Sep 17 00:00:00 2001
From: manuroe <manu@matrix.org>
Date: Fri, 9 Mar 2018 18:31:17 +0100
Subject: [PATCH 3/9] BF: Cannot join from a room preview for room with a long
 topic (#1645)

If necessary, crop the room name in "You are trying to access %@. Would you like to join in order to participate in the discussion?" so that it does not eat all the screen.
---
 Riot/Views/RoomTitle/PreviewRoomTitleView.m | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/Riot/Views/RoomTitle/PreviewRoomTitleView.m b/Riot/Views/RoomTitle/PreviewRoomTitleView.m
index 7b723562b3..d719a1ba05 100644
--- a/Riot/Views/RoomTitle/PreviewRoomTitleView.m
+++ b/Riot/Views/RoomTitle/PreviewRoomTitleView.m
@@ -147,6 +147,13 @@ - (void)refreshDisplay
             {
                 roomName = NSLocalizedStringFromTable(@"room_preview_try_join_an_unknown_room_default", @"Vector", nil);
             }
+            else if (roomName.length > 20)
+            {
+                // Would have nice to get the cropped string displayed by self.displayNameTextField
+                // but the value is not accessible. Cut it off by hand
+                roomName = [NSString stringWithFormat:@"%@…",[roomName substringToIndex:20]];
+            }
+
             self.previewLabel.text = [NSString stringWithFormat:NSLocalizedStringFromTable(@"room_preview_try_join_an_unknown_room", @"Vector", nil), roomName];
         }
     }

From 3df7e196eb9792c98c4d559e69ae08107f5424c7 Mon Sep 17 00:00:00 2001
From: manuroe <manu@matrix.org>
Date: Mon, 12 Mar 2018 14:59:59 +0100
Subject: [PATCH 4/9] BF: Cannot join from a room preview for room with a long
 topic (#1645)

Update the room preview header height according to the screen height
---
 Riot/ViewController/RoomViewController.m | 28 +++++++++++++++++++++++-
 1 file changed, 27 insertions(+), 1 deletion(-)

diff --git a/Riot/ViewController/RoomViewController.m b/Riot/ViewController/RoomViewController.m
index 004881301d..4afd9fb214 100644
--- a/Riot/ViewController/RoomViewController.m
+++ b/Riot/ViewController/RoomViewController.m
@@ -1605,7 +1605,33 @@ - (void)refreshPreviewHeader:(BOOL)isLandscapeOriented
         {
             previewHeader.mainHeaderContainer.hidden = NO;
             previewHeader.mainHeaderBackgroundHeightConstraint.constant = previewHeader.mainHeaderContainer.frame.size.height;
-            
+
+            if ([previewHeader isKindOfClass:PreviewRoomTitleView.class])
+            {
+                // In case of preview, update the header height so that we can
+                // display as much as possible the room topic in this header.
+                // Note: the header height is the previewHeader.mainHeaderBackgroundHeightConstraint
+                // just computed
+                PreviewRoomTitleView *previewRoomTitleView = (PreviewRoomTitleView *)previewHeader;
+
+                // Compute the height required to display all the room topic
+                CGSize sizeThatFitsTextView = [previewRoomTitleView.roomTopic sizeThatFits:CGSizeMake(previewRoomTitleView.roomTopic.frame.size.width, MAXFLOAT)];
+
+                // Increase the preview header height according to the room topic height
+                // but limit it in order to let room for room messages at the screen bottom.
+                // This free space depends on the device.
+                // On an iphone 5 screen, the room topic height cannot be more than 50px.
+                // Then, on larger screen, we can allow it a bit more height but we
+                // apply a factor to give more priority to the display of more messages.
+                CGFloat screenHeight = [[UIScreen mainScreen] bounds].size.height;
+                CGFloat maxRoomTopicHeight = 50 + (screenHeight - 568) / 3;
+
+                CGFloat additionalHeight = MIN(maxRoomTopicHeight, sizeThatFitsTextView.height)
+                                            - previewRoomTitleView.roomTopic.frame.size.height;
+
+                previewHeader.mainHeaderBackgroundHeightConstraint.constant += additionalHeight;
+            }
+
             [self setRoomTitleViewClass:RoomAvatarTitleView.class];
             // Note the avatar title view does not define tap gesture.
             

From c5d9957be54eaf9a381af13f607269c04e974b13 Mon Sep 17 00:00:00 2001
From: manuroe <manu@matrix.org>
Date: Mon, 12 Mar 2018 15:00:56 +0100
Subject: [PATCH 5/9] BF: Cannot join from a room preview for room with a long
 topic (#1645)

Topic textview: open it on the top of the topic text
---
 Riot/Views/RoomTitle/PreviewRoomTitleView.m   | 4 ++++
 Riot/Views/RoomTitle/PreviewRoomTitleView.xib | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/Riot/Views/RoomTitle/PreviewRoomTitleView.m b/Riot/Views/RoomTitle/PreviewRoomTitleView.m
index d719a1ba05..cc24732969 100644
--- a/Riot/Views/RoomTitle/PreviewRoomTitleView.m
+++ b/Riot/Views/RoomTitle/PreviewRoomTitleView.m
@@ -105,6 +105,10 @@ - (void)refreshDisplay
         
         // Room topic
         self.roomTopic.text = self.roomPreviewData.roomTopic;
+
+        [UIView setAnimationsEnabled:NO];
+        [self.roomTopic scrollRangeToVisible:NSMakeRange(0, 0)];
+        [UIView setAnimationsEnabled:YES];
         
         // Joined members count
         if (self.roomPreviewData.numJoinedMembers > 1)
diff --git a/Riot/Views/RoomTitle/PreviewRoomTitleView.xib b/Riot/Views/RoomTitle/PreviewRoomTitleView.xib
index 5f75dc10bc..31fcca2736 100644
--- a/Riot/Views/RoomTitle/PreviewRoomTitleView.xib
+++ b/Riot/Views/RoomTitle/PreviewRoomTitleView.xib
@@ -47,7 +47,7 @@
                                 <outlet property="delegate" destination="Cpa-Xt-cwB" id="BIq-nD-Pb9"/>
                             </connections>
                         </textField>
-                        <textView opaque="NO" contentMode="left" horizontalHuggingPriority="251" verticalHuggingPriority="251" text="Room Topic" textAlignment="center" translatesAutoresizingMaskIntoConstraints="NO" id="Tk0-pA-9a0">
+                        <textView opaque="NO" contentMode="left" horizontalHuggingPriority="251" verticalHuggingPriority="251" editable="NO" text="Room Topic" textAlignment="center" translatesAutoresizingMaskIntoConstraints="NO" id="Tk0-pA-9a0">
                             <rect key="frame" x="31" y="153" width="538" height="17"/>
                             <accessibility key="accessibilityConfiguration" identifier="RoomTopic"/>
                             <color key="textColor" cocoaTouchSystemColor="darkTextColor"/>

From c87be8150cee5c73665600f7975d514aa0898a98 Mon Sep 17 00:00:00 2001
From: manuroe <manu@matrix.org>
Date: Mon, 12 Mar 2018 15:56:41 +0100
Subject: [PATCH 6/9] BF: Cannot join from a room preview for room with a long
 topic (#1645)

Show the expanded room headers on iPad
---
 Riot/ViewController/RoomViewController.m | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/Riot/ViewController/RoomViewController.m b/Riot/ViewController/RoomViewController.m
index 4afd9fb214..8a737c7801 100644
--- a/Riot/ViewController/RoomViewController.m
+++ b/Riot/ViewController/RoomViewController.m
@@ -1583,7 +1583,8 @@ - (void)refreshPreviewHeader:(BOOL)isLandscapeOriented
 {
     if (previewHeader)
     {
-        if (isLandscapeOriented)
+        if (isLandscapeOriented
+            && [GBDeviceInfo deviceInfo].family == GBDeviceFamilyiPad)
         {
             CGRect frame = self.navigationController.navigationBar.frame;
             

From 1827f619df6cac1c9178a6741efde0b1690b6771 Mon Sep 17 00:00:00 2001
From: manuroe <manu@matrix.org>
Date: Mon, 12 Mar 2018 16:16:30 +0100
Subject: [PATCH 7/9] Room preview: Do not show right buttons (search and
 matrix apps) in the navigation bar, even disabled and grayed

---
 Riot/ViewController/RoomViewController.m | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/Riot/ViewController/RoomViewController.m b/Riot/ViewController/RoomViewController.m
index 8a737c7801..e115197dc1 100644
--- a/Riot/ViewController/RoomViewController.m
+++ b/Riot/ViewController/RoomViewController.m
@@ -1227,11 +1227,8 @@ - (void)refreshRoomTitle
     // Set the right room title view
     if (self.isRoomPreview)
     {
-        // Disable the right buttons
-        for (UIBarButtonItem *barButtonItem in self.navigationItem.rightBarButtonItems)
-        {
-            barButtonItem.enabled = NO;
-        }
+        // Do not show the right buttons
+        self.navigationItem.rightBarButtonItems = nil;
         
         [self showPreviewHeader:YES];
     }

From c8745e2c3471014d6cac82f156f7e2c1c4d7893f Mon Sep 17 00:00:00 2001
From: manuroe <manu@matrix.org>
Date: Mon, 12 Mar 2018 16:18:35 +0100
Subject: [PATCH 8/9] typo

---
 Riot/Views/RoomTitle/PreviewRoomTitleView.m | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/Riot/Views/RoomTitle/PreviewRoomTitleView.m b/Riot/Views/RoomTitle/PreviewRoomTitleView.m
index cc24732969..a06781b479 100644
--- a/Riot/Views/RoomTitle/PreviewRoomTitleView.m
+++ b/Riot/Views/RoomTitle/PreviewRoomTitleView.m
@@ -153,8 +153,9 @@ - (void)refreshDisplay
             }
             else if (roomName.length > 20)
             {
-                // Would have nice to get the cropped string displayed by self.displayNameTextField
-                // but the value is not accessible. Cut it off by hand
+                // Would have beed nice to get the cropped string displayed by
+                // self.displayNameTextField but the value is not accessible.
+                // Cut it off by hand
                 roomName = [NSString stringWithFormat:@"%@…",[roomName substringToIndex:20]];
             }
 

From 11378ea428b5af694a737204ee302730e5e2423a Mon Sep 17 00:00:00 2001
From: manuroe <manu@matrix.org>
Date: Mon, 12 Mar 2018 17:07:51 +0100
Subject: [PATCH 9/9] BF: Cannot join from a room preview for room with a long
 topic (#1645)

Follow Giom's remarks
---
 Riot/ViewController/RoomViewController.m    | 5 ++---
 Riot/Views/RoomTitle/PreviewRoomTitleView.m | 2 +-
 2 files changed, 3 insertions(+), 4 deletions(-)

diff --git a/Riot/ViewController/RoomViewController.m b/Riot/ViewController/RoomViewController.m
index e115197dc1..b434af949a 100644
--- a/Riot/ViewController/RoomViewController.m
+++ b/Riot/ViewController/RoomViewController.m
@@ -1581,7 +1581,7 @@ - (void)refreshPreviewHeader:(BOOL)isLandscapeOriented
     if (previewHeader)
     {
         if (isLandscapeOriented
-            && [GBDeviceInfo deviceInfo].family == GBDeviceFamilyiPad)
+            && [GBDeviceInfo deviceInfo].family != GBDeviceFamilyiPad)
         {
             CGRect frame = self.navigationController.navigationBar.frame;
             
@@ -1608,8 +1608,7 @@ - (void)refreshPreviewHeader:(BOOL)isLandscapeOriented
             {
                 // In case of preview, update the header height so that we can
                 // display as much as possible the room topic in this header.
-                // Note: the header height is the previewHeader.mainHeaderBackgroundHeightConstraint
-                // just computed
+                // Note: the header height is handled by the previewHeader.mainHeaderBackgroundHeightConstraint.
                 PreviewRoomTitleView *previewRoomTitleView = (PreviewRoomTitleView *)previewHeader;
 
                 // Compute the height required to display all the room topic
diff --git a/Riot/Views/RoomTitle/PreviewRoomTitleView.m b/Riot/Views/RoomTitle/PreviewRoomTitleView.m
index a06781b479..6281e750e0 100644
--- a/Riot/Views/RoomTitle/PreviewRoomTitleView.m
+++ b/Riot/Views/RoomTitle/PreviewRoomTitleView.m
@@ -153,7 +153,7 @@ - (void)refreshDisplay
             }
             else if (roomName.length > 20)
             {
-                // Would have beed nice to get the cropped string displayed by
+                // Would have been nice to get the cropped string displayed by
                 // self.displayNameTextField but the value is not accessible.
                 // Cut it off by hand
                 roomName = [NSString stringWithFormat:@"%@…",[roomName substringToIndex:20]];