paulmataruso/zulip-desktop
1
0
Fork 0
mirror of https://github.com/zulip/zulip-desktop.git synced 2025-11-13 10:26:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

context-menu: Fix bug in Copy Link and add copy Email.

Browse Source 
Fixes: #986
This commit is contained in:
Abhigyan Khaund
2020-06-26 17:01:44 +05:30
committed by Akash Nimare
parent 01926e1234
commit bda0dd29df
1 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
app/renderer/js/components/context-menu.ts
View File

@@ -5,6 +5,7 @@ const {clipboard, Menu} = remote;
export const contextMenu = (webContents: Electron.WebContents, event: Event, props: ContextMenuParams) => { export const contextMenu = (webContents: Electron.WebContents, event: Event, props: ContextMenuParams) => {
const isText = Boolean(props.selectionText.length); const isText = Boolean(props.selectionText.length);
const isLink = Boolean(props.linkURL); const isLink = Boolean(props.linkURL);
const isEmailAddress = Boolean(props.linkURL.startsWith('mailto:'));
const makeSuggestion = (suggestion: string) => ({ const makeSuggestion = (suggestion: string) => ({
label: suggestion, label: suggestion,
@@ -55,12 +56,12 @@ export const contextMenu = (webContents: Electron.WebContents, event: Event, pro
}, { }, {
type: 'separator' type: 'separator'
}, { }, {
label: t.__('Copy Link'), label: isEmailAddress ? t.__('Copy Email Address') : t.__('Copy Link'),
visible: isText && isLink, visible: isLink,
click(_item) { click(_item) {
clipboard.write({ clipboard.write({
bookmark: props.linkText, bookmark: props.linkText,
text: props.linkURL text: isEmailAddress ? props.linkText : props.linkURL
}); });
} }
}, { }, {