Selaa lähdekoodia

Updates clipboard error message for better clarity

main
Eric Amodio 6 vuotta sitten
vanhempi
commit
c7e162fdbc
3 muutettua tiedostoa jossa 5 lisäystä ja 5 poistoa
  1. +1
    -1
      src/commands/copyMessageToClipboard.ts
  2. +3
    -3
      src/commands/copyShaToClipboard.ts
  3. +1
    -1
      src/git/remotes/provider.ts

+ 1
- 1
src/commands/copyMessageToClipboard.ts Näytä tiedosto

@ -88,7 +88,7 @@ export class CopyMessageToClipboardCommand extends ActiveEditorCommand {
catch (ex) {
if (ex.message.includes("Couldn't find the required `xsel` binary")) {
window.showErrorMessage(
`Unable to copy message, xsel is not installed. You can install it via \`sudo apt install xsel\``
`Unable to copy message, xsel is not installed. Please install it via your package manager, e.g. \`sudo apt install xsel\``
);
return;
}

+ 3
- 3
src/commands/copyShaToClipboard.ts Näytä tiedosto

@ -59,7 +59,7 @@ export class CopyShaToClipboardCommand extends ActiveEditorCommand {
}
catch (ex) {
Logger.error(ex, 'CopyShaToClipboardCommand', `getBlameForLine(${blameline})`);
return Messages.showGenericErrorMessage('Unable to copy ommit id');
return Messages.showGenericErrorMessage('Unable to copy commit id');
}
}
@ -69,13 +69,13 @@ export class CopyShaToClipboardCommand extends ActiveEditorCommand {
catch (ex) {
if (ex.message.includes("Couldn't find the required `xsel` binary")) {
window.showErrorMessage(
`Unable to copy commit id, xsel is not installed. You can install it via \`sudo apt install xsel\``
`Unable to copy commit id, xsel is not installed. Please install it via your package manager, e.g. \`sudo apt install xsel\``
);
return;
}
Logger.error(ex, 'CopyShaToClipboardCommand');
return Messages.showGenericErrorMessage('Unable to copy ommit id');
return Messages.showGenericErrorMessage('Unable to copy commit id');
}
}
}

+ 1
- 1
src/git/remotes/provider.ts Näytä tiedosto

@ -127,7 +127,7 @@ export abstract class RemoteProvider {
catch (ex) {
if (ex.message.includes("Couldn't find the required `xsel` binary")) {
window.showErrorMessage(
`Unable to copy remote url, xsel is not installed. You can install it via \`sudo apt install xsel\``
`Unable to copy remote url, xsel is not installed. Please install it via your package manager, e.g. \`sudo apt install xsel\``
);
return;
}

Ladataan…
Peruuta
Tallenna