Skip to content

Support ln=target in LS Colors (exa PR 960)#1018

Open
jbrubake wants to merge 1 commit into
eza-community:mainfrom
jbrubake:main
Open

Support ln=target in LS Colors (exa PR 960)#1018
jbrubake wants to merge 1 commit into
eza-community:mainfrom
jbrubake:main

Conversation

@jbrubake

@jbrubake jbrubake commented Jun 6, 2024

Copy link
Copy Markdown

Setting "ln=target" highlights a link in the same color as the referred file instead of having a specific symlink color. Dangling/orphaned links are always colored using the specified "or=" color.

(Credit to @LarsHaalck who wrote the original ogham/exa#960)

@jbrubake jbrubake requested a review from PThorpe92 as a code owner June 6, 2024 21:24
@MartinFillon

Copy link
Copy Markdown
Contributor

Hey, thanks for updating the pr, I will let you fix clippy lints error and check for format error in your code, they are mentionned in the checks not passing

@jbrubake jbrubake force-pushed the main branch 4 times, most recently from 9b4bee2 to 4f3c057 Compare June 7, 2024 19:54

@MartinFillon MartinFillon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for your patience and for fixing everything.

@PThorpe92 PThorpe92 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks!

Comment thread src/theme/ui_styles.rs
Target,
}

impl Default for LinkStyle {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: can this be derived?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry. I don't know what that means. I don't actually know Rust. I just took the original patch and tweaked it to work with the current code

If you can point me in the right direction though, I'm happy to take a look

Setting "ln=target" highlights a link in the same color as the
referred file. Dangling/orphaned links are always colored using the
specified "or=" color.

(Credit to @LarsHaalck who wrote the original ogham/exa#960)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: 🆕 New

Development

Successfully merging this pull request may close these issues.

4 participants