Merge pull request #3989 from matrix-org/t3chguy/olympics

Apply the Olympic effect to SAS Emoji Verification

res/css/views/right_panel/_UserInfo.scss

@@ -211,7 +211,7 @@ limitations under the License.
         padding-bottom: 16px;
     }
 
-    .mx_UserInfo_scrollContainer:not(.mx_UserInfo_separator) {
+    .mx_UserInfo_container:not(.mx_UserInfo_separator) {
         padding-top: 16px;
         padding-bottom: 0;
 

res/css/views/verification/_VerificationShowSas.scss

@@ -28,21 +28,38 @@ limitations under the License.
 
 .mx_VerificationShowSas_emojiSas {
     text-align: center;
+    display: flex;
+    flex-wrap: wrap;
+    justify-content: center;
+    margin: 25px 0;
 }
 
 .mx_VerificationShowSas_emojiSas_block {
     display: inline-block;
-    margin-left: 15px;
+    margin-bottom: 30px;
-    margin-right: 15px;
+    position: relative;
-    text-align: center;
+    // allow the blocks to grow to space themselves evenly up to a limit of 100px
-    margin-bottom: 20px;
+    flex-grow: 1;
+    max-width: 100px;
 }
 
 .mx_VerificationShowSas_emojiSas_emoji {
     font-size: 48px;
 }
 
-.mx_VerificationShowSas_emojiSas_label {
+.mx_UserInfo .mx_VerificationShowSas_emojiSas_emoji {
-    text-align: center;
+    font-size: 32px; // in UserInfo use a smaller font-size to fit in a smaller space
-    font-weight: bold;
+}
+
+.mx_VerificationShowSas_emojiSas_label {
+    font-weight: bold;
+    // allow the text to overflow the parent by 15px on each side
+    // this is to keep the width of the parent consistent for spacing centrally via flexbox
+    position: absolute;
+    left: -15px;
+    right: -15px;
+}
+
+.mx_VerificationShowSas_emojiSas_break {
+    flex-basis: 100%;
 }

src/components/views/verification/VerificationShowSas.js

@@ -62,7 +62,9 @@ export default class VerificationShowSas extends React.Component {
                 </div>,
             );
             sasDisplay = <div className="mx_VerificationShowSas_emojiSas">
-                {emojiBlocks}
+                {emojiBlocks.slice(0, 4)}
+                <div className="mx_VerificationShowSas_emojiSas_break" />
+                {emojiBlocks.slice(4)}
             </div>;
             sasCaption = _t(
                 "Verify this user by confirming the following emoji appear on their screen.",
@@ -105,8 +107,8 @@ export default class VerificationShowSas extends React.Component {
 
         return <div className="mx_VerificationShowSas">
             <p>{sasCaption}</p>
-            <p>{_t("To be secure, do this in person or use a trusted way to communicate.")}</p>
             {sasDisplay}
+            <p>{_t("To be secure, do this in person or use a trusted way to communicate.")}</p>
             {confirm}
         </div>;
     }