Electrux / ls_extended

ls with coloring and icons
BSD 3-Clause "New" or "Revised" License
201 stars 14 forks source link

Feature request: Trim some whitespace to make output more compact #37

Closed Tobbe closed 3 years ago

Tobbe commented 3 years ago

First of all, thank you for writing ls_extended. It makes the ls output both nicer to look at and more efficient to work with!

This is ls_extended compared to regular ls

image

Would it be possible to have less space around the username and the group? And also around the hard link count. When working in a split screen I often get files wrapping to the next line when they would easily have fit on one line had the output just been a bit more compact (more like the regular ls output)

Electrux commented 3 years ago

Hey! Thanks for the kind words! :) This is actually weird. It shouldn't add that much space. I am not able to replicate it. Which build of the program are you using? That looks like an older code's build actually.

Electrux commented 3 years ago

My output (uses latest master branch)

Tobbe commented 3 years ago

Wow, yeah, yours is much more compact! :) I installed using homebrew on my M1 MacBook Pro

Tobbe commented 3 years ago

image

Tobbe commented 3 years ago

Noticed you just released a new version, so I tried to update

image

Still have the same 1.0.8 version with the same behavior I posted above

Electrux commented 3 years ago

Oh okay, yes apologies, I had forgotten to update the homebrew version. That has been done now. Could you please uninstall ls_extended, update homebrew formula, and reinstall ls_extended? The archive has just been updated - archive name will be v1.tar.gz Version will be 1.0.0

Tobbe commented 3 years ago

After uninstalling and updating I get this error when I try to reinstall

==> Downloading https://github.com/Electrux/ls_extended/archive/refs/tags/v1.tar.gz
Already downloaded: /Users/tobbe/Library/Caches/Homebrew/downloads/ccef4579e792c33b5d4f0664ef18adf920fab39035a71d7727c39b2aa60ef651--ls_extended-1.tar.gz
Error: ls_extended: SHA256 mismatch
Expected: c6af08877e889fe0732f25f7e6f850c7816cf7788fee191bcc80dbf1288b3740
  Actual: c751e8198356cbbbbfe6191a6dd379d0525fea469299355ac8331090f2db63fe
    File: /Users/tobbe/Library/Caches/Homebrew/downloads/ccef4579e792c33b5d4f0664ef18adf920fab39035a71d7727c39b2aa60ef651--ls_extended-1.tar.gz
To retry an incomplete download, remove the file above.

I tried deleting the file, as it says to do, but I get the same error again when I try to install

Tobbe commented 3 years ago

I created a PR for the sha https://github.com/Electrux/homebrew-taps/pull/1

Electrux commented 3 years ago

Merged the PR (thanks a bunch!). Let me know if it works after that.

Tobbe commented 3 years ago

Before and after 👍

image

Electrux commented 3 years ago

Awesome! Closing this issue. Have a good one! :)