Closed Bug 1381366 Opened 4 years ago Closed 4 years ago

The onboarding tour notifications and tour dialog close buttons should have a tooltip

Categories

(Firefox :: General, defect, P3)

defect

Tracking

()

VERIFIED FIXED
Firefox 56
Tracking Status
firefox56 --- verified
firefox58 --- verified

People

(Reporter: verdi, Assigned: Fischer)

References

Details

(Whiteboard: [photon-onboarding])

Attachments

(2 files)

Attached image tooltip.png
I inadvertently left out a tooltip to be shown on hover of the close button in the Firefox 56 tour notifications. It should read, "Don't show this again"
Whiteboard: [photon-onboarding] → [photon-onboarding][triage]
Flags: qe-verify+
Priority: P3 → P2
QA Contact: jwilliams
Whiteboard: [photon-onboarding][triage] → [photon-onboarding]
Target Milestone: --- → Firefox 56
Assignee: nobody → fliu
Status: NEW → ASSIGNED
(In reply to Verdi [:verdi] from comment #0)
> Created attachment 8886905 [details]
> tooltip.png
> 
> I inadvertently left out a tooltip to be shown on hover of the close button
> in the Firefox 56 tour notifications. It should read, "Don't show this again"
Hi Verdi,
Just afraid that "Don't show this again" may cause some confusion because the next notification for next tour would still be prompted after click it. Not sure if user would think why this notification is showing again. Let's have a discussion on the next con-call meeting.
Flags: needinfo?(mverdi)
Priority: P2 → P1
based on discussion happened in the weekly meeting with PM/UX, we should see user's feedback before making any further change on the existing design. change it to P3 and it will be revised in Fx57 nightly.
Assignee: fliu → nobody
Status: ASSIGNED → NEW
Priority: P1 → P3
Version: 53 Branch → unspecified
Duplicate of this bug: 1377496
Summary: Fx 56 tour notifications close button should have a tooltip → The onboarding tour notifications close button should have a tooltip
(In reply to Fischer [:Fischer] from comment #1)
> (In reply to Verdi [:verdi] from comment #0)
> > Created attachment 8886905 [details]
> > tooltip.png
> > 
> > I inadvertently left out a tooltip to be shown on hover of the close button
> > in the Firefox 56 tour notifications. It should read, "Don't show this again"
> Hi Verdi,
> Just afraid that "Don't show this again" may cause some confusion because
> the next notification for next tour would still be prompted after click it.
> Not sure if user would think why this notification is showing again. Let's
> have a discussion on the next con-call meeting.

Michelle and I talked this over and decided we should use "Dismiss" instead.
Flags: needinfo?(mverdi)
Assignee: nobody → fliu
Status: NEW → ASSIGNED
Summary: The onboarding tour notifications close button should have a tooltip → The onboarding tour notifications and tour dialog close buttons should have a tooltip
Per the bug 1377283 comment 3, the tour dialog close button's tooltip is "Close".
Comment on attachment 8890693 [details]
Bug 1381366 - The onboarding tour notifications and tour dialog close buttons should have a tooltip,

https://reviewboard.mozilla.org/r/161878/#review167600
Attachment #8890693 - Flags: review?(dtownsend) → review+
Keywords: checkin-needed
Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e8d5f44ef5eb
The onboarding tour notifications and tour dialog close buttons should have a tooltip, r=mossop
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/e8d5f44ef5eb
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
I can confirm that this is fixed.
Status: RESOLVED → VERIFIED
I have verified that the x on the notification has a tooltip on Win 10 x64, Win 7 x32, Ubuntu 16.04 x32, and Mac 10.12 with Firefox 58.
You need to log in before you can comment on or make changes to this bug.