Incorrect leading spaces

Question

Incorrect leading spaces

Closed this issue 6 years ago · 6 comments

with version 2.3.0:

// 1 leading space
require('wrap-ansi')(' a ', process.stdout.columns, {trim: false})

Out:

// 2 leading spaces
'  a '

stroncium earned $80.00 by resolving this issue!

Checkout the Issuehunt explorer to discover more funded issues.

Need some help from other developers? Add your repositories on Issuehunt to raise funds.

Answer 1 · 2018-03-01T15:13:15.000Z

I think we need to make sure that we aren't in the beginning of a line with a '' character here https://github.com/chalk/wrap-ansi/blob/master/index.js#L94-L103.

Answer 2 · 2018-03-04T21:16:04.000Z

@egoist this definitely seems wrong to me too. Could I convince you to add a failing test? Would also happily help you land a fix, if you decide you're interested in taking on this issue.

Answer 3 · 2018-03-05T02:22:19.000Z

Here's how you make a failing test.

Answer 4 · 2019-02-19T09:02:38.000Z

@IssueHunt has funded $80.00 to this issue. See it on IssueHunt

Answer 5 · 2019-02-26T04:33:21.000Z

@egoist Try removing both the white spaces before and after 'a'.

Answer 6 · 2019-03-03T06:25:26.000Z

@sindresorhus has rewarded $72.00 to @stroncium. See it on IssueHunt

💰 Total deposit: $80.00
🎉 Repository reward(0%): $0.00
🔧 Service fee(10%): $8.00