Skip to content

Commit

Permalink
message action buttons [nfc]: Add TODOs for zulip#2792 and zulip#2793
Browse files Browse the repository at this point in the history
  • Loading branch information
chrisbobbe authored and gnprice committed Nov 5, 2021
1 parent 6180479 commit 9c9cd0b
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions src/action-sheets/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -406,13 +406,15 @@ export const constructMessageActionButtons = ({
buttons.push(shareMessage);
}
if (
// TODO(#2792): Don't show if message isn't editable.
message.sender_id === ownUser.user_id
// Our "edit message" UI only works in certain kinds of narrows.
&& (isStreamOrTopicNarrow(narrow) || isPmNarrow(narrow))
) {
buttons.push(editMessage);
}
if (message.sender_id === ownUser.user_id && messageNotDeleted(message)) {
// TODO(#2793): Don't show if message isn't deletable.
buttons.push(deleteMessage);
}
if (message.id in flags.starred) {
Expand Down

0 comments on commit 9c9cd0b

Please sign in to comment.