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

[Feature Request] Add alignment/ordering options for Stat Summary stat names and values #96

Open
raethkcj opened this issue Sep 16, 2022 · 4 comments
Labels
enhancement New feature or request

Comments

@raethkcj
Copy link
Owner

raethkcj commented Sep 16, 2022

Originally posted by @thehallowedfire in #9 (comment)

Not sure if I should open another issue. Will write here for now.
Want to make addon a little bit more shiny and closer to the original game look.

This is a screenshot from the original 3.3.5a client. I don't know why we don't have the same stats difference information on WotLK Classic client but w/e. You can see how these stats are ordered: number name_of_stat
image

Also the color of stat name is white which makes it look like other stats of item (look at the item description above on the screenshot). Green/red color for numbers are perfect but this is when you compare item and you see a difference between two items. I'm not sure what color would be better for the case when you just see stat summary. Perhaps it should be white as well as on basic item stats.
We have additional information such as converted from ratings crit chance/haste/etc. These numbers (not stat names) could have more contrast color. I personally changed color to cyan just to make it look different from original numbers.
image
(this is how it looks in game now for me)

image
(this is how I would like it to look)

On the last screenshot you can see text:

29.5 Spell Power
0.21% Critical Strike Chance    ## It uses cyan color because this stat is calculated from other stats and it isn't actually 'present' on the item
1.01% Spell Haste    ## Same as with previous stat it uses cyan because it is calculated. Text is considerable, maybe it should be just 'Haste', not sure

And I moved item level text (on russian it's 'Уровень предмета: 123', on english it's 'Item Level 123') to its original position but not sure if it's possible for you to implement text this way. And it's white.


I'm not sure what color to use for 'calculated stats' when we compare items.
Drew two options:
image
and
image

I can't really choose the best and more suitable.

@raethkcj raethkcj changed the title Not sure if I should open another issue. Will write here for now. [Feature Request] Add font color options for Stat Summary stat names and values Sep 16, 2022
@raethkcj raethkcj added the enhancement New feature or request label Sep 16, 2022
@raethkcj
Copy link
Owner Author

Adding color options for the stat names and values in the summary is pretty reasonable and straightforward.

Having a separate color for "calculated" stats is more complex than it may seem; for example, many items have Attack Power directly on them, but might also provide Attack Power from Agility, Intellect, Stamina, or Armor.

Allowing the user to choose left/right alignment for both names and values is in theory doable, but I'd have to think how I want the settings menu for it to look.

@thehallowedfire
Copy link

Having a separate color for "calculated" stats is more complex than it may seem; for example, many items have Attack Power directly on them, but might also provide Attack Power from Agility, Intellect, Stamina, or Armor.

Well you're right. So if we don't compare items the numbers probably should have just the same color (white or custom like cyan).

raethkcj added a commit that referenced this issue Sep 20, 2022
@raethkcj
Copy link
Owner Author

raethkcj commented Sep 22, 2022

The options to change the color of text and numbers were added in 1.5.8 (they still need to be localized). Changing the left/right position will take more time.
image

@raethkcj
Copy link
Owner Author

raethkcj commented Sep 22, 2022

ItemLevel and ItemID aren't getting colored by this change, I'll add those.

@raethkcj raethkcj changed the title [Feature Request] Add font color options for Stat Summary stat names and values [Feature Request] Add alignment/ordering options for Stat Summary stat names and values Aug 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants