Option to use absolute column width about rewrap

[stkb]

Hmm, how you describe (absolute column) is exactly how it should be working, and if it's not, it's a bug.

Are you indenting with tabs or spaces?

[calmh]

Tabs. Perhaps the issue, they are seen as one character each?

[stkb]

Yeah :) Trying your example with 4-space indents I got the correct result

// wrapped at column
// twenty-two.

    // wrapped at
    // column
    // twenty-two.

But when using tabs it came out like this, as you described.

// wrapped at column
// twenty-two.

    // wrapped at column
    // twenty-two.

The issue is indeed that it's counting tabs as one character each for the purpose of working out where to wrap. VSCode doesn't (yet) make the tab display width available to extensions. But hopefully I can detect it in some way.

Genuine general question though, because I've always used spaces: How does fixing everything to a specific column width make sense when using tabs? Because if you for example, render tabs at 2 spaces and then make all your code wrap at column 80, and then someone else reads the code but renders the tabs at 4 spaces each, the line wrapping's all gonna be off. Or does it just not matter?

[calmh]

You're right, in that case the wrapping will be off for one of us. I'd say that doesn't matter as it's an inherent property of using tabs for indentation, but that it would be nice if this plugin did the right thing based on the settings it can figure out when it's invoked. :)

In the meantime, this isn't exactly a blocker and I'm happy to have some wrapping functionality in vscode, so feel free to put this on the backburner until the tab width config is available to the plugin somehow.

[stkb]

I've just pushed out a new version that should fix this issue (v0.4.1). Could you let me know if it's now working for you?

[calmh]

Looks good 👍

[stkb]

Great. Thanks for reporting!