Skip to content

[HOLD for payment 2022-05-23] [$1000] Reduce Tooltip over Group chats (containing multiple members) in LHN #7486

Closed

Description

@mountiny

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Action Performed:

  1. Have a group chat with many members with potentially long emails.
  2. Hover over the group avatars in LHN and observe a long tooltip shown.
  3. This tooltip takes large part of the screen.

Expected Result:

The tooltip is ended with ellipsis after a character limit is met (so the tooltip does not span the entire page).

For the group chat tooltips, we would like to show this as a box, let's give it max width (same as LHN width I would propose and max height (half of the max-width). The dimensions are still matter of discussion and we could decide to update it once we will see the end result.

If the emails do not fit this box tooltip, clip it using ellipsis: ...

Actual Result:

For a large groupchats the tooltip is one line which is very wide.

Workaround:

Can the user still use Expensify without this being fixed? Have you informed them of the workaround?

Platform:

Where is this issue occurring?

  • Web
  • Desktop App

Version Number:
Reproducible in staging?: N
Reproducible in production?: N
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by: @mountiny
Slack conversation: Coming from #7459

View all open jobs on GitHub

Activity

  1. self-assigned this
    on Jan 31, 2022
  2. mountiny commented on Feb 9, 2022

    @mountiny
    ContributorAuthor

    This is still waiting for the PR to be merged. Not Overdue.

  3. mountiny commented on Feb 15, 2022

    @mountiny
    ContributorAuthor

    Still on hold

  4. changed the title [-]The tooltip over groupchats in LHN with many members is very wide creating not a great UX.[/-] [+][HOLD] The tooltip over groupchats in LHN with many members is very wide creating not a great UX.[/+] on Feb 15, 2022
  5. changed the title [-][HOLD] The tooltip over groupchats in LHN with many members is very wide creating not a great UX.[/-] [+]The tooltip over groupchats in LHN with many members is very wide creating not a great UX.[/+] on Mar 4, 2022
  6. added
    ExternalAdded to denote the issue can be worked on by a contributor
    on Mar 4, 2022
  7. MelvinBot commented on Mar 4, 2022

    @MelvinBot
    Contributor

    Triggered auto assignment to @jliexpensify (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

  8. 94 remaining items

  9. changed the title [-][$1000] Reduce Tooltip over Group chats (containing multiple members) in LHN[/-] [+][HOLD for payment 2022-05-23] [$1000] Reduce Tooltip over Group chats (containing multiple members) in LHN[/+] on May 16, 2022
  10. melvin-bot commented on May 16, 2022

    @melvin-bot

    The solution for this issue has been 🚀 deployed to production 🚀 in version 1.1.60-3 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

    If no regressions arise, payment will be issued on 2022-05-23. 🎊

  11. Beamanator commented on May 25, 2022

    @Beamanator
    Contributor

    Looks like payment is due, has it happened yet @MitchExpensify ?

  12. parasharrajat commented on May 26, 2022

    @parasharrajat
    Member

    @Beamanator MitchExpensify is OOO so we need someone else here.

  13. Beamanator commented on May 26, 2022

    @Beamanator
    Contributor

    Good call @parasharrajat - Mitch is out at least another week, I'll ping internal channels to get a new CM assigned

  14. mallenexpensify commented on May 26, 2022

    @mallenexpensify
    Contributor

    Paid @LucioChavezFuentes $1k for the job.
    Hired @parasharrajat to pay $1k for C+, let me know when you've accepted Rajat
    https://www.upwork.com/jobs/~01aab77c791969cfcd

  15. parasharrajat commented on May 26, 2022

    @parasharrajat
    Member
  16. mallenexpensify commented on May 26, 2022

    @mallenexpensify
    Contributor

    Paid Rajat $1k for C+, closing.
    Thanks for the help Lucio and Rajat!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

Awaiting PaymentAuto-added when associated PR is deployed to productionDailyKSv2DesignEngineeringExternalAdded to denote the issue can be worked on by a contributor

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions