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

Add custom item color in lists for magic weapons/items #285

Closed
wants to merge 43 commits into from

Conversation

Nagath
Copy link
Contributor

@Nagath Nagath commented Nov 13, 2014

Quality of Life improvement:
coded by skittles1

Any weapon with an enchantment (cursed, glow, holder, Colhorr weapon, enchant weapon etc.) will show up in a separate color in all lists. Currently this defaults to orange and can be set by the client with other list colors (Options -> Colors -> List boxes -> Magic items).

All weapons and any 'defense modifier' (armor, shields, robes, etc.) with an enchantment will show up in either orange, the default color, or whatever the user chooses.

@@ -61,6 +62,6 @@ void ColorsRestoreDefaults(void);
M59EXPORT HBRUSH DialogCtlColor(HWND hwnd, HDC hdc, HWND hwndChild, int type);
HBRUSH MainCtlColor(HWND hwnd, HDC hdc, HWND hwndChild, int type);

WORD GetItemListColor(HWND hwnd, int type);
WORD GetItemListColor(HWND hwnd, int type, int flag);
Copy link
Contributor

Choose a reason for hiding this comment

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

Should be flags, not flag.

moved viObject_flags = APPLY_YES | GETTABLE_YES to wand.kod
moved viObject_flags = APPLY_YES | GETTABLE_YES from spellitem.kod to wands.kod
corrected flag to flags
@Nagath
Copy link
Contributor Author

Nagath commented Jan 2, 2015

both adjusted

@jrotunda
Copy link

This would be a good addition for visual information and intuitive game play, currently there is no way to tell if an item is enchanted without clicking on the description, and would make for a nice visual if also combined with the game's colored lighting system.

@Koveras
Copy link
Contributor

Koveras commented Feb 1, 2015

+1. this sounds like a great idea!

MorbusM59 pushed a commit to MorbusM59/Meridian59 that referenced this pull request Dec 11, 2016
Fix display of news articles and mail posted from Ogre client.
@davidklaw
Copy link
Contributor

davidklaw commented Jul 4, 2022

Seems like a positive change for the reasons described above, however, it's long out of date and the conflict with clientd3d/resource.h is one I'm not sure how to get past, so I can't compile and provide some test results.

@davidklaw
Copy link
Contributor

@Meridian59 Abandoned and a candidate for closure.

@akirmse akirmse closed this Jul 13, 2022
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.

None yet

6 participants