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

Commit

Permalink
get rid of backspace nav shortcuts
Browse files Browse the repository at this point in the history
fix #5323

Auditors: @bbondy

Test Plan:
1. go to any site
2. in the same tab, go to example.com
3. click to focus the webview, then hit backspace. nothing should happen.
  • Loading branch information
diracdeltas committed Nov 1, 2016
1 parent 8a13404 commit e89aa8a
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions app/extensions/brave/content/scripts/inputHandler.js
Original file line number Diff line number Diff line change
Expand Up @@ -139,11 +139,6 @@ document.addEventListener('keydown', (e /*: Event*/) => {
chrome.ipc.sendToHost('stop-load')
}
break
case KeyEvent.DOM_VK_BACK_SPACE:
if (!isEditable(document.activeElement)) {
e.shiftKey ?chrome.ipc.sendToHost('go-forward') :chrome.ipc.sendToHost('go-back')
}
break
case KeyEvent.DOM_VK_LEFT:
if (e.metaKey && !isEditable(document.activeElement) && isPlatformOSX()) {
chrome.ipc.sendToHost('go-back')
Expand Down

1 comment on commit e89aa8a

@bbondy
Copy link
Member

@bbondy bbondy commented on e89aa8a Nov 1, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Please sign in to comment.