colors.js icon indicating copy to clipboard operation
colors.js copied to clipboard

Added support for orange and bgOrange

Open abhishekc-sharma opened this issue 8 years ago • 12 comments

Fixes #123

abhishekc-sharma avatar Oct 24 '15 11:10 abhishekc-sharma

+1, halloween is near

petrosagg avatar Oct 27 '15 14:10 petrosagg

+1

ryanmurakami avatar Nov 13 '15 08:11 ryanmurakami

Was this ever merged?

mikeerickson avatar Jan 03 '16 18:01 mikeerickson

@portgasd666 Was this ever merged? If not, is there a branch which has this added?

mikeerickson avatar Dec 29 '16 23:12 mikeerickson

somebody click "merge"

ORESoftware avatar Dec 30 '16 00:12 ORESoftware

Doesnt work for my computer. And ansi code 38 is fully supported for my machine

Andrews54757 avatar Jan 16 '17 03:01 Andrews54757

Is there some reason orange isn't always supportable? See comments at bottom of #123 ... generally this seems like it should be merged, but feels like a little extra clarity is needed (either here, in the docs, etc.) about why orange is different and why it might/might not work in different environments...

DABH avatar Feb 12 '18 19:02 DABH

@Marak please merge thx

ORESoftware avatar Feb 12 '18 20:02 ORESoftware

Orange is not an ANSI color, would something like #76 be preferred? (Just all-out support custom colors... why stop at orange...)

DABH avatar Feb 14 '18 00:02 DABH

I'm guessing if certain colors and styles work on some systems ( but not all systems ), we should consider adding the API methods.

For example, if orange method is supported for some shells we should allow that color. If a user attempts to use orange in a non-supported environment, we could produce a warning message based on feature detection of the environment.

I'm guessing for additional colors we'll want to add true color / 24 bit color mode. Maybe look at https://github.com/irisli/fullcolor for inspiration. Might make sense to wrap all this under that functionality.

Marak avatar Feb 14 '18 01:02 Marak

see: https://gist.github.com/XVilka/8346728 for more details

Marak avatar Feb 14 '18 01:02 Marak

+1 for orange!

mesqueeb avatar Oct 16 '18 05:10 mesqueeb