I have hidden the edit option for the audio/video message, and revert…

…ed previous approach of showing popup message
RocketChat · Dec 19, 2024 · 0b2ef3e · 0b2ef3e
1 parent 884dab4
commit 0b2ef3e
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 19 deletions.
diff --git a/packages/react/src/views/Message/Message.js b/packages/react/src/views/Message/Message.js
@@ -114,23 +114,6 @@ const Message = ({
     }
   };
 
-  const handleEditMessage = () => {
-
-    if(message.msg == '' && message.files[0]) {
-      dispatchToastMessage({
-        type: 'error',
-        message: 'Editing Audio/Video/File messages is not supported at the moment.',
-      });
-      return;
-    }
-
-    if (editMessage._id === message._id) {
-      setEditMessage({});
-    } else {
-      setEditMessage(message);
-    }
-  };
-
   const handleDeleteMessage = async (msg) => {
     const res = await RCInstance.deleteMessage(msg._id);
 
@@ -230,7 +213,13 @@ const Message = ({
                     handleDeleteMessage={handleDeleteMessage}
                     handleStarMessage={handleStarMessage}
                     handlePinMessage={handlePinMessage}
-                    handleEditMessage={handleEditMessage}
+                    handleEditMessage={() => {
+                      if (editMessage._id === message._id) {
+                        setEditMessage({});
+                      } else {
+                        setEditMessage(message);
+                      }
+                    }}
                     handleQuoteMessage={() => addQuoteMessage(message)}
                     handleEmojiClick={handleEmojiClick}
                     handlerReportMessage={() => {

diff --git a/packages/react/src/views/Message/MessageToolbox.js b/packages/react/src/views/Message/MessageToolbox.js
@@ -120,7 +120,7 @@ export const MessageToolbox = ({
         id: 'edit',
         onClick: () => handleEditMessage(message),
         iconName: 'edit',
-        visible: message.u._id === authenticatedUserId,
+        visible: (message.u._id === authenticatedUserId) && (message.files?.[0].type !== 'audio/mpeg') && (message.files?.[0].type !== 'video/mp4'),
         color: isEditing ? 'secondary' : 'default',
         ghost: !isEditing,
       },