banesullivan / scooby

🐶 🕵️ Great Dane turned Python environment detective

MIT License

47 stars 12 forks source link

Introduce a compact layout for ncol=1 #89

Closed prisae closed 2 years ago

prisae commented 2 years ago

My proposition as an alternative to #82 @akaszynski - what do you think?

It does the following:

If ncol=1, it left-aligns everything.
It adds a new max-width parameter, with which the user can control the max-width of the table.
I tried to include all the other little cleaning-up bits you put into #82.

An example how this "compact" layout looks: 2022-07-22-01

codecov-commenter commented 2 years ago

Codecov Report

Merging #89 (0c2708b) into main (999d9ca) will decrease coverage by 0.22%. The diff coverage is 81.81%.

@@            Coverage Diff             @@
##             main      #89      +/-   ##
==========================================
- Coverage   87.15%   86.93%   -0.23%     
==========================================
  Files           4        4              
  Lines         366      375       +9     
==========================================
+ Hits          319      326       +7     
- Misses         47       49       +2

akaszynski commented 2 years ago

Looks way better @prisae, this is good to go provided CI passes post-merge with main.