Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Commit

Permalink
Revert "Added options for tabsToolbar context menu"
Browse files Browse the repository at this point in the history
This reverts commit 5df2bd3.
  • Loading branch information
philkloose committed Apr 26, 2017
1 parent 5df2bd3 commit c55ba0a
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 37 deletions.
20 changes: 0 additions & 20 deletions app/common/commonMenu.js
Original file line number Diff line number Diff line change
Expand Up @@ -135,15 +135,6 @@ module.exports.reopenLastClosedTabItem = () => {
}
}

module.exports.muteAllTabsMenuItem = () => {
return {
label: locale.translation('muteTabs'),
click: (item) => {
windowActions.muteAllAudio(framePropsList, true)
}
}
}

module.exports.separatorMenuItem = {
type: 'separator'
}
Expand Down Expand Up @@ -344,17 +335,6 @@ module.exports.bookmarksToolbarMenuItem = () => {
}
}

module.exports.showTabPreviewsMenuItem = () => {
return {
label: locale.translation('showTabPreviews'),
type: 'checkbox',
checked: getSetting(settings.SHOW_TAB_PREVIEWS),
click: (item, focusedWindow) => {
appActions.changeSetting(settings.SHOW_TAB_PREVIEWS, !getSetting(settings.SHOW_TAB_PREVIEWS))
}
}
}

module.exports.autoHideMenuBarMenuItem = () => {
const autoHideMenuBar = getSetting(settings.AUTO_HIDE_MENU)
return {
Expand Down
2 changes: 1 addition & 1 deletion app/extensions/brave/locales/en-US/menu.properties
Original file line number Diff line number Diff line change
Expand Up @@ -97,6 +97,7 @@ braveryPayments=Brave Payments…
braveryStartUsingPayments=Start using Brave Payments…
addBookmark=Add Bookmark…
addFolder=Add Folder…
unmuteTabs=Unmute Tabs
muteTabs=Mute Tabs
muteOtherTabs=Mute other Tabs
editFolder=Edit Folder…
Expand Down Expand Up @@ -164,4 +165,3 @@ downloadItemDelete=Delete Download
downloadItemClear=Clear Download
downloadToolbarHide=Hide downloads bar
downloadItemClearCompleted=Clear completed downloads
showTabPreviews=Show Tab Previews
1 change: 0 additions & 1 deletion app/locale.js
Original file line number Diff line number Diff line change
Expand Up @@ -172,7 +172,6 @@ var rendererIdentifiers = function () {
'learnSpelling',
'ignoreSpelling',
'lookupSelection',
'showTabPreviews',
// Other identifiers
'aboutBlankTitle',
'urlCopied',
Expand Down
16 changes: 1 addition & 15 deletions js/contextMenus.js
Original file line number Diff line number Diff line change
Expand Up @@ -636,21 +636,7 @@ function tabsTemplateInit () {

const template = []

template.push( CommonMenu.newTabMenuItem(),
CommonMenu.separatorMenuItem,
CommonMenu.newPrivateTabMenuItem(),
CommonMenu.newPartitionedTabMenuItem(),
CommonMenu.newWindowMenuItem(),
CommonMenu.separatorMenuItem,
CommonMenu.muteAllTabsMenuItem(),
CommonMenu.showTabPreviewsMenuItem(),
CommonMenu.separatorMenuItem,
CommonMenu.bookmarksManagerMenuItem(),
CommonMenu.bookmarksToolbarMenuItem(),
CommonMenu.separatorMenuItem
)

template.push(Object.assign({},
template.push(Object.assign({},
CommonMenu.reopenLastClosedTabItem(),
{ enabled: windowStore.getState().get('closedFrames').size > 0 }
))
Expand Down

0 comments on commit c55ba0a

Please sign in to comment.