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

Blue is too dark #15

Closed
Ming-Tang opened this issue Jan 22, 2016 · 4 comments · Fixed by #58
Closed

Blue is too dark #15

Ming-Tang opened this issue Jan 22, 2016 · 4 comments · Fixed by #58

Comments

@Ming-Tang
Copy link

I'm using wscat in a terminal with black background, and when blue is used, I can't see the text. In Vim, I can do set bg=dark to make the text colors lighter. There should be a way to change or disable the colors.

@markwillis75
Copy link

It's not an ideal solution, but it's possible to edit the /npm/node_modules/wscat/bin/wscat file to use the colours you wish.

Locate the .on('message'...) conditional and change the color from Console.Colors.Blue to something more suitable. Select from the existing colors defined in the Console.Colors array, or add new element to the array and use that.

@eaded110
Copy link

eaded110 commented Jul 7, 2016

comment on issue

@genetique-techno
Copy link

Yes, yes it is...

@nwwells
Copy link
Collaborator

nwwells commented Jan 24, 2017

I'd be happy to review a PR for a --no-color option. --raw is also under consideration for #2 and #18, which would also solve this, but no reason we couldn't do both.

iggyvolz added a commit to iggyvolz/wscat that referenced this issue May 30, 2017
iggyvolz added a commit to iggyvolz/wscat that referenced this issue May 30, 2017
iggyvolz added a commit to iggyvolz/wscat that referenced this issue May 30, 2017
@lpinca lpinca closed this as completed in 69d83a6 May 30, 2017
@lotkao lotkao mentioned this issue Dec 3, 2017
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 a pull request may close this issue.

5 participants