lsd icon indicating copy to clipboard operation
lsd copied to clipboard

Add space for executable construction icon

Open Mersid opened this issue 1 year ago • 1 comments

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

  • [X] Use cargo fmt
  • [X] Add necessary tests
  • [X] Update default config/theme in README (if applicable)
  • [X] Update man page at lsd/doc/lsd.md (if applicable)

Mersid avatar Jun 25 '24 03:06 Mersid

[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:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

muniu-bot[bot] avatar Jun 25 '24 03:06 muniu-bot[bot]