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

Use wrap-ansi to force wrap lines #304

Closed
SBoudrias opened this issue Nov 19, 2015 · 0 comments · Fixed by #1106
Closed

Use wrap-ansi to force wrap lines #304

SBoudrias opened this issue Nov 19, 2015 · 0 comments · Fixed by #1106

Comments

@SBoudrias
Copy link
Owner

Should simplify the code by a great amount https://github.com/chalk/wrap-ansi

starpit added a commit to starpit/Inquirer.js that referenced this issue Apr 24, 2022
This PR incorrect line wrapping with `terminal-link`. It does so by employing the `wrap-ansi` npm.

Fixes SBoudrias#1014
Fixes SBoudrias#304
starpit added a commit to starpit/Inquirer.js that referenced this issue Apr 24, 2022
This PR addresses incorrect line wrapping with `terminal-link`. It does so by employing the `wrap-ansi` npm.

Fixes SBoudrias#1014
Fixes SBoudrias#304
starpit added a commit to starpit/Inquirer.js that referenced this issue Apr 24, 2022
This PR addresses incorrect line wrapping with `terminal-link`. It does so by employing the `wrap-ansi` npm.

Fixes SBoudrias#1014
Fixes SBoudrias#304
starpit added a commit to starpit/Inquirer.js that referenced this issue Apr 24, 2022
This PR addresses incorrect line wrapping with `terminal-link`. It does so by employing the `wrap-ansi` npm.

Fixes SBoudrias#1014
Fixes SBoudrias#304
SBoudrias pushed a commit that referenced this issue Apr 26, 2022
* fix: port breakLines to wrap-ansi

This PR addresses incorrect line wrapping with `terminal-link`. It does so by employing the `wrap-ansi` npm.

Fixes #1014
Fixes #304

* - update core utils.js to use wrapAnsi as well
- move wrapAnsi dependence out of the top-level

* revert yarn.lock

* add examples/terminal-link.js
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant