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

Configure highlight color for current context & namespace #53

Merged
merged 4 commits into from
Jun 7, 2018

Conversation

prabhu43
Copy link
Contributor

@prabhu43 prabhu43 commented Jun 5, 2018

No description provided.

Copy link
Owner

@ahmetb ahmetb left a comment

Choose a reason for hiding this comment

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

Code LGTM. Left a few comments below.

README.md Outdated

To change background colour to White for highlighting current context:

`export KUBECTX_CURRENT_FGCOLOR=$(tput setaf 7)`
Copy link
Owner

Choose a reason for hiding this comment

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

this should be _BGCOLOR.

README.md Outdated
@@ -73,6 +73,21 @@ Active namespace is "default".

-----

### Highlight Current Context
By default, current context is highlighted by yellow foreground & black background
Copy link
Owner

Choose a reason for hiding this comment

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

no need to include the defaults here. delete this sentence

README.md Outdated
### Highlight Current Context
By default, current context is highlighted by yellow foreground & black background

It is configurable by setting environment variables `KUBECTX_CURRENT_FGCOLOR` & `KUBECTX_CURRENT_BGCOLOR`
Copy link
Owner

Choose a reason for hiding this comment

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

we need to shorten this section. consider the following rewrite.

Customizing current context colors

If you like to customize the colors indicating the current namespace or context, set the environment variables KUBECTX_CURRENT_FGCOLOR and KUBECTX_CURRENT_BGCOLOR:

export KUBECTX_CURRENT_FGCOLOR=$(tput setaf 6) # blue text
export KUBECTX_CURRENT_FGCOLOR=$(tput setaf 7) # white background

Refer color codes here.

README.md Outdated
@@ -73,6 +73,21 @@ Active namespace is "default".

-----

### Highlight Current Context
Copy link
Owner

Choose a reason for hiding this comment

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

move this after "## Installation" and call it the title I described below.

@ahmetb ahmetb mentioned this pull request Jun 6, 2018
@ahmetb
Copy link
Owner

ahmetb commented Jun 7, 2018

@prabhu43 Thanks for the PR. I only requested a few docs changes, when you have time please take a look.

@prabhu43
Copy link
Contributor Author

prabhu43 commented Jun 7, 2018

@ahmetb I updated the doc with your inputs.

@ahmetb ahmetb merged commit 428fb00 into ahmetb:master Jun 7, 2018
@ahmetb ahmetb added this to the v0.6.0 milestone Aug 22, 2018
This pull request was closed.
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