Skip to content

Feature request: more color setup #179

New issue

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

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

Already on GitHub? # to your account

Open
EmilyGraceSeville7cf opened this issue Dec 1, 2021 · 3 comments
Open

Feature request: more color setup #179

EmilyGraceSeville7cf opened this issue Dec 1, 2021 · 3 comments

Comments

@EmilyGraceSeville7cf
Copy link

EmilyGraceSeville7cf commented Dec 1, 2021

I suggest split TLDR_COLOR_COMMAND into the two variables:

Variable Meaning
TLDR_COLOR_COMMAND Color for only the first token in the code example
TLDR_COLOR_OPTION Color for only the long/old/short options and flags

and add envs to control braces and pipe symbol colors:

Variable Meaning
TLDR_COLOR_BRACE Color for the opening/closing braces
TLDR_COLOR_PIPE Color for only the pipe symbol

Maybe I can try to create PR for this issue. ;)

@MasterOdin
Copy link
Collaborator

Getting to be quite a lot of environment variables, and time to have a configuration file instead, especially on adding more specific theming stuff.

Might be good to look at interop with the tldr-node-client and move to having a .tldrrc file that you can use to configure settings, and follow what they have there for color options: https://github.com/tldr-pages/tldr-node-client#configuration

Could then add the brace and pipe on top of that, with a suggestion to add it there.

@EmilyGraceSeville7cf
Copy link
Author

EmilyGraceSeville7cf commented Dec 2, 2021

Well, as I understood we have to use environment variables for basic customization and .tldrrc for advanced one. Right? What has more priority: environment variables or .tldrrc?

@MasterOdin
Copy link
Collaborator

Eh, for now, I'm fine with you just implementing these as new environment variables. I'll make a separate issue to track implementing .tldrrc.

What has more priority: environment variables or .tldrrc?

environment variables should have top priority, so that it allows doing TLDR_LANGUAGE=es in a given terminal session and just use that, without overwriting the default permanently.

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

No branches or pull requests

2 participants