compose: In content-input placeholder, remove @ for 1:1 DMs, to match…

… web In web, the compose box for 1:1 DMs has a placeholder like "Message Tim Abbott", not "Message @tim Abbott". Update mobile to match.
chrisbobbe · Jan 4, 2024 · ecfd8d4 · ecfd8d4
1 parent b5e8c42
commit ecfd8d4
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/compose/__tests__/getComposeInputPlaceholder-test.js b/src/compose/__tests__/getComposeInputPlaceholder-test.js
@@ -16,12 +16,12 @@ describe('getComposeInputPlaceholder', () => {
   const ownUserId = eg.selfUser.user_id;
   const streamsById = getStreamsById(eg.plusReduxState);
 
-  test('returns "Message @ThisPerson" object for person narrow', () => {
+  test('returns "Message This Person" object for person narrow', () => {
     const narrow = deepFreeze(pm1to1NarrowFromUser(eg.otherUser));
     const placeholder = getComposeInputPlaceholder(narrow, ownUserId, usersById, streamsById);
     expect(placeholder).toEqual({
       text: 'Message {recipient}',
-      values: { recipient: `@${eg.otherUser.full_name}` },
+      values: { recipient: eg.otherUser.full_name },
     });
   });
 

diff --git a/src/compose/getComposeInputPlaceholder.js b/src/compose/getComposeInputPlaceholder.js
@@ -28,7 +28,7 @@ export default (
 
         return {
           text: 'Message {recipient}',
-          values: { recipient: `@${user.full_name}` },
+          values: { recipient: user.full_name },
         };
       },
       stream: streamId => {