Merge pull request brave#3444 from bsclifton/about-history-context-fix

bsclifton · web-flow · commit 12391d993484 · 2016-08-25T23:28:01.000-07:00
Fixes extra separator in about:history context menu
diff --git a/js/contextMenus.js b/js/contextMenus.js
@@ -262,12 +262,12 @@ function siteDetailTemplateInit (siteDetail, activeFrame) {
       {
         label: locale.translation(deleteLabel),
         click: () => appActions.removeSite(siteDetail, deleteTag)
-      },
-      CommonMenu.separatorMenuItem)
+      })
   }
 
   if (!isHistoryEntry) {
     template.push(
+      CommonMenu.separatorMenuItem,
       addBookmarkMenuItem('addBookmark', siteUtil.getDetailFromFrame(activeFrame, siteTags.BOOKMARK), siteDetail, true),
       addFolderMenuItem(siteDetail, true))
   }

Original file line number	Diff line number	Diff line change
`@@ -262,12 +262,12 @@ function siteDetailTemplateInit (siteDetail, activeFrame) {`
`262`	`262`	`{`
`263`	`263`	`label: locale.translation(deleteLabel),`
`264`	`264`	`click: () => appActions.removeSite(siteDetail, deleteTag)`
`265`		`- },`
`266`		`- CommonMenu.separatorMenuItem)`
	`265`	`+ })`
`267`	`266`	`}`
`268`	`267`
`269`	`268`	`if (!isHistoryEntry) {`
`270`	`269`	`template.push(`
	`270`	`+ CommonMenu.separatorMenuItem,`
`271`	`271`	`addBookmarkMenuItem('addBookmark', siteUtil.getDetailFromFrame(activeFrame, siteTags.BOOKMARK), siteDetail, true),`
`272`	`272`	`addFolderMenuItem(siteDetail, true))`
`273`	`273`	`}`