Merge pull request #3003 from rohanthacker/updated-hamburger-menu

Reordered Hamburger Menu with layout mentioned in #1893
brave · Aug 18, 2016 · 30b00cc · 30b00cc
2 parents 6cfe4cc + cdae492
commit 30b00cc
Showing 1 changed file with 10 additions and 15 deletions.
diff --git a/js/contextMenus.js b/js/contextMenus.js
@@ -534,19 +534,10 @@ function getEditableItems (selection, editFlags) {
 
 function hamburgerTemplateInit (location, e) {
   const template = [
-    {
-      l10nLabelId: 'new',
-      submenu: [
-        CommonMenu.newTabMenuItem(),
-        CommonMenu.newPrivateTabMenuItem(),
-        CommonMenu.newPartitionedTabMenuItem(),
-        CommonMenu.separatorMenuItem,
-        CommonMenu.newWindowMenuItem()
-      ]
-    },
-    CommonMenu.separatorMenuItem,
-    CommonMenu.findOnPageMenuItem(),
-    CommonMenu.printMenuItem(),
+    CommonMenu.newTabMenuItem(),
+    CommonMenu.newPrivateTabMenuItem(),
+    CommonMenu.newPartitionedTabMenuItem(),
+    CommonMenu.newWindowMenuItem(),
     CommonMenu.separatorMenuItem,
     {
       l10nLabelId: 'zoom',
@@ -570,7 +561,6 @@ function hamburgerTemplateInit (location, e) {
       }]
     },
     CommonMenu.separatorMenuItem,
-    CommonMenu.preferencesMenuItem(),
     {
       label: locale.translation('bookmarks'),
       submenu: [
@@ -587,6 +577,10 @@ function hamburgerTemplateInit (location, e) {
       ]
     },
     CommonMenu.downloadsMenuItem(),
+    CommonMenu.findOnPageMenuItem(),
+    CommonMenu.printMenuItem(),
+    CommonMenu.separatorMenuItem,
+    CommonMenu.preferencesMenuItem(),
     CommonMenu.separatorMenuItem,
     {
       label: locale.translation('help'),
@@ -598,7 +592,8 @@ function hamburgerTemplateInit (location, e) {
         CommonMenu.reportAnIssueMenuItem(),
         CommonMenu.submitFeedbackMenuItem()
       ]
-    }
+    },
+    CommonMenu.quitMenuItem()
   ]
   return template
 }