Browse Source

Closes #63 - Switch commit message and author in commit pick list

main
Eric Amodio 7 years ago
parent
commit
a5be6f0533
3 changed files with 14 additions and 11 deletions
  1. +7
    -0
      src/git/models/logCommit.ts
  2. +1
    -5
      src/quickPicks/commitDetails.ts
  3. +6
    -6
      src/quickPicks/common.ts

+ 7
- 0
src/git/models/logCommit.ts View File

@ -51,4 +51,11 @@ export class GitLogCommit extends GitCommit {
get nextUri(): Uri {
return this.nextFileName ? Uri.file(path.resolve(this.repoPath, this.nextFileName)) : this.uri;
}
getDiffStatus(): string {
const added = this.fileStatuses.filter(_ => _.status === 'A' || _.status === '?').length;
const deleted = this.fileStatuses.filter(_ => _.status === 'D').length;
const changed = this.fileStatuses.filter(_ => _.status !== 'A' && _.status !== '?' && _.status !== 'D').length;
return `+${added} ~${changed} -${deleted}`;
}
}

+ 1
- 5
src/quickPicks/commitDetails.ts View File

@ -117,13 +117,9 @@ export class CommitDetailsQuickPick {
description: `\u00a0 \u2014 \u00a0\u00a0 $(git-commit) ${commit.shortSha} \u00a0 $(git-compare) \u00a0 $(file-directory) Working Tree`
}, Commands.DiffDirectory, [uri, commit.sha]));
const added = commit.fileStatuses.filter(_ => _.status === 'A' || _.status === '?').length;
const deleted = commit.fileStatuses.filter(_ => _.status === 'D').length;
const changed = commit.fileStatuses.filter(_ => _.status !== 'A' && _.status !== '?' && _.status !== 'D').length;
items.splice(index++, 0, new CommandQuickPickItem({
label: `Changed Files`,
description: `+${added} ~${changed} -${deleted}`
description: commit.getDiffStatus()
}, Commands.ShowQuickCommitDetails, [uri, commit.sha, commit, goBackCommand, repoLog]));
items.push(new OpenCommitFilesCommandQuickPickItem(commit));

+ 6
- 6
src/quickPicks/common.ts View File

@ -118,14 +118,14 @@ export class CommitQuickPickItem implements QuickPickItem {
constructor(public commit: GitCommit) {
if (commit instanceof GitStashCommit) {
this.label = `${commit.stashName}, ${moment(commit.date).fromNow()}`;
this.description = `\u00a0\u00a0\u2014\u00a0\u00a0 ${commit.fileNames}`; //\u00a0\u00a0 $(git-commit) ${commit.shortSha}
this.label = `${commit.stashName}\u00a0\u2022\u00a0${commit.message}`;
this.description = null;
this.detail = `\u00a0 ${moment(commit.date).fromNow()}\u00a0\u00a0\u2022\u00a0 ${commit.getDiffStatus()}`;
}
else {
this.label = `${commit.author}, ${moment(commit.date).fromNow()}`;
const suffix = commit.type === 'branch' ? ` \u2014 ${(commit as GitLogCommit).fileNames}` : '';
this.description = `\u00a0 \u2014 \u00a0\u00a0 $(git-commit) ${commit.shortSha}${suffix}`;
this.label = commit.message;
this.description = `\u00a0$(git-commit)\u00a0 ${commit.shortSha}`;
this.detailspan> class="o">= `\u00a0 ${pan class="nx">commit.author}, ${moment class="p">(commit.="nx">date).fromNow()}${(commit.type === 'branch') ? `\u00a0\u00a0\u2022\u00a0 ${(commit as GitLogCommit).getDiffStatus()}` : ''}`;
}
this.detail = commit.message;
}
}

Loading…
Cancel
Save