Merge pull request #3580 from Expensify/amechler-sms-tooltip

Remove SMS domains from tooltips
Expensify · Jun 15, 2021 · 7a521b9 · 7a521b9
2 parents 384aa5e + f8691d1
commit 7a521b9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/pages/home/HeaderView.js b/src/pages/home/HeaderView.js
@@ -65,7 +65,7 @@ const HeaderView = (props) => {
 
             return {
                 displayName: (isMultipleParticipant ? firstName : displayNameTrimmed) || Str.removeSMSDomain(login),
-                tooltip: login,
+                tooltip: Str.removeSMSDomain(login),
             };
         },
     );

diff --git a/src/pages/home/sidebar/OptionRow.js b/src/pages/home/sidebar/OptionRow.js
@@ -135,7 +135,7 @@ const OptionRow = ({
 
             return {
                 displayName: (isMultipleParticipant ? firstName : displayNameTrimmed) || Str.removeSMSDomain(login),
-                tooltip: login,
+                tooltip: Str.removeSMSDomain(login),
             };
         },
     );