LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues 596 Pull requests 6 Projects Releases 238 Packages Wiki Activity

Fix context-menu #1671

Merged
btzr-io merged 5 commits from fix-paste into master 2018-07-03 06:38:28 +02:00
Conversation 2 Commits 5 Files changed 5 +7 -7
1 changed files with 7 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files
Showing only changes of commit 8debe7dcc9 - Show all commits

14
src/renderer/util/contextMenu.js
View file

@ -21,11 +21,11 @@ function injectDevelopmentTemplate(event, templates) {
return templates;
}
export function openContextMenu(event, templates = []) {
export function openContextMenu(event, templates = [], canEdit = false) {
const { type, value } = event.target;
const isSomethingSelected = window.getSelection().toString().length > 0;
const { type } = event.target;
const isInput = event.target.matches('input') && (type === 'text' || type === 'number');
const { value } = event.target;
const isTextField = canEdit || isInput || event.target.matches('textarea');
templates.push({
label: 'Copy',
@ -36,12 +36,12 @@ export function openContextMenu(event, templates = []) {
// If context menu is opened on Input and there is text on the input and something is selected.
const { selectionStart, selectionEnd } = event.target;
if (!!value && isInput && selectionStart !== selectionEnd) {
if (!!value && isTextField && selectionStart !== selectionEnd) {
templates.push({ label: 'Cut', accelerator: 'CmdOrCtrl+X', role: 'cut' });
}
// If context menu is opened on Input and text is present on clipboard
if (clipboard.readText().length > 0 && isInput) {
if (clipboard.readText().length > 0 && isTextField) {
templates.push({
label: 'Paste',
accelerator: 'CmdOrCtrl+V',
@ -50,7 +50,7 @@ export function openContextMenu(event, templates = []) {
}
// If context menu is opened on Input
if (isInput && value) {
if (isTextField && value) {
templates.push({
label: 'Select All',
accelerator: 'CmdOrCtrl+A',
@ -70,5 +70,5 @@ export function openCopyLinkMenu(text, event) {
},
},
];
openContextMenu(event, templates, false);
openContextMenu(event, templates);
}