padding is not based on the true length (wcwidth) of paddingChr

[danyshaanan]

For instance, { paddingChr: chalk.gray('.') } will break format.

Problem is within utils.js::repeatString which uses slice.

[timoxley]

Good call.

[danyshaanan]

@timoxley : when wcwidth(str)===1, (like with chalk.gray('.')) droping the slice(0,len) solves it (relevant line), but when wcwidth(str)>1 it's not always possible to multiply it several times to get the required length, and it is required to trim the last occurrence. Any idea how to properly trim such a string?

i.e. How can I slice the first two visual chars of chalk.gray('123') without messing the colors?

[timoxley]

I think the only way to do this would be to pass it through something like sindresorhus/strip-ansi first.

[danyshaanan]

That would enable slicing out the actual wanted length, but is there any reasonable way to colour it again before concatenating to the padding?

I'm starting to think that there is no way to do this, meaning we should disallow wcwidth(padding) > 1, wcwidth(padding) !== padding.length), or preferably, the combination of the two.