Join GitHub today
GitHub is home to over 36 million developers working together to host and review code, manage projects, and build software together.
Sign upRecolor all book icons systematically #7461
Conversation
This comment has been minimized.
This comment has been minimized.
|
Might want to add the color keys to doc/JSON_INFO.md for future references and for modders. |
This comment has been minimized.
This comment has been minimized.
|
@BevapDin: It is done. |
This comment has been minimized.
This comment has been minimized.
|
Gonna object to "waste book" as it further calls out the recipe_anarch and _elfa books v. their decoys. Likewise, I think we can get by with fewer colors: no need to color-code for different skill-thresholds or morale-boosts. Color-coding religious v. non-religious fun-books might be reasonable (religious books are usually visually distinctive) but seeing something across the room and knowing that it's +3 to morale seems over-mechanical. |
This comment has been minimized.
This comment has been minimized.
|
Oh, I have forgot to mention that I made exceptions for those two decoys (and their companion books). I’ve thought of that. The decoy books are the “color givers” and their companions simply use that color instead. Well, maybe at least one of the colors for morale boosters can be taken away for those religious books. |
This comment has been minimized.
This comment has been minimized.
|
Cool! |
This comment has been minimized.
This comment has been minimized.
|
I'm not really a big fan of the concept of color coding based on meta-game properties of the items, particularly different levels of the same thing. If we're color-coding, it'd rather it align with broad, visually-distinctive properties of the items, like large reference manuals vs novels vs magazines, etc. |
This comment has been minimized.
This comment has been minimized.
|
So, any suggestions for a different color code?
On the other hand, I think your argument about giving the player knowledge about meta-data just by looking at is flawed, since the color is really just a shortcut. The information exposed with the color is nothing secret to the player. The player can currently expose to which level a book will teach and to what degree the book will boost the morale just by examining them. With the |
This comment has been minimized.
This comment has been minimized.
|
Exposing all that information at a distance is a flaw in my opinion. |
This comment has been minimized.
This comment has been minimized.
But then you can just knock it down to "pink book" or "cyan book", Kevin! Think of the bloat! :-P was joking; agrees that magazines should be distinct from novels, should be distinct from textbooks, should be distinct from technical docs/binders, should be distinct from religious texts, etc. Maaayyybe color-code by skill (cookbooks look different than comp-sci, etc) too, but that's the extent to which we should color-code, IMO. |
Wuzzy2
referenced this pull request
May 8, 2014
Merged
Recolor all books systematically (second try) #7587
This comment has been minimized.
This comment has been minimized.
|
I close this issue as outdated and in favor of #7587. |
Wuzzy2 commentedApr 28, 2014
The book colors got a bit messy lately. This PR changes the color of books systematically.
Color key:
There is also a thread with more details about my intentions here: http://smf.cataclysmdda.com/index.php?topic=6289.0