Skip to content

[HOLD for payment 2024-04-03][$250] Chat - Missing Tooltip for Video Expand Button #36890

Closed

Description

@lanitochka17

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


Version Number: 1.4.43-2
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers): abebemiherat@gmail.com
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:

Action Performed:

1, Navigate to the conversion section
2, Add a video. The expand button appears alongside the video

Expected Result:

The expand button for videos should display a tooltip

Actual Result:

The expand button for videos does not display a tooltip

Workaround:

Unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6385596_1708427639159.Screen_Recording_2024-02-19_at_10.59.45_PM.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01e702a28b240448c2
  • Upwork Job ID: 1759949924144463872
  • Last Price Increase: 2024-02-20
  • Automatic offers:
    • ntdiary | Reviewer | 0
    • neonbhai | Contributor | 0
Issue OwnerCurrent Issue Owner: @muttmuure

Activity

  1. added
    DeployBlockerCashThis issue or pull request should block deployment
    ExternalAdded to denote the issue can be worked on by a contributor
    on Feb 20, 2024
  2. melvin-bot commented on Feb 20, 2024

    @melvin-bot
  3. changed the title [-]Chat - Missing Tooltip for Video Expand Button[/-] [+][$500] Chat - Missing Tooltip for Video Expand Button[/+] on Feb 20, 2024
  4. added
    Help WantedApply this label when an issue is open to proposals by contributors
    on Feb 20, 2024
  5. melvin-bot commented on Feb 20, 2024

    @melvin-bot

    Triggered auto assignment to Contributor-plus team member for initial proposal review - @ntdiary (External)

  6. github-actions commented on Feb 20, 2024

    @github-actions
    Contributor

    👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

    1. Identify the pull request that introduced this issue and revert it.
    2. Find someone who can quickly fix the issue.
    3. Fix the issue yourself.
  7. melvin-bot commented on Feb 20, 2024

    @melvin-bot

    Triggered auto assignment to @arosiclair (Engineering), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

  8. MahmudjonToraqulov commented on Feb 20, 2024

    @MahmudjonToraqulov
    Contributor

    Proposal

    Please re-state the problem that we are trying to solve in this issue.

    Chat - Missing Tooltip for Video Expand Button

    What is the root cause of that problem?

    We are missing the logic for showing a tooltip.

    What changes do you think we should make in order to solve the problem?

    We should wrap the icon here with Tooltip:

     <Tooltip text={translate('videoPlayer.expand')}>
     ...
     </Tooltip>

    What alternative solutions did you explore? (Optional)

    N/A

  9. 55 remaining items

  10. muttmuure commented on Apr 10, 2024

    @muttmuure
    Contributor

    @ntdiary - $250 C+
    @neonbhai - $250 C

  11. muttmuure commented on Apr 10, 2024

    @muttmuure
    Contributor

    @neonbhai paid, just waiting for @ntdiary to accept offer

  12. ntdiary commented on Apr 11, 2024

    @ntdiary
    Contributor

    @neonbhai paid, just waiting for @ntdiary to accept offer

    @muttmuure, Thank you! Accepted it. 😄

  13. arosiclair commented on Apr 12, 2024

    @arosiclair
    Contributor

    Sounds like this is done ^

  14. ntdiary commented on Sep 6, 2024

    @ntdiary
    Contributor

    Hi, @muttmuure, I just noticed that this issue hasn't been paid yet.
    Since I've switched to NewDot for payments and no longer use upwork. Can I request payment through NewDot? 😂

  15. ntdiary commented on Sep 10, 2024

    @ntdiary
    Contributor

    Hi, @muttmuure, I just noticed that this issue hasn't been paid yet. Since I've switched to NewDot for payments and no longer use upwork. Can I request payment through NewDot? 😂

    Hi, @arosiclair, can you please reopen this issue so we can process the payment? 😄
    image

    cc @muttmuure

  16. mallenexpensify commented on Sep 17, 2024

    @mallenexpensify
    Contributor

    @ntdiary can you please accept the job and reply here once you have?
    https://www.upwork.com/jobs/~021836106341219430298

    Your assigned date on this issue was a handful of days before your eligibility date
    image

    image
  17. ntdiary commented on Sep 18, 2024

    @ntdiary
    Contributor

    @mallenexpensify, Thank you! I have accepted the new job, and ended the previous contract. 😄

  18. muttmuure commented on Sep 18, 2024

    @muttmuure
    Contributor

    Paid - sorry for the oversight!

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 productionBugSomething is broken. Auto assigns a BugZero manager.DailyKSv2InternalRequires API changes or must be handled by Expensify staff

Type

No type

Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions