-
Notifications
You must be signed in to change notification settings - Fork 36
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
enable tab compatbility #28
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
13c60b1
to
b98d895
Compare
If someone is finding this pr and is looking for a fast fix: I published a version with my fix on npm. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM @dmnd?
when is it going to be merged? |
@SorryKim I wouldn't expect this to be merged in a timely manner. The project is inactive for over 3 years now. If you really need this feature, I would recommend you using dedent-tabs which is a fork of this project I still maintain and has this feature built in. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
💯 this looks great, thanks @adrianjost!
That's fast. |
This is a basic implementation to allow tab usage. I handle each tab like a single space character. In most cases this should be fine as it's really rare that tabs and spaces are mixed.
Issue: #10