Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Theme colors for inode and links are now being applied to the output #768

Merged
merged 1 commit into from
Nov 14, 2022

Conversation

d-sonuga
Copy link
Contributor

This is a fix for #763

@codecov-commenter
Copy link

Codecov Report

Merging #768 (fb77493) into master (f014138) will decrease coverage by 0.10%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #768      +/-   ##
==========================================
- Coverage   86.78%   86.68%   -0.11%     
==========================================
  Files          43       44       +1     
  Lines        4313     4332      +19     
==========================================
+ Hits         3743     3755      +12     
- Misses        570      577       +7     
Impacted Files Coverage Δ
src/color.rs 49.43% <0.00%> (+0.27%) ⬆️
src/flags/display.rs 98.11% <0.00%> (-1.89%) ⬇️
src/meta/windows_utils.rs 0.00% <0.00%> (ø)
src/app.rs 75.71% <0.00%> (+0.35%) ⬆️
src/meta/mod.rs 43.47% <0.00%> (+1.17%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@meain
Copy link
Member

meain commented Nov 14, 2022

Thanks for the PR! :D

@meain meain merged commit c2cf835 into lsd-rs:master Nov 14, 2022
@brainsik
Copy link

brainsik commented Feb 11, 2023

Looking at this code, it only fixes inode colorings. Coloring links according to theme still doesn't work.

Ensured I'm using the latest (master) and unset LS_COLORS:

$ cargo install --list
lsd v0.23.1 (https://github.com/Peltoche/lsd.git?branch=master#ea56a7c3):
    lsd
$ unset LS_COLORS # to be sure this isn't causing problems

Configured theme to make it obvious it was working or not:

links:
  valid: [255, 121, 198] # Pink
  invalid: [255, 121, 198] # Pink

Output:

image

@cochon-git
Copy link

Coloring links according to theme still doesn't work.

Only just arrived here as a recent convert to lsd, searching for this exact colour problem. However there seems to be confusion here over terminology and expectations with respect to the term 'link'.

The original issue, my wish and indeed this fix was to control the colour of the inode and [hard] link count columns displayed by lsd -1 --blocks links,inode not to highlight valid/hanging symlinks which can at least be controlled by LS_COLORS.

The terminology in colors.yaml certainly seems to imply symlink validity however, which is confusing, as the 'invalid:' setting seems irrelevant to the links column, and would be a fringe abnormality for the inode column too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants