Closed Bug 1496073 Opened 6 years ago Closed 6 years ago

Notifications entry is displayed in Settings sub-menu

Categories

(Firefox for Android Graveyard :: Settings and Preferences, defect)

Firefox 64
ARM
Android
defect
Not set
normal

Tracking

(firefox64 verified)

VERIFIED FIXED
Firefox 64
Tracking Status
firefox64 --- verified

People

(Reporter: sflorean, Assigned: petru)

Details

Attachments

(2 files)

Environment: 
Device: Samsung Galaxy Tab S3(Android 8.0);
Build: Nightly 64.0a1 (2018-10-02);

Steps to reproduce:
1. Go to three dots menu->Settings;
2. Check the submenus.

Expected result:
Submenu "Notifications" is not displayed in the menu.

Actual result:
Submenu "Notifications" is displayed.

Notes: 
- in about:experiments the options "what's new" and "leanplum" are disabled.
- there is no content if we open the "notifications" menu
Assignee: nobody → petru.lingurar
Status: NEW → ASSIGNED
Followup to bug 1456110 which didn't account for the Settings headers
used on tablets.

Currently the Notification settings screen lets the user enable/disable
two types of notifications, both depending on Switchboard experiments.
If none of those experiments are available for the user, the entire settings
group will be hidden to avoid any confusion.
Keywords: checkin-needed
Pushed by cbrindusan@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/9eff0dcbd18f
[Tablets] Hide Notification settings header when unnecessary; r=jchen
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/9eff0dcbd18f
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64
Flags: qe-verify+
Verified as fixed on latest Nightly build (2018-10-07).
Device: Samsung Galaxy Tab S3(Android 8.0).
Status: RESOLVED → VERIFIED
Flags: qe-verify+
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: