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

Feature/wrap to screen #1

Open
wants to merge 27 commits into
base: master
Choose a base branch
from
Open

Feature/wrap to screen #1

wants to merge 27 commits into from

Conversation

martinheidegger
Copy link

Alternative approach to wrap the text in the terminal. This one seems to work with lists as well.

@dominictarr
Copy link
Owner

did you mean to make this pull request against https://github.com/firede/msee ?

@martinheidegger
Copy link
Author

No, you seem to have been interested in this. This PR is supposed to be a continued development of msee for wrapping.

@dominictarr
Copy link
Owner

oh I see, this is a pull request into my pull request. I see that there are several open pull requests by you, and the one I made, over a year ago, I don't think @firede is maintaining that project anymore. You should a) ask for the module from him, or b) publish a fork and maintain that your self. Users will see the open pull requests and figure out to use your version.

@martinheidegger
Copy link
Author

@dominictarr I have now commit rights and @firede did merge a few PR's yesterday.

@martinheidegger
Copy link
Author

Note: I think my implementation doesn't work 100% right (debugging would be mighty appreciated). But more importantly: The implementation right now is slow because of: tokuhirom/visualwidth-js#8

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

Successfully merging this pull request may close these issues.

6 participants