Skip to content
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

Tabs can have variable width, which could lead to errors. #37

Open
serkonda7 opened this issue Apr 11, 2022 · 2 comments
Open

Tabs can have variable width, which could lead to errors. #37

serkonda7 opened this issue Apr 11, 2022 · 2 comments

Comments

@serkonda7
Copy link

Tabs can have variable width, which could lead to errors.

I suggest that the extension warns, if tabs are detected as indentation.

@PanAeon PanAeon changed the title tabs lead to errors Tabs can have variable width, which could lead to errors. Apr 13, 2022
@mdaverde
Copy link

Just ran into this with snapshots.

Seems to me we still render tabs in the snapshot for the original line if it contained it. We could detect this, in the case of indentations, and do the same for the rendered snapshot tree?

@PanAeon
Copy link
Owner

PanAeon commented May 27, 2023

@mdaverde could you give a specific concise example and elaborate more on the proposed solution?
The only thing I can think of to remedy the situation is to specify tab width in the configuration for the project.

Unrelated - seems like a good remedy to the infamous tabs-with-spaces discussions git automatic tab-space converter .

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

No branches or pull requests

3 participants