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

Standardize player info window colors #36178

Merged
merged 1 commit into from
Dec 18, 2019

Conversation

mark7
Copy link
Contributor

@mark7 mark7 commented Dec 17, 2019

Summary

SUMMARY: Interface "Standardize player info colors"

Purpose of change

Previously, the colors used in the player info window had varied from
tab to tab. Make all tabs conform to the color convention used in the
STATS tab: Magenta is reserved for text that must be set off from the
rest, white for numeric stats to make them stand out, light gray for
everything else.

Added one missing period to "hands" encumbrance description.

Testing

Hit "@" to bring up the player info window, scrolled through each tab.

@anothersimulacrum
Copy link
Member

Could you show pictures?

@mark7
Copy link
Contributor Author

mark7 commented Dec 17, 2019

Hmm. I can do a before/after of the encumbrance tab and a before/after of the effects tab, which are probably the most-interesting -- the others are really just changing to use light gray for the text in the description area.

enc_before
enc_after

effects_before
effects_after

@mark7
Copy link
Contributor Author

mark7 commented Dec 17, 2019

Each of these originally just used magenta in the description area; now the description is light gray, with the stats numbers highlighted in white.

@Night-Pryanik
Copy link
Contributor

Maybe make each value start on a new line, like the values in STATS tab?

@mark7
Copy link
Contributor Author

mark7 commented Dec 17, 2019

In the encumbrance tab? Sure, looks like there's room for all the current (non-localized) text to put each stat on a line for each body part if you want...but note that that the field is pretty tight on space, and it might be preferable to leave space to describe any future mechanics that are affected. That'd take the hands description down to only one blank line remaining.

@mark7
Copy link
Contributor Author

mark7 commented Dec 17, 2019

Eh, I'll just go ahead and do it. Someone can go change the field up if they need more space.

Previously, the colors and format used on the player info window had
varied from tab to tab.  Make all tabs conform to the color convention
used in the STATS tab: Magenta is reserved for text that must be set
off from the rest, white for numeric stats to make them stand out,
light gray for everything else.

Convert "sentence"-based structure of ENCUMBRANCE AND WARMTH tab to
one-line-per-stat format.
@mark7 mark7 force-pushed the wb-playerstatus-color branch from 559e4cf to a034395 Compare December 17, 2019 06:57
@mark7
Copy link
Contributor Author

mark7 commented Dec 17, 2019

effects_torso_after
effects_hands_after

@Night-Pryanik
Copy link
Contributor

Yeah, that's what I meant.

@I-am-Erk I-am-Erk added [C++] Changes (can be) made in C++. Previously named `Code` Accessibility Issues regarding accessibility Info / User Interface Game - player communication, menus, etc. Quality of Life QoL: degree to which players are comfortable, and able to enjoy CDDA labels Dec 17, 2019
@mark7
Copy link
Contributor Author

mark7 commented Dec 17, 2019

The continuous integration failure doesn't look related to the patch to me.

@ZhilkinSerg ZhilkinSerg merged commit e8be1ac into CleverRaven:master Dec 18, 2019
fengjixuchui added a commit to fengjixuchui/Cataclysm-DDA that referenced this pull request Dec 19, 2019
Merge pull request CleverRaven#36178 from mark7/wb-playerstatus-color
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accessibility Issues regarding accessibility [C++] Changes (can be) made in C++. Previously named `Code` Info / User Interface Game - player communication, menus, etc. Quality of Life QoL: degree to which players are comfortable, and able to enjoy CDDA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants