zD12 / pircbotx

Automatically exported from code.google.com/p/pircbotx
0 stars 0 forks source link

Update colors class #168

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
The Color class is almost 100% original PircBot code mostly because finding 
good documentation on IRC colors is difficult. It also just works right now. 

Possible features
 - Split up the line into a map with the color as the key and whatever part of the line has that color as the value
 - Better as an enum?
 - Lookup color value by name

Original issue reported on code.google.com by Lord.Qua...@gmail.com on 21 Feb 2014 at 12:46

GoogleCodeExporter commented 9 years ago
Given the fact that these color codes are just a list of members with a certain 
value that is part of the mIRC documentation I see not reason to consider it 
copyrighted. It just doesn't reach any kind of level that could become 
copyrighted. Of course that does not aply to the remove* methods in that class.

Original comment by firewi...@googlemail.com on 22 Jul 2014 at 12:28

GoogleCodeExporter commented 9 years ago
Consider adding the (mIRC?) italics character:  
Possibly add some support for fontcolor,backgroundcolor? 
Currently Colors.RED + "," + Colors.BLACK is a bit silly looking.

Original comment by Tnade...@gmail.com on 20 Nov 2014 at 2:26

GoogleCodeExporter commented 9 years ago
Added Colors.ITALICS ( Revision c98c8aa369ce ) and Colors.LOOKUP_TABLE and 
Colors.lookup() ( Revision: 8773dc8873f6 )

Decided against making it an enum (stupid) and rewriting the remove* methods 
(yes they are GPL'd but they work and were not close to removing all the code. 
Better to do later)

Original comment by Lord.Qua...@gmail.com on 20 Nov 2014 at 4:40