Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Display visual column width in selection#1203

Open
jscipione wants to merge 1 commit into
textmate:mastertextmate/textmate:masterfrom
jscipione:masterjscipione/textmate:masterCopy head branch name to clipboard
Open

Display visual column width in selection#1203
jscipione wants to merge 1 commit into
textmate:mastertextmate/textmate:masterfrom
jscipione:masterjscipione/textmate:masterCopy head branch name to clipboard

Conversation

@jscipione

Copy link
Copy Markdown
  • displays tab width instead of counting hard tabs as 1 character wide,
    which is how TextMate 1 worked.

For example if your tab width is set to 4 and you're on the second
indent position the column will display as 9 (4 * 2) + 1, before this patch
it would display as 3 (1 * 2) + 1.

displays tab width instead of counting hard tabs as 1 character wide.

Indent position the column will display as 9 (4 * 2) + 1,
before this it would display as 3 (1 * 2) + 1.

Limit tab width calculation to first 4096 characters of each line.
@jscipione

Copy link
Copy Markdown
Author

After all these years yes this is still a bug. The main complaint on my patch was that it would take too long to compute on long lines so I've arbitrarily limited this calculation to the first 4096 characters of a line.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Morty Proxy This is a proxified and sanitized view of the page, visit original site.