Alya Abbott

Results 1722 comments of Alya Abbott

Nice, super excited to see progress on this feature! A few questions/comments after playing with the PR: 1. Let's drop the "Loading" tooltip; I think we're better off not showing...

Ah, sorry about the typos when testing -- my bad! OK, I think the only two actual issues to look into are: 1. Let's drop the "Loading" tooltip; I think...

Ah, thanks for the question @sofbe ! It looks like this issue was accidentally closed without all the points being completed: For "Deactivate organization", point (1) has been completed, but...

Please feel free to submit a new PR for any of the above!

> For the first checkbox item, I am not sure how to solve (2). How do you want the tooltip to work? Should it be displayed to the user when...

Looks good to me in terms of implementing the issue description, but see [discussion on CZO](https://chat.zulip.org/#narrow/stream/9-issues/topic/message.20edit.20timer.20text/near/1427592) regarding whether additional changes may be needed in order to merge this PR.

OK, based on the discussion linked above, as a prerequisite to integrating this PR, we should remove all the text on the bottom right of the message UI ("View source",...

> I tested this PR and noticed one thing. We also show "View message source" option when user is allowed to move message. So, when I click on "View message...

> We may also want to update the tooltip of pencil icon (shown to the left of three-dot menu) to "Edit message". Yes, good catch! We should change Edit ->...

I just tested a settings configuration where it looks like the "Move message" option is incorrectly missing from the message menu. Settings: ![Screen Shot 2022-08-30 at 10 22 47 AM](https://user-images.githubusercontent.com/2090066/187503288-50efc510-8a5d-4a4d-8d05-2425b9abfd63.png)...