lsd-rs / lsd

The next gen ls command
Apache License 2.0
12.8k stars 413 forks source link

Add space for executable construction icon #1068

Open Mersid opened 1 week ago

Mersid commented 1 week ago

Continuation of #1056, which was closed due to incorrect branch. We should find a way to dynamically calculate the size so we don't have to go around adding spaces like this.

Makes executable files go from looking like this: image

To this: image

Notice in the original image there's no spacing between the construction crane icon and the file's name.


TODO

muniu-bot[bot] commented 1 week ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Mersid Once this PR has been reviewed and has the lgtm label, please assign zwpaper for approval by writing /assign @zwpaper in a comment. For more information see:The Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files: - **[OWNERS](https://github.com/lsd-rs/lsd/blob/master/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment