[HOLD for payment 2021-11-04] Add Instant tooltip & add tooltip to absolute positioned icons.

[parasharrajat]

If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!

---

cc: @mountiny

This issue is created based on the discussion here.

1. It will add support for Instant tooltips Add Tooltips to all non-text actions/buttons in New Expensify #4499 (comment).
2. I already added support for absolute positioned targets in Tooltip (Add edit icon #5346) & this issue is created to compensate that extra effort as well as add missing tooltips Add Tooltips to all non-text actions/buttons in New Expensify #4499 (comment).

Action Performed:

1. Go to NewDot on the web.
2. Hover over any icon that has a tooltip, observe that tooltip is shown with animation.
3. Now move your cursor to other icons tooltip should be shown instantly without animation.
4. Move away from your cursor from the tooltip target for > 1sec.
5. Move back to any target and now tooltip should be shown with animation.

Expected Result:

1. Instant effect explained in the Actions.
2. Add missing tooltips.

Actual Result:

1. every tooltip is shown with animation-delay.
2. Missing tooltip from avatar pencil icon and etc.

Workaround:

UX thing...

Platform:

Where is this issue occurring?

• Web
• Desktop App

Version Number: v1.99.x
Reproducible in staging?: Yes
Reproducible in production?: yes
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by: @parasharrajat
Slack conversation:

View all open jobs on GitHub

[MelvinBot]

Triggered auto assignment to @kevinksullivan (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

[mountiny]

Taking this issue on as we have already discussed this with Rajat in previous issue.

@parasharrajat Can you please write up a formal proposal?

I think this issue is not so important in terms of our deadline so I would say feel free to work on this, but we would not merge it before 9/24 most likely (unless we will decide we want this in the app). As it is a bit larger UI change, I would prefer to leave it for after the deadline in case some regressions show up. Please, excuse any delays which might occur with issue. Thank you for understanding 🙇

cc @shawnborton as you have proposed the instant switching of tooltips here #4499 (comment)

[shawnborton]

Ah I simply mentioned that I noticed another app that did instant tooltip launching after at least one tooltip had been launched. Not necessarily suggesting we need to do the same here, but it might be a nice-to-have.

[mountiny]

No worries. I agree it is nice-to-have and Rajat quickly found a solution for it so it is worthy update. However, not a priority for n6 for sure.

[mountiny]

As per other issue, since we are now focusing on urgent N6 deadline, I will add Weekly label, so MelvinBot does not ping us here. I will put it back to normal next week, when we can focus on this again! Thank you for understanding 🙇

[mountiny]

Still on hold.

[mountiny]

This is still on hold, but @parasharrajat feel free to create the PR for this if you will have time and I can review. We will export this issue soon.

[mountiny]

PR is moving ahead 🎉

[mountiny]

The PR is being reviewed, but still on hold.

[mountiny]

I realized, there was not n6-hold label on this issue. As this has not been requested part of n6 polish, we will wait with merging this code after the hold is lifted, just to avoid any problems before the release.

[parasharrajat]

@mountiny This is not exported yet.

[mountiny]

@parasharrajat Thank you for a bump, I will try to get this sorted.

[MelvinBot]

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

[mountiny]

@MitchExpensify Can we please export this job on Upwork and hire @parasharrajat for it? Thank you very much for your help and time 🙇

[dylanexpensify]

On it!

[dylanexpensify]

Internal: https://www.upwork.com/ab/applicants/1451213441358077952/job-details
External: https://www.upwork.com/jobs/~017c55caffa4d61530

[mountiny]

@parasharrajat Feel free to apply for the Upwork job!

[parasharrajat]

Thanks for the ping @mountiny. @dylanexpensify Done.

[dylanexpensify]

@parasharrajat offer sent

[mountiny]

Lovely to work with yous ❤️

[MitchExpensify]

Thanks for grabbing while I was at SuiteWorld @dylanexpensify! I can take it from here (i.e. payment when ready)

[dylanexpensify]

Woooot! No worries @MitchExpensify!

[parasharrajat]

Ping for

[MitchExpensify]

Paid with bonus!

[parasharrajat]

@dylanexpensify This was on N6 Hold #5347 (comment).

[parasharrajat]

@MitchExpensify I think you missed the N6-hold bonus for this one. And also closing it. Thanks.

[MitchExpensify]

Ah I see what happened @parasharrajat : This job was paid including the n6 hold bonus but I missed the reporting bonus paying that now!

[MitchExpensify]

Paid!