Skip to content

Conversation

@eamodio
Copy link
Contributor

@eamodio eamodio commented Sep 19, 2017

This should close #27408, since we don't want to change the default -- there is now an explicit way for an extension to target the active view column.

Once this is in, the Git extension can be updated to use this re: #34459

FYI, I used -1 to help avoid any falsey issues.

/cc @jrieken @bpasero

@jrieken jrieken added api feature-request Request for new features or functionality labels Sep 20, 2017
@jrieken jrieken added this to the September 2017 milestone Sep 20, 2017
@jrieken
Copy link
Member

jrieken commented Sep 20, 2017

Very nice. Thanks @eamodio

@jrieken jrieken merged commit 7c0116c into microsoft:master Sep 20, 2017
@jrieken jrieken added on-testplan release-notes Release notes issues labels Sep 25, 2017
@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api feature-request Request for new features or functionality on-testplan release-notes Release notes issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

showTextDocument & vscode.diff always default to ViewColumn.One

2 participants