Skip to content

Typo: Uncommited → Uncommitted#109119

Merged
eamodio merged 1 commit intomicrosoft:masterfrom
mojodna:patch-1
Nov 12, 2020
Merged

Typo: Uncommited → Uncommitted#109119
eamodio merged 1 commit intomicrosoft:masterfrom
mojodna:patch-1

Conversation

@mojodna
Copy link
Contributor

@mojodna mojodna commented Oct 21, 2020

The Timeline extension contains a typo when there are uncommitted changes: "Uncommited Changes". This fixes that.

const date = new Date();

const item = new GitTimelineItem('', index ? '~' : 'HEAD', localize('git.timeline.uncommitedChanges', 'Uncommited Changes'), date.getTime(), 'working', 'git:file:working');
const item = new GitTimelineItem('', index ? '~' : 'HEAD', localize('git.timeline.uncommitedChanges', 'Uncommitted Changes'), date.getTime(), 'working', 'git:file:working');
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
const item = new GitTimelineItem('', index ? '~' : 'HEAD', localize('git.timeline.uncommitedChanges', 'Uncommitted Changes'), date.getTime(), 'working', 'git:file:working');
const item = new GitTimelineItem('', index ? '~' : 'HEAD', localize('git.timeline.uncommittedChanges', 'Uncommitted Changes'), date.getTime(), 'working', 'git:file:working');

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I skipped the localization key with the assumption that translations (which are external) would need to also be updated.

@eamodio eamodio added this to the November 2020 milestone Nov 12, 2020
@eamodio eamodio merged commit 217ad5a into microsoft:master Nov 12, 2020
@eamodio
Copy link
Contributor

eamodio commented Nov 12, 2020

Looks good -- Thanks!

@github-actions github-actions bot locked and limited conversation to collaborators Dec 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants