Closed
Bug 1711719
Opened 4 years ago
Closed 3 years ago
Locking browser.newtabpage.enabled doesn't disable the new tab dropdown in preferences
Categories
(Firefox :: Settings UI, defect, P2)
Firefox
Settings UI
Tracking
()
VERIFIED
FIXED
101 Branch
People
(Reporter: mkaply, Assigned: mkaply)
Details
Attachments
(1 file)
48 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-esr91+
|
Details | Review |
If you lock browser.newtabpage.enabled:
pref("browser.newtabpage.enabled", false, locked);
in firefox.js for instance,
It should disable the UI in preferences.
Assignee | ||
Comment 1•3 years ago
|
||
Pushed by mozilla@kaply.com:
https://hg.mozilla.org/integration/autoland/rev/b4eff9f3373a
Disable new tab menulist if preference is locked. r=Gijs,preferences-reviewers
Comment 3•3 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
status-firefox101:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 101 Branch
Assignee | ||
Comment 4•3 years ago
|
||
Comment on attachment 9272121 [details]
Bug 1711719 - Disable new tab menulist if preference is locked. r?Gijs
ESR Uplift Approval Request
- If this is not a sec:{high,crit} bug, please state case for ESR consideration: Parity with 101
- User impact if declined: New tab pref is changeable even when locked (doesn't have an effect)
- Fix Landed on Version: 101
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Very small isolated change.
Attachment #9272121 -
Flags: approval-mozilla-esr91?
Comment 5•3 years ago
|
||
Comment on attachment 9272121 [details]
Bug 1711719 - Disable new tab menulist if preference is locked. r?Gijs
Approved for 91.10esr.
Attachment #9272121 -
Flags: approval-mozilla-esr91? → approval-mozilla-esr91+
Comment 6•3 years ago
|
||
bugherder uplift |
status-firefox-esr91:
--- → fixed
Updated•3 years ago
|
Flags: qe-verify+
Updated•3 years ago
|
QA Whiteboard: [qa-triaged]
Comment 7•3 years ago
•
|
||
Verified fix on 101.0 Firefox and 91.10.0 esr FF versions.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
You need to log in
before you can comment on or make changes to this bug.
Description
•