-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Can't skin skinless #34221
Can't skin skinless #34221
Conversation
uilist smenu; | ||
smenu.desc_enabled = true; | ||
smenu.text = _( "Choose type of butchery:" ); | ||
|
||
smenu.addentry_col( BUTCHER, enough_light, 'B', _( "Quick butchery" ), cut_time( BUTCHER ), | ||
const std::string cannot_see = colorize( _( "can't see!" ), c_red ); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe change it to something like "too dark"? Otherwise players might think that there are some problems with their eyes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Huh, so what happens if the player is boomered etc and actually cannot see? Will they be able to start butchering or is this message displayed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i didn't check with boomered, but i checked with a blindfold.
Summary
SUMMARY: Bugfixes "Skinning action no longer available when corpse has no skin to begin with"
Purpose of change
Fixes #28209
and reduces keypresses for butchery when dark.
Describe the solution
Checks the harvest list of the mtype related to the item when butchering, and puts that logic into the UI
Additional context
The text descriptions in the code were very long, so i broke them up a little bit.