Added recipe activity level to crafting screen. #49026
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Maybe adding a color corresponding to the level of the activity? the higher it is the more red the text gets? |
|
@Fris0uman: Too fiddly imho (not enough shades of red in curses) and also VERY unfriendly to color-blind folks. |
actual-nh
added
0.F Feature Freeze
<Enhancement / Feature>
|
Ok but maybe a grey/yellow/red thing then, something that tells you "pay attention this is exhausting work" |
Co-authored-by: Saicchi <[email protected]>
wapcaplet
added a commit
to wapcaplet/Cataclysm-DDA
that referenced
this pull request
Aug 22, 2021
This namespace is presently only used as a wrapper to make `activity_level_str` accessible to the crafting GUI, per CleverRaven#49026. I intend to move several other descriptive functions defined in `panels.h|cpp` into this namespace, making it a more general-purpose collection of descriptive text and color translation methods. Hence, the namespace is here renamed to `display`.
wapcaplet
added a commit
to wapcaplet/Cataclysm-DDA
that referenced
this pull request
Aug 24, 2021
This namespace is presently only used as a wrapper to make `activity_level_str` accessible to the crafting GUI, per CleverRaven#49026. I intend to move several other descriptive functions defined in `panels.h|cpp` into this namespace, making it a more general-purpose collection of descriptive text and color translation methods. Hence, the namespace is here renamed to `display`.
wapcaplet
added a commit
to wapcaplet/Cataclysm-DDA
that referenced
this pull request
Aug 26, 2021
This namespace is presently only used as a wrapper to make `activity_level_str` accessible to the crafting GUI, per CleverRaven#49026. I intend to move several other descriptive functions defined in `panels.h|cpp` into this namespace, making it a more general-purpose collection of descriptive text and color translation methods. Hence, the namespace is here renamed to `display`.
Venera3
pushed a commit
to Venera3/Cataclysm-DDA
that referenced
this pull request
Sep 21, 2021
This namespace is presently only used as a wrapper to make `activity_level_str` accessible to the crafting GUI, per CleverRaven#49026. I intend to move several other descriptive functions defined in `panels.h|cpp` into this namespace, making it a more general-purpose collection of descriptive text and color translation methods. Hence, the namespace is here renamed to `display`.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
Interface "Added recipe activity level to crafting screen"
Purpose of change
I'd like to know if boiling water for three hours will make me tired.
Describe the solution
Added "Activity level" line to the crafting screen.
Describe alternatives you've considered
I'm not very happy with leaving
activity_level_strinpanels.cppif it's used outside, but I didn't find any good place to put it.Testing
Loaded a game and looked at the crafting screen.
Additional context
(I guess it won't)