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 "rich" for terminal rendering #261

Open
dlax opened this issue Dec 24, 2021 · 5 comments
Open

use "rich" for terminal rendering #261

dlax opened this issue Dec 24, 2021 · 5 comments
Assignees

Comments

@dlax
Copy link
Member

dlax commented Dec 24, 2021

I'm thinking of replacing blessed with rich as a terminal rendering library. Rich is now mature and provides high level features that would be nice for us as we could drop a bunch of custom logic on our side. In particular:

  • no more need to handle available vertical space to limit the number of processes rendered (the limit() decorator)
  • automatic cropping or wrapping of long lines
  • better handling of text split with terminal sequence (would allow to handle horizontal scrolling as suggested in scrolling and wrapping #260 without loosing the formatting, or introducing a bunch of custom logic on our side)
  • native table rendering
  • "layouts" allowing to split the screen into multiple areas (possibly with fixed size, and automatic overflow handling)
  • live display, allowing to refresh only part of the ui

That would be a large refactoring. But, after having played a bit with it, I think it's worth as it will make UI development easier long term.

Thoughts?

@blogh
Copy link
Collaborator

blogh commented Dec 28, 2021

That looks like a great idea. :)

@blogh
Copy link
Collaborator

blogh commented Dec 28, 2021

It would be nice, if we could solve #222 during the revamp. (I noticed that if there is wrapping only the first line is colored).

@dlax
Copy link
Member Author

dlax commented Jan 12, 2022 via email

@blogh
Copy link
Collaborator

blogh commented Jan 17, 2022

The fact that the size of the columns is potentially changing at each refresh is a little distracting.
What do you think about this ?

@dlax
Copy link
Member Author

dlax commented Jan 17, 2022 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants
@dlax @blogh and others