Add row wrapping for long lines in search result table #3566
Conversation
| @@ -67,6 +67,8 @@ def search_datasets( | |||
| stdout.print(create_download_url(branch, selection)) | |||
| else: | |||
| table = rich.table.Table(show_header=False) | |||
| table.add_column("") | |||
There was a problem hiding this comment.
Maybe these also needs the wrapping for longer file names?
There was a problem hiding this comment.
No, this line describes the first COLUMN (containing "File:", "Nextflow Import:", "Download Link").
All rows in the second column will now be wrapped ;)
The table will always wrap text when it contains multiple words, but it doesn't know how to wrap the url, so the default behavior is to "ellipsis" it.
In essence, if the table gets too small it will do something like:
File
---------
Nextflow
Import
---------
Download
Link
Codecov ReportAttention: Patch coverage is
☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Description
Changes the output table of the
test-datasets searchsub-command to wrap long one-word rows to essentially avoid ellipsis-cutting URLs.This change was suggested in this website PR comment.
For copy-pasting urls the
-uflag is still more suitableBefore Changes:
After Changes:
PR checklist
CHANGELOG.mdis updateddocsis updated