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

feat(std/fmt): add bright color variations #7241

Merged
merged 1 commit into from
Aug 29, 2020

Conversation

caspervonb
Copy link
Contributor

@caspervonb caspervonb commented Aug 29, 2020

This adds bright colors variations of the colors defined in std/fmt/colors.ts.

Closes #7240

Comment on lines +123 to +126
return brightBlack(str);
}

export function brightBlack(str: string): string {
Copy link
Member

Choose a reason for hiding this comment

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

Is change to gray intentional?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, did not want to remove any identifiers here but gray was returning 90 which is "bright black" so grey becomes an alias of it.

Copy link
Contributor Author

@caspervonb caspervonb Aug 29, 2020

Choose a reason for hiding this comment

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

For reference, Wikipedia has a pretty color table https://en.wikipedia.org/wiki/ANSI_escape_code#Terminal_output_sequences

Copy link
Member

@bartlomieju bartlomieju left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @caspervonb

@bartlomieju bartlomieju merged commit 8ca903f into denoland:master Aug 29, 2020
caspervonb added a commit to caspervonb/deno_std that referenced this pull request Jan 21, 2021
caspervonb added a commit to caspervonb/deno_std that referenced this pull request Jan 24, 2021
caspervonb added a commit to caspervonb/deno_std that referenced this pull request Jan 24, 2021
caspervonb added a commit to caspervonb/deno_std that referenced this pull request Jan 24, 2021
caspervonb added a commit to caspervonb/deno_std that referenced this pull request Jan 31, 2021
caspervonb added a commit to caspervonb/deno_std that referenced this pull request Jan 31, 2021
caspervonb added a commit to caspervonb/deno_std that referenced this pull request Jan 31, 2021
caspervonb added a commit to caspervonb/deno_std that referenced this pull request Jan 31, 2021
caspervonb added a commit to caspervonb/deno_std that referenced this pull request Feb 1, 2021
@caspervonb caspervonb deleted the feat-std-fmt-bright-colors branch May 24, 2021 13:48
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.

fmt/color.ts: add support for bright variants
2 participants