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

Option to change look of cursors in parent dirs, rename an option. #1106

Merged
merged 2 commits into from
Feb 25, 2023

Conversation

ilyagr
Copy link
Collaborator

@ilyagr ilyagr commented Feb 12, 2023

This follows up on #1086.

The new cursorparentfmt option defaults to the same value as cursorfmt, so the default look of lf is unchanged. However, I find it useful to make it different in my config, since I seem to have so much trouble figuring out which of my cursors is the active one. :)

For example, here's what it looks like if one has set cursorparentfmt "\033[7;90m" in their lfrc:

image


In a separate commit, it also seemed appropriate to rename cursorfmt into cursoractivefmt at the same time, but this can easily be undone.

This PR does nothing to resolve the issue from #1086 where @gokcehan and @laktak weren't very happy with the work "cursor", but if we can come up with a better word, I can rename it here.

@ilyagr ilyagr force-pushed the cursorparentfmt branch 4 times, most recently from 15298a4 to 56bbe8d Compare February 12, 2023 08:02
@ilyagr ilyagr changed the title Add option to change look of cursors in parent dirs, rename an option. Option to change look of cursors in parent dirs, rename an option. Feb 12, 2023
The option defaults to the same value as `cursorfmt`, so the default look of `lf` is unchanged. However, I find it useful to make it different, since I seem to have so much trouble figuring out which of my cursors is the active one. :)

For example, here's what it looks like if one has `set cursorparentfmt "\033[7;90m"` in their `lfrc`:

https://user-images.githubusercontent.com/4123047/218299130-09ac411f-307a-489e-a4f0-4691710f8549.png
This seems appropriate after introducing `cursorparentfmt`
@gokcehan gokcehan merged commit 6afefa5 into gokcehan:master Feb 25, 2023
@ilyagr ilyagr deleted the cursorparentfmt branch February 25, 2023 22:19
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.

2 participants