From bba799a70c9292139ac4fbd810cbd183044638b6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Florian=20M=C3=BCllner?= Date: Thu, 12 Aug 2021 00:33:33 +0200 Subject: [PATCH] appMenu: Remove separator between "new window" and actions We hide our own "New Window" item if the app itself includes a "new-window" action. That means that the separator between the built-in item and desktop actions introduces a small inconsistency depending on whether a "New Window" item is provided by the desktop action or ourselves. There's no good reason for that from a user's perspective, so remove the separator. Part-of: --- js/ui/appMenu.js | 2 -- 1 file changed, 2 deletions(-) diff --git a/js/ui/appMenu.js b/js/ui/appMenu.js index 9d1bcd30b..a5011cc31 100644 --- a/js/ui/appMenu.js +++ b/js/ui/appMenu.js @@ -57,8 +57,6 @@ var AppMenu = class AppMenu extends PopupMenu.PopupMenu { Main.overview.hide(); }); - this.addMenuItem(new PopupMenu.PopupSeparatorMenuItem()); - this._actionSection = new PopupMenu.PopupMenuSection(); this.addMenuItem(this._actionSection);