Closed Bug 1439840 Opened 6 years ago Closed 6 years ago

Activity Streams search bar is displayed behind the Tour notification

Categories

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

60 Branch
defect

Tracking

()

VERIFIED FIXED
Firefox 63
Iteration:
63.2 - July 23
Tracking Status
firefox-esr52 --- unaffected
firefox-esr60 --- wontfix
firefox60 --- wontfix
firefox61 --- wontfix
firefox62 --- wontfix
firefox63 --- verified

People

(Reporter: emilghitta, Assigned: Mardak)

References

Details

(Keywords: regression)

Attachments

(3 files)

Attached image Resize.gif
[Affected versions]:
60.0a1 (BuildId:20180220220044)

[Unaffected versions]:
59.0b11 (BuildId:20180219114835)
58.0.2 (BuildId:20180206200532)
52.6.0 esr (BuildId:20180118122319)

[Affected platforms]:
Windows 10 64bit.
macOS 10.13.
Ubuntu 16.04 64bit.

[Steps to reproduce]:
1. Launch Firefox with a clean profile.
2. Access the about:newtab page.
3. Resize the Firefox window.	

[Expected result]:
The window is successfully resized and all the page elements are properly displayed.

[Actual result]:
The Search Bar is displayed under the Firefox tour notification.
It seems that the notification is not dismissed even if the user starts typing inside the search bar. 

The notification gets dismissed if further shrinking the Firefox window.


[Regression range]:
This is a regression:

Last good revision: fe8269ac41f5a40fd1e5a2e06ae193bc9462438c
First bad revision: 22f78ddb57200a9d8b4577629cc24ef7c51d4ed5

Pushlog:
https://hg.mozilla.org/integration/autoland/pushloghtml?fromchange=fe8269ac41f5a40fd1e5a2e06ae193bc9462438c&tochange=22f78ddb57200a9d8b4577629cc24ef7c51d4ed5

[Note]
For further information regarding this issue please observe the attached screencast.
Component: Activity Streams: Newtab → Tours
QA Contact: iulia.cristescu → cristian.comorasu
This affects the Onboarding overlay feature, can anyone from you guys take a peek at this?
Flags: needinfo?(evan)
Flags: needinfo?(chsiang)
Hi Cristian, Evan is no longer with Mozilla. 

Chris, do you know any engineers who are taking onboarding work and can take a look at this?
Flags: needinfo?(evan)
Flags: needinfo?(chsiang)
Flags: needinfo?(chrismore.bugzilla)
Component: Tours → Activity Streams: Newtab
we can take this one, :chsiang
Flags: needinfo?(chrismore.bugzilla)
Priority: -- → P3
uifeedback: Adjust the onboarding "show as dot instead of message" threshold so that it only auto-expands if it would not overlap with the search box (for both narrow and wide layouts -- [ed] although I believe activity stream is responsive enough that it will happen to switch to wide layout before there's enough space to show onboarding message in narrow view)

Wontfix for 62 but we'll want this in for 63.
Iteration: --- → 63.2 - July 23
Keywords: uiwanted
Priority: P3 → P2
Attached image 1365px min width
I shared this picture with bryan: "Then it flips to a dot [when 1px narrower]? Sure"
Assignee: nobody → edilee
Status: NEW → ASSIGNED
Comment on attachment 8990462 [details]
Bug 1439840 - Activity Streams search bar is displayed behind the Tour notification.

https://reviewboard.mozilla.org/r/255534/#review262552

This works as expected, R+ to be as long as we are aware of the possible impact of the speech bubble showing up for less users (if screen sizes are smaller than the threshold)
Attachment #8990462 - Flags: review?(khudson) → review+
(In reply to Kate Hudson :k88hudson from comment #9)
> if screen sizes are smaller than the threshold
Nod. And this will impact the default profile new window behavior. Aaron confirmed that the long term plan is to just get rid of this bubble message as it was a temporary solution to help drive people to the onboarding modal.
Pushed by edilee@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/2a354f4c70b1
Activity Streams search bar is displayed behind the Tour notification. r=k88hudson
Iteration: 63.2 - July 23 → 63.3 - Aug 6
https://hg.mozilla.org/mozilla-central/rev/2a354f4c70b1
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
Iteration: 63.3 - Aug 6 → 63.2 - July 23
I have verified that the issue is no longer reproducible on Windows 10 x64, Arch Linux and Mac 10.13, with the latest Nightly build (63.0a1 Build ID - 20180724100052) installed. Now the "Tour Notification" no longer overlaps the "Search Bar".
Status: RESOLVED → VERIFIED
Component: Activity Streams: Newtab → New Tab Page
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: