Closed Bug 1374496 Opened 7 years ago Closed 7 years ago

Search suggestions tip overlaps the onboarding fox icon

Categories

(Firefox :: New Tab Page, defect, P1)

defect

Tracking

()

RESOLVED WONTFIX
Firefox 56

People

(Reporter: mossop, Assigned: gasolin)

References

Details

Attachments

(7 files)

Attached image Screenshot (deleted) —
Discovered in a brand new profile, when I first opened a new tab I immediately saw the search suggestions tip which overlaps the fox icon making it less likely I'd see the onboarding flow.
Whiteboard: [photon-onboarding] → [photon-onboarding][triage]
hi Michael, could you provide suggestion on this one? thank you very much
Flags: needinfo?(mverdi)
It looks like some of the time the page content is pushed down so it doesn't overlap. I'm not sure what causes it to not push down the page content. My guess is pushing the content down is the desired action and it should be fixed to happen all the time. That would solve our problem here. Additionally, it might be better to either: * Show this tip when we expand the address bar in the tour overlay instead of on the first new tab. or * Show this tip in the overlay IN ADDITION to showing it on this first new tab. Maybe this can be investigated if we have time.
Flags: needinfo?(mverdi)
hi Michael, 1. which tour in the tour overlay you would like to associate with this tip? 2. could you please provide STR for your screen shot? 3. Per dev team's comment, changing tip showing timing could impact search bar team's feature. its high risk. thank you very much
Flags: needinfo?(mverdi)
(In reply to Francis Lee [:frlee] from comment #3) > hi Michael, > > 1. which tour in the tour overlay you would like to associate with this tip? The 57 search tour. > 2. could you please provide STR for your screen shot? 1. Install Nightly with a new profile 2. Open a new tab by clicking the + button * A subsequent attempt resulted in the tip overlapping the fox icon. > 3. Per dev team's comment, changing tip showing timing could impact search > bar team's feature. its high risk. > Yes. I'm investigating what the plans are for this feature and the intended UX to see what our options are.
Flags: needinfo?(mverdi)
Attached image pushup reason (deleted) —
By the Engineer team's triage, seems it's not the tooltip pushed down the viewport, but the global notification box, which is for showing some messages for about add-on. It happened that urlbar's tooltip covered it. If the notification dox doesn't shown or get closed, the fox icon got covered again. (Actually the notification box is not shown in mac version.) The urlbar's tooltip is a popup panel which is designated to show over the original UI, so it may not be a trivial change to let it embed above the viewport to push down the webpage. ni? Verdi for confirming this result. And if no additional possibilities we may just investigate the feasibility two suggestions in comment 2.
Flags: needinfo?(mverdi)
Since this issue block the QA signoff, we need to figure out the right UX ASAP. If there's no final decision yet, does moving the icon far away from the top is an acceptable temp solution? The screenshot moved fox icon double the distance (30px -> 60px) to the top and it won't be overlapped by search tips anymore.
Attachment #8884753 - Flags: ui-review?(mverdi)
Here's another view after stretch the distance (30px -> 48px) to top, while search tip did not pop.
Attachment #8884753 - Flags: ui-review?(mverdi)
Attached image move down the fox icon (deleted) —
I moved down the fox icon (30px -> 48px) so it won't be overlaped by search suggestions tip.
Attachment #8885605 - Flags: ui-review?(mverdi)
From today's meeting, Peter(PM) said "just do it" so it wont block QA testing. And we can still figure out other solution in the next version.
Assignee: nobody → gasolin
Status: NEW → ASSIGNED
Flags: needinfo?(mverdi)
Flags: qe-verify+
Priority: -- → P1
Whiteboard: [photon-onboarding][triage] → [photon-onboarding]
Target Milestone: --- → Firefox 56
(In reply to Fred Lin [:gasolin] from comment #9) > Created attachment 8885605 [details] > move down the fox icon > > I moved down the fox icon (30px -> 48px) so it won't be overlaped by search > suggestions tip. Personally, I think current position for the fox icon is fine. When two not-so-related messages are placed closely like that, at least one has to be in the foreground for the user to focus. It's also looking more visually balance by keeping the top and left padding the same for the icon.
Comment on attachment 8885601 [details] Bug 1374496 - move down the onboarding fox icon so Search suggestions tip wont overlaps the fox icon; https://reviewboard.mozilla.org/r/156460/#review162128
Attachment #8885601 - Flags: review?(dtownsend) → review+
(In reply to Bryant Mao [:bryantmao] from comment #11) > (In reply to Fred Lin [:gasolin] from comment #9) > > Created attachment 8885605 [details] > > move down the fox icon > > > > I moved down the fox icon (30px -> 48px) so it won't be overlaped by search > > suggestions tip. > > Personally, I think current position for the fox icon is fine. When two > not-so-related messages are placed closely like that, at least one has to be > in the foreground for the user to focus. It's also looking more visually > balance by keeping the top and left padding the same for the icon. Thanks for feedback, I'm fine to either land the patch or set this issue as won't fix. let's wait Verdi's UI review.
Verdi, how do you think about change the top distance?
Flags: needinfo?(mverdi)
QA Contact: jwilliams
Blocks: 1354046
Peter and I talked this over and decided it would be best to leave the tour overlay icon at the top of the page as designed. I filed bug 1384050 to prevent the address bar from auto-expanding on focus so it doesn't overlay the tour icon.
Flags: needinfo?(mverdi)
Thanks for making the decision, set this issue as WONTFIX
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Flags: qe-verify+
Resolution: --- → WONTFIX
Whiteboard: [photon-onboarding]
Attachment #8885605 - Flags: ui-review?(mverdi)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: