seqeralabs / tower-cli

Nextflow Tower CLI tool
Apache License 2.0
41 stars 9 forks source link

Improve TableList to wrap in multiple lines long values #42

Open jordeu opened 2 years ago

jordeu commented 2 years ago

Long fields make the table to seem broken:

$ towr organizations view -n BBGLab

  Details for organization 'Barcelona Biomedical Genomics'

   -------------+--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
    ID          | 175513926776370                                                                                                                                                                                                                                                                      
    Name        | BBGLab                                                                                                                                                                                                                                                                               
    Full Name   | Barcelona Biomedical Genomics                                                                                                                                                                                                                                                        
    Description | Our research is focused on the study of cancer from a genomics perspective. We are particularly interested in the 
identification of cancer driver mutations, genes and pathways across tumour types and in the study of the mutational processes
 across tissues and genomic regions. 
    Website     |                                                                                                                                                                                                                                                                                      

Split it in multiple lines keeping the table correct.

swampie commented 2 years ago

A possibile solution is to add an ellipses /easy solution) at the end otherwise the table component needs to be changed (more work).

@mbosio85 your call

mbosio85 commented 2 years ago

I would go for the ellipses solution. It solves the visual issue and we can move forward. If later on we notice that ellipses are a big enough issue somewhere (causes enough pain to enough users), we can change the table component for a future version.

pditommaso commented 2 years ago

Make sense to cut. The user can use describe to see the details