Skip to content

fixes #2507 - bad formatting for formatonkey if convertTabsToSpaces === false#2687

Merged
dbaeumer merged 1 commit into
microsoft:masterfrom
dbaeumer:feature/fix2507
Apr 19, 2015
Merged

fixes #2507 - bad formatting for formatonkey if convertTabsToSpaces === false#2687
dbaeumer merged 1 commit into
microsoft:masterfrom
dbaeumer:feature/fix2507

Conversation

@dbaeumer

@dbaeumer dbaeumer commented Apr 9, 2015

Copy link
Copy Markdown
Member

See issue #2507

@mhegazy

mhegazy commented Apr 9, 2015

Copy link
Copy Markdown
Contributor

👍

@DanielRosenwasser

Copy link
Copy Markdown
Member

A question about how this is being used: what happens if someone uses Shift+Tab?

@dbaeumer

Copy link
Copy Markdown
Member Author

@DanielRosenwasser this is independent of tab ot shift tab. The existing code adjusts the indent position but it only considers that the formatter indents using spaces. The pull request also takes into account the fact that the formatter uses tabs as indentation characters if requested to do so.

@mhegazy

mhegazy commented Apr 18, 2015

Copy link
Copy Markdown
Contributor

Feel free to merge whenever you are ready.

dbaeumer added a commit that referenced this pull request Apr 19, 2015
fixes #2507 - bad formatting for formatonkey if convertTabsToSpaces === false
@dbaeumer dbaeumer merged commit 62b15de into microsoft:master Apr 19, 2015
@microsoft microsoft locked and limited conversation to collaborators Jun 18, 2018
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