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

Commit

Permalink
Merge pull request #5251 from maaz93/single-history-clear
Browse files Browse the repository at this point in the history
Removing clear cache and clear site data options from the history menu
  • Loading branch information
bsclifton authored Oct 30, 2016
2 parents 3649f29 + a3aebb1 commit 3368def
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 11 deletions.
12 changes: 1 addition & 11 deletions app/browser/menu.js
Original file line number Diff line number Diff line change
Expand Up @@ -322,21 +322,11 @@ const createHistorySubmenu = () => {
CommonMenu.separatorMenuItem,
*/
{
label: locale.translation('clearHistory'),
label: locale.translation('clearBrowsingData'),
accelerator: 'Shift+CmdOrCtrl+Delete',
click: function (item, focusedWindow) {
CommonMenu.sendToFocusedWindow(focusedWindow, [messages.SHORTCUT_OPEN_CLEAR_BROWSING_DATA_PANEL, {browserHistory: true}])
}
}, {
label: locale.translation('clearCache'),
click: function (item, focusedWindow) {
CommonMenu.sendToFocusedWindow(focusedWindow, [messages.SHORTCUT_OPEN_CLEAR_BROWSING_DATA_PANEL, {cachedImagesAndFiles: true}])
}
}, {
label: locale.translation('clearSiteData'),
click: function (item, focusedWindow) {
CommonMenu.sendToFocusedWindow(focusedWindow, [messages.SHORTCUT_OPEN_CLEAR_BROWSING_DATA_PANEL, {allSiteCookies: true, cachedImagesAndFiles: true}])
}
}
]
submenu = submenu.concat(menuUtil.createRecentlyClosedMenuItems(Immutable.fromJS(Object.keys(closedFrames).map(key => closedFrames[key]))))
Expand Down
1 change: 1 addition & 0 deletions app/extensions/brave/locales/en-US/menu.properties
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,7 @@ back=Back
forward=Forward
reopenLastClosedWindow=Reopen Last Closed Window
showAllHistory=Show History
clearBrowsingData=Clear Browsing Data…
clearHistory=Clear History…
clearCache=Clear Cache…
clearSiteData=Clear All Cookies and Site Data…
Expand Down
1 change: 1 addition & 0 deletions app/locale.js
Original file line number Diff line number Diff line change
Expand Up @@ -121,6 +121,7 @@ var rendererIdentifiers = function () {
'clearCache',
'clearHistory',
'clearSiteData',
'clearBrowsingData',
'recentlyClosed',
'recentlyVisited',
'bookmarks',
Expand Down

0 comments on commit 3368def

Please sign in to comment.