-
Notifications
You must be signed in to change notification settings - Fork 19
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 RTW “header” to terminal with all data #134
base: master
Are you sure you want to change the base?
Conversation
Hi @atticusrussell thanks for your contribution! This definitely goes into the direction I was thinking, but it is not exactly that. I was hoping to have at the top of the terminal, this filed will all information and not in each line. Nevertheless, let's see what other think on this solution. A few nit picks:
@StoglRobotics/stoglrobotics-non-client-repositories what do you think about this? |
First thanks for this PR. I am discussing the concept, not the implementation. so basically it goes from 3 long lines with direct result above prompt
to 8 lines with the result below the previous promp but 4 lines above the current prompt and the last command also a bit lost in the 3 line.
I can give my opinion about this concept: I think a header in 2 lines per line won't work for me. I prefer a short line indeed, but was quite ok with the current state (indeed half a terminal). I need to see if the initial idea would work better.
|
@gwalck what I understood, there is a possibility to create static content at the top of the terminal that should not break any scrolling. Maybe I have understood something wrong, but that would be the idea to realize. |
@atticusrussell do you have some time to check this? |
@destogl yes I have time to work on it. This is my first PR in this repo so not entirely sure what my next step should be: Should I modify the code to match the example at the end of @gwalck 's feedback to create a static header? Also personally, I'm not a huge fan of keeping [no-workspace] or as you mentioned, and I prefer showing the whole path to the current dir, as opposed to only the last folder. Does it make sense that I create boolean variables that enable/disable those features, and toggle between full path and last folder? I could make the default states of them match what you and @gwalck requested. |
@atticusrussell yes you can implement this what @gwalck proposed. But we should add in the second line in header the full path. It is often annoying to me to have a long path ahead of command. And yes, we can add flag to enable/disable this - this is the easiest part :) |
addresses issue #115 Add RTW "header" to terminal with all data
additionally splits apart prompt construction in a way that is simpler to understand
new behavior: