-
-
Notifications
You must be signed in to change notification settings - Fork 721
Panels Can Fit To Content #1096
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
base: master
Are you sure you want to change the base?
Panels Can Fit To Content #1096
Conversation
📝 Docs preview for commit 1e1a256 at: https://a93b4aa2.typertiangolo.pages.dev Modified Pages |
📝 Docs preview for commit b97e79e at: https://52aba9d8.typertiangolo.pages.dev Modified Pages |
Any chances to merge this PR soon? Can someone please restart CI? |
📝 Docs preview for commit 03860a5 at: https://7b0a9feb.typertiangolo.pages.dev Modified Pages |
I believe what's missing is test coverage for my new code and logic, which is not my strong suit. I would appreciate any help or suggestions on how to approach writing tests for this. |
Provide an option for using Rich
Panel.fit()
akaPanel(expand=False)
for help panels and errors. This was mentioned in discussion #607 and #680. To opt into this new, non-default behavior, just use:I think this can look nicer on very wide terminals, especially for CLIs with few parameters, or short parameter descriptions, so I would prefer to have the option to choose this style sometimes.
Status
Example