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

Add support for max term width #653

Closed
kbknapp opened this issue Sep 10, 2016 · 4 comments
Closed

Add support for max term width #653

kbknapp opened this issue Sep 10, 2016 · 4 comments
Labels
A-builder Area: Builder API A-help Area: documentation, including docs.rs, readme, examples, etc... C-enhancement Category: Raise on the bar on expectations
Milestone

Comments

@kbknapp
Copy link
Member

kbknapp commented Sep 10, 2016

Similar to the default, except that even when term size can be determined don't go above max

@kbknapp kbknapp added C-enhancement Category: Raise on the bar on expectations A-help Area: documentation, including docs.rs, readme, examples, etc... A-builder Area: Builder API D: easy labels Sep 10, 2016
@kbknapp kbknapp added this to the 2.12.0 milestone Sep 10, 2016
@nabijaczleweli
Copy link
Contributor

As in #639 or?

@kbknapp
Copy link
Member Author

kbknapp commented Sep 10, 2016

Right now if the term size can't be determined it defaults to 120. But if the term size can be determined and it's really big (I.e. I use a full screen term most of the time), I'd like the ability to say if the term size is huge go ahead and wrap at 120 or whatever number.

@nabijaczleweli
Copy link
Contributor

nabijaczleweli commented Sep 10, 2016

IOW force term width override?

@kbknapp
Copy link
Member Author

kbknapp commented Sep 10, 2016

Kind of, I don't want to say always wrap at a specific width because the width may be smaller which would create weird line breaks. I want to say wrap at x width unless it's smaller then wrap there instead

homu added a commit that referenced this issue Sep 10, 2016
… r=kbknapp

Allow for limiting detected term width

Closes #653
homu added a commit that referenced this issue Sep 10, 2016
… r=kbknapp

Allow for limiting detected term width

Closes #653
@homu homu closed this as completed in #654 Sep 10, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-builder Area: Builder API A-help Area: documentation, including docs.rs, readme, examples, etc... C-enhancement Category: Raise on the bar on expectations
Projects
None yet
Development

No branches or pull requests

2 participants