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

move item contents info to bottom of iteminfo screen #40260

Merged
merged 1 commit into from
May 6, 2020

Conversation

KorGgenT
Copy link
Member

@KorGgenT KorGgenT commented May 6, 2020

Summary

SUMMARY: None

Purpose of change

If an item has a lot of contents, it really clogs up the info screen because it shows the name and description of each item. So i moved the full list of contents to the bottom - I did not delete this because this also shows contents of magazines, which do not show as pockets in the item info screen as of now.

Describe alternatives you've considered

Leave it there

Testing

started the game, looked at an item with a bunch of stuff in it

Additional context

This PR is mostly due to some complaints i've heard from players - and I agree with them. It will also be worth coming in later and condensing some of the pocket contents data to show multiple items of the same type in one line.

@KorGgenT KorGgenT added Info / User Interface Game - player communication, menus, etc. Quality of Life QoL: degree to which players are comfortable, and able to enjoy CDDA labels May 6, 2020
@ZhilkinSerg ZhilkinSerg merged commit 79c0d14 into CleverRaven:master May 6, 2020
@KorGgenT KorGgenT deleted the move-item-contents-info branch August 13, 2020 04:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

2 participants