wrapped logic in the variable 'isVisibleForMessageType' for this chec…

…k and used it
RocketChat · Dec 22, 2024 · 2e21533 · 2e21533
1 parent d09cfdc
commit 2e21533
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/packages/react/src/views/Message/MessageToolbox.js b/packages/react/src/views/Message/MessageToolbox.js
@@ -75,11 +75,17 @@ export const MessageToolbox = ({
   };
 
   const isAllowedToPin = userRoles.some((role) => pinRoles.has(role));
+
   const isAllowedToEditMessage = userRoles.some((role) =>
     editMessageRoles.has(role)
   )
     ? true
     : message.u._id === authenticatedUserId;
+
+  const isVisibleForMessageType =
+    message.files?.[0].type !== 'audio/mpeg' &&
+    message.files?.[0].type !== 'video/mp4';
+
   const options = useMemo(
     () => ({
       reply: {
@@ -130,10 +136,7 @@ export const MessageToolbox = ({
         id: 'edit',
         onClick: () => handleEditMessage(message),
         iconName: 'edit',
-        visible:
-          isAllowedToEditMessage &&
-          message.files?.[0].type !== 'audio/mpeg' &&
-          message.files?.[0].type !== 'video/mp4',
+        visible: isAllowedToEditMessage && isVisibleForMessageType,
         color: isEditing ? 'secondary' : 'default',
         ghost: !isEditing,
       },