Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Simplify comment action menu for most common actions. #26927

Closed
JmillsExpensify opened this issue Sep 7, 2023 · 38 comments
Closed

Simplify comment action menu for most common actions. #26927

JmillsExpensify opened this issue Sep 7, 2023 · 38 comments
Assignees
Labels
Internal Requires API changes or must be handled by Expensify staff NewFeature Something to build that is a new item. Weekly KSv2

Comments

@JmillsExpensify
Copy link

JmillsExpensify commented Sep 7, 2023

Problem

As we increasingly build out our chat-focused features, the amount of available comment actions has increased pretty dramatically from a couple to ten and counting. It's pretty daunting to say the least. Just as importantly, this creates a dynamic where our least common actions have equal screen real estate next to actions that are incredibly common, which introduces decide fatigue when you just want to come into New Expensify, collaborate, and generally get great stuff done.

Solution

In order to reduce decision-fatigue, free up precious real estate and help users find the most common things they'd like to do, let's simplify the comment action menu, as follows:

  • Reduce one-tap emojis down to three (from four). Those emojis will be thumbs up (:+1:), heart (:heart:) and joy (:joy)
  • Keep the emoji picker, thread, mark as unread and edit always visible
  • Move copy message, delete and flag to a three-dot overflow menu

That looks like this for large screens.
image

Overflow menu selected
image

And it ends up looking like this for small screens.
image

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01e41d6a1dad5b7475
  • Upwork Job ID: 1699665316818046976
  • Last Price Increase: 2023-09-07
@JmillsExpensify JmillsExpensify added Weekly KSv2 Internal Requires API changes or must be handled by Expensify staff NewFeature Something to build that is a new item. labels Sep 7, 2023
@JmillsExpensify JmillsExpensify self-assigned this Sep 7, 2023
@melvin-bot
Copy link

melvin-bot bot commented Sep 7, 2023

Current assignee @JmillsExpensify is eligible for the NewFeature assigner, not assigning anyone new.

@melvin-bot
Copy link

melvin-bot bot commented Sep 7, 2023

Job added to Upwork: https://www.upwork.com/jobs/~01e41d6a1dad5b7475

@melvin-bot
Copy link

melvin-bot bot commented Sep 7, 2023

Triggered auto assignment to Contributor Plus for review of internal employee PR - @eVoloshchak (Internal)

@JmillsExpensify
Copy link
Author

Marked this as internal because we'll work with an agency or C+ on this one.

@JmillsExpensify
Copy link
Author

cc @shawnborton @dannymcclain @trjExpensify Any other comments before I move this one forward?

@JmillsExpensify
Copy link
Author

Actually on that note: One doubt I still have is that I currently have three quick emoji actions mocked up for large screens, but because of how we have a dedicated row for small screens, there's just a lot more space for four plus the add emoji icon. Maybe that's ok even though it's technically "inconsistent"?

@trjExpensify
Copy link
Contributor

there's just a lot more space for four plus the add emoji icon. Maybe that's ok even though it's technically "inconsistent"?

Hm, I'm not sure about that inconsistency to be honest. Is one more emoji to stay consistent really that egregious? Especially as you can also right-click on desktop to see the same menu as mobile, so there can be a visible diff on the same screen as well.

@shawnborton
Copy link
Contributor

Hmm it's a fair point. How did we decide only three emojis on desktop and not four?

Either way though I don't think it's a huge deal, but let's just be clear about our decision as I can already anticipate the bug reports coming in.

@dannymcclain
Copy link
Contributor

I also don't think it's a big deal and don't really see it as an "inconsistency" as much as just adapting the UI to the viewport it's being display on. Like if we didn't show emojis on one but did on the other, I would call that an inconsistency.

But I don't have super duper strong feelings either way to be honest. I did want to echo what Shawn said—we should be sure we document the decision to avoid unnecessary bug reports 🐛.

@trjExpensify
Copy link
Contributor

Yeah, I just can't see a super clear reason for it. For example, if the extra emoji on desktop/web causes the menu items to overlap reactions or messages to the left, that would make sense. But I don't think they do, so I would just keep the number of them consistent across all platforms personally.

@JmillsExpensify
Copy link
Author

We're going to add a join thread option to the comment menu, so if we keep 4 emoji and then add that, then we're back to the current amount of options in the main comment action menu. At that point, I kind of question why we should even add an three dot overflow menu. Unless we're saying that that'll automatically go into the overflow menu?

@trjExpensify
Copy link
Contributor

Why wouldn't we put join thread in the overflow menu?

@JmillsExpensify
Copy link
Author

Just speaking for myself, I use that one a bunch.

@trjExpensify
Copy link
Contributor

Same, but I think it's a power user thing. I've never been frustrated with where it is in Slack either, so I think it's fine in the overflow menu.

@melvin-bot melvin-bot bot added the Overdue label Sep 25, 2023
@JmillsExpensify
Copy link
Author

Ok cool. Down with that approach then.

@melvin-bot melvin-bot bot removed the Overdue label Sep 25, 2023
@melvin-bot melvin-bot bot added the Overdue label Oct 3, 2023
@JmillsExpensify
Copy link
Author

We still need to get to this, though admittedly it doesn't feel like a priority right now.

@melvin-bot melvin-bot bot removed the Overdue label Oct 10, 2023
@JmillsExpensify
Copy link
Author

Still feels like low priority.

@JmillsExpensify
Copy link
Author

Same as above. We'll come back to this later.

@melvin-bot melvin-bot bot added the Overdue label Nov 2, 2023
@JmillsExpensify
Copy link
Author

Not a priority

@melvin-bot melvin-bot bot removed the Overdue label Nov 8, 2023
@melvin-bot melvin-bot bot added the Overdue label Nov 20, 2023
@JmillsExpensify
Copy link
Author

@dubielzyk-expensify This might be a good issue for you to lead given that you're doing quite a bit of polish and cleaning up designs through out the app. No worries though if you have a lot going on!

@melvin-bot melvin-bot bot removed the Overdue label Nov 24, 2023
@dubielzyk-expensify
Copy link
Contributor

Happy to take it on. I think we should stick with 3 emojis in both cases personally. Having looked through the different states etc it looks to me like this is the list of actions:

(Emoji’s)
Thumbs up
Heart
Joy
(Action)
Add emoji reaction
Reply in thread
Edit (Your comment only)
Download (Attachment only)
(Overflow)
Mark as unread
Subscribe to thread
Copy to clipboard
Report
Delete comment

With the above it'll be [Emoji 1, Emoji 2, Emoji 3, Add emoji, Action 1, Action 2, overflow]. Whatever don't fit goes in the overflow.

I also took the liberty to clean up the pixels a bit from what we have today to make it less overwhelming and make the tap targets a bit bigger. This also eliminates the space between the actions:
CleanShot 2023-11-28 at 16 35 55

Putting it all together we get:
image
CleanShot 2023-11-28 at 16 38 39@2x

Also suggest we add a new Mark as unread icon to make it clearer. Here's my take:
image

Would love takes from @Expensify/design on this too

@shawnborton
Copy link
Contributor

No hot takes from me, I love all of this and I agree where you landed with your exploration. Nice job!

@quinthar
Copy link
Contributor

Hm, I use "mark unread" all the time, easily 10x more often than the sum of everything else. Can we please not put that behind an overflow?

@quinthar
Copy link
Contributor

"Subscribe to thread" can definitely go (I didn't even know it existed).

@quinthar
Copy link
Contributor

Copy link also seems like a kind of poweruser thing that happens very rarely

@quinthar
Copy link
Contributor

I sorta think we could move "open thread" behind the overflow as well, and just make pressing the chat itself open the thread.

@JmillsExpensify
Copy link
Author

I use "subscribe to thread" and "open thread" pretty much all the time, less so "mark as unread". That said, I could live with "subscribe to thread going behind the three dot overflow menu. I think the issue we're going to face is that we all use products differently.

@JmillsExpensify
Copy link
Author

All in all, I think Jon nailed it!

@dannymcclain
Copy link
Contributor

Love what you've done Jon!

I think both of your updated mark as unread icons are good, but I have a slight preference for the chat bubble one. Would be totally happy with either though!

@melvin-bot melvin-bot bot added the Overdue label Dec 7, 2023
@JmillsExpensify
Copy link
Author

Are we good to move forward with this one? Not exactly a priority but I still think it's nice polish for the chat experience.

@melvin-bot melvin-bot bot removed the Overdue label Dec 13, 2023
@dubielzyk-expensify
Copy link
Contributor

✅ This is ready:


Specced up and ready here:

Your comment (with and without attachment)
CleanShot 2023-12-14 at 11 40 48@2x

Comment from someone (with and without attachment)
CleanShot 2023-12-14 at 11 39 56@2x


New "Reply in thread" icon

I do think we'll need a new "Reply in thread" icon. I'll do that work as a separate ticket and implementation, but I suggest something like this:

image

I've started a thread in Slack for this.

@JmillsExpensify
Copy link
Author

Great call on Reply in thread. This is looking super great!

@dubielzyk-expensify
Copy link
Contributor

Not overdue. Think this is ready though

@melvin-bot melvin-bot bot removed the Overdue label Dec 23, 2023
@JmillsExpensify
Copy link
Author

Yes, I think we're aligned on this point. Would you mind posting a summary in #product in Slack as a final confirmation? Then we can get this assigned out.

@dubielzyk-expensify
Copy link
Contributor

Done and done. Slack link

@JmillsExpensify
Copy link
Author

Looks like you have lots of support in the Slack thread!

@JmillsExpensify
Copy link
Author

@quinthar Should we close this issue? I saw you created a separate one: #33573.

@melvin-bot melvin-bot bot added the Overdue label Jan 12, 2024
@JmillsExpensify
Copy link
Author

Closing since we have a duplicate issue now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Internal Requires API changes or must be handled by Expensify staff NewFeature Something to build that is a new item. Weekly KSv2
Projects
None yet
Development

No branches or pull requests

7 participants