Waterfox icon indicating copy to clipboard operation
Waterfox copied to clipboard

Restart menu item missing in 5.1.6

Open Ketrel opened this issue 2 years ago • 10 comments

What happened?

As of 5.1.6, the "Restart (Developer)" item in the file menu item is missing.

This was not tested on Firefox, as I don't recall this being a Firefox feature to begin with

Reproducible?

  • [x] I have tested this on Firefox.

Version

G5

What platform are you seeing the problem on?

Windows

Relevant log output

No response

EDIT: it looks like it's incomplete unless I mark that I tested in firefox, which doesn't make any sense in this context, but firefox doesn't have it either, but it didn't used to like waterfox did.

Ketrel avatar May 15 '23 21:05 Ketrel

This continues to be the case in 5.1.7

Ketrel avatar May 20 '23 23:05 Ketrel

I can confirm this. Even the keyboard shortcut Ctrl + Alt + R is gone.

thunderstone135 avatar May 21 '23 12:05 thunderstone135

In the Settings menu there's an option "Show restart button in PanelUI" (or something similar, translated), indicating that the underlying method for this is still available.

However, since I've hidden the hamburger menu in the top right via some css, this change is indeed semi-optimal. Being able to quickly restart when experimenting with different addons by pressing ALT, then selecting File → Restart was pretty neat.

Any word on why this option was removed and if we can get it back?

P.S.: I tried looking for installers for older versions here (as indicated on the official download page) but it seems like this has been neglected and there is not a single G5 version on there, leaving (potentially sketchy) third-party websites as the only option?

Hurri04 avatar Jun 04 '23 12:06 Hurri04

On Windows:

image

On macOS:

image

The keyboard shortcut will be missing, as MOZILLA_OFFICIAL was defined as it should have been in the first release of G5. Essentially, it disables all "developer" focused options, and the keyboard shortcut was part of that.

Can I just confirm it's just the keyboard shortcut missing?

MrAlex94 avatar Jun 05 '23 19:06 MrAlex94

Does this MOZILLA_OFFICIALflag affect Windows and macOS differently? Because on Windows the "Restart" menu item is missing from the File menu, as opposed to your macOS screenshot: RestartMissing

The "Restart" option from your Windows screenshot seems to be controlled by the first checkbox here: RestartSetting

Hurri04 avatar Jun 05 '23 19:06 Hurri04

Can I just confirm it's just the keyboard shortcut missing?

Both keyboard shortcut (Ctrl + Alt + R) and File menu item [File > Restart (Developer)] are gone.

thunderstone135 avatar Jun 05 '23 22:06 thunderstone135

Can I just confirm it's just the keyboard shortcut missing?

Both keyboard shortcut (Ctrl + Alt + R) and File menu item [File > Restart (Developer)] are gone.

Yes, Restart (Developer) was not meant to be shown at all, it is meant for "local" builds.

@Hurri04 not sure, I suppose the File menu is kind of forgotten on Windows, which is why we added it to the Panel menu instead.

MrAlex94 avatar Jun 06 '23 10:06 MrAlex94

Yes, Restart (Developer) was not meant to be shown at all, it is meant for "local" builds.

If the file menu item will no longer be available in Waterfox, can you at least restore the keyboard shortcut?

thunderstone135 avatar Jun 06 '23 13:06 thunderstone135

Update:

I noticed that this entry is missing in omni.ja 👇

 * License, v. 2.0. If a copy of the MPL was not distributed with this
 * file, You can obtain one at http://mozilla.org/MPL/2.0/. */

/**
 * Extra features for local development. This file isn't loaded in
 * non-local builds.
 */

var DevelopmentHelpers = {
  init() {
    this.quickRestart = this.quickRestart.bind(this);
    this.addRestartShortcut();
  },

  quickRestart() {
    Services.obs.notifyObservers(null, "startupcache-invalidate");

    let env = Cc["@mozilla.org/process/environment;1"].getService(
      Ci.nsIEnvironment
    );
    env.set("MOZ_DISABLE_SAFE_MODE_KEY", "1");

    Services.startup.quit(
      Ci.nsIAppStartup.eAttemptQuit | Ci.nsIAppStartup.eRestart
    );
  },

  addRestartShortcut() {
    let command = document.createXULElement("command");
    command.setAttribute("id", "cmd_quickRestart");
    command.addEventListener("command", this.quickRestart, true);
    document.getElementById("mainCommandSet").prepend(command);

    let key = document.createXULElement("key");
    key.setAttribute("id", "key_quickRestart");
    key.setAttribute("key", "r");
    key.setAttribute("modifiers", "accel,alt");
    key.setAttribute("command", "cmd_quickRestart");
    document.getElementById("mainKeyset").prepend(key);

    let menuitem = document.createXULElement("menuitem");
    menuitem.setAttribute("id", "menu_FileRestartItem");
    menuitem.setAttribute("key", "key_quickRestart");
    menuitem.setAttribute("label", "Restart (Developer)");
    menuitem.addEventListener("command", this.quickRestart, true);
    document.getElementById("menu_FilePopup").appendChild(menuitem);
  },
};

This file is browser-development-helpers.js which was located in omni.ja\chrome\browser\content\browser. I just really want the Ctrl + Alt + R keyboard shortcut back in Waterfox.

thunderstone135 avatar Jul 04 '23 02:07 thunderstone135

Update:

I noticed that this entry is missing in omni.ja 👇

 * License, v. 2.0. If a copy of the MPL was not distributed with this
 * file, You can obtain one at http://mozilla.org/MPL/2.0/. */

/**
 * Extra features for local development. This file isn't loaded in
 * non-local builds.
 */

var DevelopmentHelpers = {
  init() {
    this.quickRestart = this.quickRestart.bind(this);
    this.addRestartShortcut();
  },

  quickRestart() {
    Services.obs.notifyObservers(null, "startupcache-invalidate");

    let env = Cc["@mozilla.org/process/environment;1"].getService(
      Ci.nsIEnvironment
    );
    env.set("MOZ_DISABLE_SAFE_MODE_KEY", "1");

    Services.startup.quit(
      Ci.nsIAppStartup.eAttemptQuit | Ci.nsIAppStartup.eRestart
    );
  },

  addRestartShortcut() {
    let command = document.createXULElement("command");
    command.setAttribute("id", "cmd_quickRestart");
    command.addEventListener("command", this.quickRestart, true);
    document.getElementById("mainCommandSet").prepend(command);

    let key = document.createXULElement("key");
    key.setAttribute("id", "key_quickRestart");
    key.setAttribute("key", "r");
    key.setAttribute("modifiers", "accel,alt");
    key.setAttribute("command", "cmd_quickRestart");
    document.getElementById("mainKeyset").prepend(key);

    let menuitem = document.createXULElement("menuitem");
    menuitem.setAttribute("id", "menu_FileRestartItem");
    menuitem.setAttribute("key", "key_quickRestart");
    menuitem.setAttribute("label", "Restart (Developer)");
    menuitem.addEventListener("command", this.quickRestart, true);
    document.getElementById("menu_FilePopup").appendChild(menuitem);
  },
};

This file is browser-development-helpers.js which was located in omni.ja\chrome\browser\content\browser. I just really want the Ctrl + Alt + R keyboard shortcut back in Waterfox.

Thanks. I was searching for a way to remove the annoying 'Restart (developer)' menu item from LibreWolf and your comment helped me. I just added the following to my userChrome.css:

#menu_FileRestartItem {
  display: none;
}

It does nothing about the keyboard shortcut conflicting with that for "Enter Reader View", but small problems (especially since I never use reader view).

keithbowes avatar Apr 06 '25 14:04 keithbowes