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

Revise durability messages again, showing before/after transitions #30183

Merged

Conversation

esotericist
Copy link
Contributor

@esotericist esotericist commented May 2, 2019

Summary

SUMMARY: Interface "Revise durability messages again, showing before/after transitions"

Purpose of change

In the comments of #30167, when I last touched these messages, there was feedback suggesting the new format of the messages was not universally intuitive, and requested a 'before and after' version. On reflection I realized a variation to that effect should be more generally communicative to players what has changed and how.

Describe the solution

Shifted the majority of the messages from the format of the You action your ?? thing! to You action your thing! ( ?? -> ??), with the exception of vehicle repairs, which simply append (was ??) of the messages I added (since vehicle repairs cannot presently fail, only be interrupted).

To further enable this, moved a lot of the prefix logic out of item::tname() into a new method, item::durability_indicator(), which works whether the player has the Item Health Bars configuration setting enabled or not.

Describe alternatives you've considered

Coulda left well enough alone? While I'm reasonably confident the last change I made was a benefit to more people than a detriment, I'd rather try to make it a genuine improvement for as many people as possible.

Additional context

Unsuccessful repair:
image

Successful repairs:
image

Without item health bars:
image

Vehicle:
image

@ifreund ifreund added <Enhancement / Feature> New features, or enhancements on existing [C++] Changes (can be) made in C++. Previously named `Code` Info / User Interface Game - player communication, menus, etc. Items / Item Actions / Item Qualities Items and how they work and interact labels May 2, 2019
@Night-Pryanik
Copy link
Contributor

I suggest move the condition change part ( ?? -> ?? ) to the place after you repair/damage part. So it would look like You repair ( |. -> |\ ) your pair of light gloves!.

@kevingranade kevingranade merged commit 516a9a9 into CleverRaven:master May 4, 2019
@tinukedaya
Copy link
Contributor

This is definitely an improvement! I like this.

@esotericist esotericist deleted the durability-messages-redux branch August 3, 2019 02:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C++] Changes (can be) made in C++. Previously named `Code` <Enhancement / Feature> New features, or enhancements on existing Info / User Interface Game - player communication, menus, etc. Items / Item Actions / Item Qualities Items and how they work and interact
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants