mozilla / policy-templates

Policy Templates for Firefox
Mozilla Public License 2.0
1.15k stars 411 forks source link

Firefox View #1078

Open Jee-Hex opened 8 months ago

Jee-Hex commented 8 months ago

Add a policy for the following:

firefox view icon (enable/disable) browser.tabs.firefox-view (true/false)

This can be set via the Preferences policy. I'm not going to add an explicit policy at this time for Firefox View

Originally posted by @mkaply in https://github.com/mozilla/policy-templates/issues/964#issuecomment-1292189337


browser.tabs.firefox-view and browser.tabs.firefox-view-next were removed in FF123 and the prefs in about:config currently don't control anything (bug 1882114 comment 2). Will you to able to reconsider adding a policy for Firefox View?

mkaply commented 8 months ago

We consider Firefox an integral (and useful) feature in Firefox now.

If a user wants to remove it, they can right click and remove the icon.

Can I ask why you want to remove it?

Jee-Hex commented 8 months ago

Can I ask why you want to remove it?

Because it is a "feature" that prompts users to sign up for a Firefox Account, adds a bunch of whitespaces in the process, and doesn't do anything new (at least for me anyway) on top of the old Library. It's pretty telling when the ~recently closed tabs~ history section in fxview-next has a Show all history button that simply points you back to... the Library?

If a user wants to remove it, they can right click and remove the icon.

How would you expect users to deploy this? I mean I could always modify the default value of browser.uiCustomization.state with something like

{
  "policies": {
    "Preferences": {
        "browser.uiCustomization.state": {
            "Value": "{\"placements\":{\"widget-overflow-fixed-list\":[],\"unified-extensions-area\":[],\"nav-bar\":[\"back-button\",\"forward-button\",\"stop-reload-button\",\"customizableui-special-spring1\",\"urlbar-container\",\"customizableui-special-spring2\",\"save-to-pocket-button\",\"downloads-button\",\"fxa-toolbar-menu-button\",\"unified-extensions-button\"],\"toolbar-menubar\":[\"menubar-items\"],\"TabsToolbar\":[\"tabbrowser-tabs\",\"new-tab-button\",\"alltabs-button\"],\"PersonalToolbar\":[\"import-button\",\"personal-bookmarks\"]},\"seen\":[\"save-to-pocket-button\",\"developer-button\"],\"dirtyAreaCache\":[],\"currentVersion\":19,\"newElementCount\":2}",
            "Status": "default"
        }
    }
  }
}

, but that would also mean I would have to keep up with changes to CustomizableUI.sys.mjs every now and then because the value of browser.uiCustomization.state is empty by default. I also can't just deploy, for example, only the TabsToolbar part because Firefox would just treat the value as invalid.

You do have a policy for something even more minute (the home button) anyway. It would be nice if users can depoly this in a simplier way.

mkaply commented 8 months ago

We're resolving the issue that the Sync login appears on the page, but at this point, we don't have plans to add a policy to remove Firefox View.

I'll add it to the backlog.

FYI, there's a lot of new features coming to that page that will make it better than library (the search is already better).