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

Commit

Permalink
Enable context menu selection options for URL bar
Browse files Browse the repository at this point in the history
Also added some wiki context menu item manual tests

Fix #1707

Auditors: @aekeus
  • Loading branch information
bbondy committed May 12, 2016
1 parent b8fea72 commit 4524537
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions js/contextMenus.js
Original file line number Diff line number Diff line change
Expand Up @@ -74,10 +74,7 @@ function tabPageTemplateInit (framePropsList) {
}

function urlBarTemplateInit (searchDetail, activeFrame, e) {
const hasSelection = e.target.selectionStart !== undefined &&
e.target.selectionEnd !== undefined &&
e.target.selectionStart !== e.target.selectionEnd
const items = getEditableItems(hasSelection)
const items = getEditableItems(window.getSelection().toString())
const clipboardText = clipboard.readText()
const hasClipboard = clipboardText && clipboardText.length > 0
const isLocationUrl = hasClipboard && isUrl(clipboardText)
Expand Down

1 comment on commit 4524537

@aekeus
Copy link
Member

@aekeus aekeus commented on 4524537 May 13, 2016

Choose a reason for hiding this comment

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

++ Looks good. Copy item back in URL context menu.

Please sign in to comment.