Setting browser.pocket.enabled to 'false' properly removes the Pocket button. However, it does not remove the "view pocket list" item which can be seen when left-clicking on the bookmarks menu button. Some users find the current behavior alarming. They see a third-party in their browser which they didn't ask to be there and which they can't completely remove, and that worries them.
The normal/supported way to remove Pocket works fine: Right click the button, select "Remove from toolbar". https://support.mozilla.org/en-US/kb/disable-pocket-firefox The .enabled pref doesn't really do anything more, and there really no reason to use it. But we should still probably fix this.
Note that browser.pocket.enabled set to "false" doesn't really disable the feature (the bookmark entry remains fully functional, afaict) but just hides the UI elements. The backend needs to be deactivated as well to satisfy the user's expectation on what this preference actually does.
4 years ago
status-firefox38: --- → unaffected
status-firefox38.0.5: --- → affected
status-firefox39: --- → affected
status-firefox40: --- → affected
Summary: browser.pocket.enabled should fully disable the Pocket integration → browser.pocket.enabled doesnfully disable the Pocket integration
Version: Trunk → 38 Branch
4 years ago
status-firefox38: unaffected → ---
Summary: browser.pocket.enabled doesnfully disable the Pocket integration → browser.pocket.enabled doesn't fully disable the Pocket integration
4 years ago
See Also: → bug 1172126
(In reply to rsx11m from comment #3) > The backend needs to be deactivated as well to satisfy the > user's expectation on what this preference actually does. There is no "backend" to disable. Pocket-related code only runs when the Pocket button is clicked -- so no button, no code. The bug here is simply that some of the integration points (menu entries) only check to see if the Pocket button has been removed or not, and setting .enabled to false essentially just hides the button instead of removing it. We should fix this bug by removing the browser.pocket.enabled pref, and make sure that anyone who has the pref set to false automagically has the button removed as if they had done the "Remove From Toolbar" action.
Will this be done for 39 release? http://mxr.mozilla.org/mozilla-release/source/browser/base/content/browser-places.js#1434 > let hidden = !CustomizableUI.getPlacementOfWidget("pocket-button") || !Services.prefs.getBoolPref("browser.pocket.enabled");
Summary: browser.pocket.enabled doesn't fully disable the Pocket integration → browser.pocket.enabled doesn't fully disable the Pocket integration ("View Pocket List" entry remaining in bookmarks menu)
(In reply to Justin Dolske [:Dolske] from comment #4) > We should fix this bug by removing the browser.pocket.enabled pref, and make > sure that anyone who has the pref set to false automagically has the button > removed as if they had done the "Remove From Toolbar" action. How would this work from the central-deployment perspective? If you want certain preferences disabled by default (e.g., an employer doesn't want their users to use Hello/Loop or Pocket and disables it in a central installation), you'd just set the respective pref (loop.enable or browser.pocket.enabled) and lock it, then you are done. In contrast, browser.uiCustomization.state can't be fixed in this way as it needs to be customizable in a user's profile. Personally, I'd prefer the solution suggested in comment #6, which seems easy enough. It also would be rather confusing if some features have an easy ".enable"/".disable" pref whereas others don't.
Was this fixed as part of mozilla-central changeset 8d96b303d33a from bug 1164942 - do not load pocket's jsm or other scripts until used with extra getter to satisfy tests? I can no longer reproduce in a current 42.0a1 build. Setting browser.pocket.enabled to false doesn't have any immediate effect, but upon restart hides /both/ the Toolbar and Bookmarks menu buttons. Apparently "pocket-button" is no longer loaded in CustomizableUI at startup with the pref set to false and hence the test on it being present fails in BookmarkingUI.updatePocketItemVisibility() later. FWIW, I've tested the proposed fix in comment #6 and it works for the purpose of hiding the Bookmarks menuitem unconditionally, but that solution may be obsolete now with the fix from the other bug.
Ok, it's no longer reproducible in the current 39.0 release either, so that's probably a WFM now.
With the browser.pocket.enabled set to 'false', a Pocket button still appears in Reader View on the left sidebar. This button should probably be hidden as well.
Resolved per whiteboard and Comment 10
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → WORKSFORME
There is another case in which setting browser.pocket.enabled to false does not remove the pocket entry from the bookmark menu: see bug 1197871. Does not involve previous use of beta versions. What does help is: set browser.pocket.enabled to true, restart Firefox, manually add the icon to the toolbar, manually remove it from the toolbar.
Question: I have been having a similar issue where extensions.pocket.enabled = false does not completely disable Pocket. In particular new web requests to getpocket.com continue to be made by Firefox even though I thought I disabled it. I noticed you guys are talking about a different config: browser.pocket.enabled. I don't see such a setting in my about:config. Was this setting removed? I'm using Firefox 57 Nightly. Thanks! Ryan P.S. Sorry if this isn't the right place to ask this question.
Oh, nevermind, read a bit more of this thread and answered my own question. Thanks!
In Firefox 59.0b4, the "View Pocket List" entry appears in the "View History, Saved Bookmarks, and More" button. This menu doesn't provide any clear way to edit it.
You need to log in before you can comment on or make changes to this bug.