Skip to content

Commit

Permalink
[web] Hide reply tooltip button for multimedia
Browse files Browse the repository at this point in the history
Summary:
Just took a second to change this, we don't want to check whether the message is composable since that includes both text and multimedia. Now, if the message is not text we will not show the tooltip icon.

Resolves https://linear.app/comm/issue/ENG-4520/reply-button-appears-for-images-on-web

Test Plan:
See below and after to verify that the reply button no longer shows for multimedia

Before:
{F681770}

After:
{F681771}

Reviewers: atul, ginsu

Reviewed By: atul, ginsu

Subscribers: ashoat, tomek

Differential Revision: https://phab.comm.dev/D8766
  • Loading branch information
RohanK6 committed Aug 9, 2023
1 parent e01a2fa commit ab110b7
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion web/utils/tooltip-action-utils.js
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ function useMessageTooltipReplyAction(
const { addReply } = inputState;
return React.useMemo(() => {
if (
!isComposableMessageType(item.messageInfo.type) ||
item.messageInfo.type !== messageTypes.TEXT ||
!threadHasPermission(threadInfo, threadPermissions.VOICED)
) {
return null;
Expand Down

0 comments on commit ab110b7

Please sign in to comment.