Closed Bug 1377017 Opened 7 years ago Closed 7 years ago

[Onboarding] The Activity Stream option menu is above the onboarding overlay

Categories

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

enhancement

Tracking

()

VERIFIED FIXED
Firefox 57
Tracking Status
firefox56 --- fixed

People

(Reporter: gasolin, Assigned: gasolin)

Details

(Whiteboard: [photon-onboarding])

Attachments

(2 files)

When Activity Stream is enabled and the onboarding overlay is displayed, (when user click the fox icon),

When the user hover on the top right side, the user can still see the Activity Stream option menu.


It is caused by the Activity Stream option menu's z-index level is higher (12001) than the onboarding overlay's (999).
Flags: qe-verify+
Priority: -- → P1
Whiteboard: [photon-onboarding]
Target Milestone: --- → Firefox 57
Attachment #8882058 - Flags: review?(rexboy) → review+
Comment on attachment 8882058 [details]
Bug 1377017 - [Onboarding] The Activity Stream option menu is above the onboarding overlay;

https://reviewboard.mozilla.org/r/153142/#review158366
Attachment #8882058 - Flags: review?(dtownsend) → review+
Comment on attachment 8882058 [details]
Bug 1377017 - [Onboarding] The Activity Stream option menu is above the onboarding overlay;

https://reviewboard.mozilla.org/r/153142/#review158370

This patch is fine.
We may need to at least let Activity Stream's people know (or know their prefer) about usage of z-index.
Thanks for reminding, sent the note to the activity stream channel on slack https://mozilla.slack.com/messages/C4CGG9L9W/
Assignee: nobody → gasolin
Keywords: checkin-needed
Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0dfd83d33bfb
[Onboarding] The Activity Stream option menu is above the onboarding overlay;r=mossop,rexboy
Keywords: checkin-needed
QA Contact: jwilliams
Status: NEW → ASSIGNED
https://hg.mozilla.org/mozilla-central/rev/0dfd83d33bfb
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
This issue has been verified on Win 10 x64.
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: