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

Spoilable Items Clog Inventory Views #24498

Closed
OrenAudeles opened this issue Jul 27, 2018 · 5 comments

Comments

Projects
None yet
5 participants
@OrenAudeles
Copy link
Contributor

commented Jul 27, 2018

Issue:

Spoilable items in Cataclysm currently do not stack if they have differing rot timers. This will often lead to situations where batch crafting comestibles such as Pemmican, Fruit Leather, in large amounts will fill the inventory with many stacks that cannot be combined. As time-to-rot extends into the future the player gets progressively less concerned with that value.

Discussion Topic: How can this issue be solved?

My opinion:

I think that a progressive granularity system would work best. This system would group like items that are unable to be stacked due to differing rot times into more useful sets. I propose the following groupings:

  • 1 year +
  • 1 season - 1 year
  • 1 week - 1 season
  • 1 day - 1 week
  • 1 turn - 1 day

At less than 1 day until spoilage it might be undesirable to group comestibles

Selecting a comestible for consumption from a granular group should consume an item that will rot at the nearest future point in time.

Implementation of this granular grouping may be done entirely as a UI construction whenever opening a relevant UI. There will be times that finer detail grouping is necessary, and a sorting option/toggle may be desirable. Because grouping is a UI construction it will have no effect on the comestibles directly and allow them to be grouped dynamically as conditions change.

Example Cluttered Inventory: Red boxed comestibles can be compressed (not comprehensive)
cata_food_stacking

Example Grouped Inventory: Red arrowed comestibles have been compressed (not comprehensive)
cata_food_stacking_compressed

Please share your thoughts, concerns, and ideas so we can find a viable solution to this issue.

@cainiaowu

This comment has been minimized.

Copy link
Contributor

commented Jul 27, 2018

Actually there are already some info about item freshness in the item info.
We really need to group items with same freshness into one group for display.

@FulcrumA

This comment has been minimized.

Copy link
Contributor

commented Jul 27, 2018

Stacking in general seems to be requiring some work and is inconsistent on several fronts (#23528, #23946, #23933).

@kevingranade

This comment has been minimized.

Copy link
Member

commented Jul 27, 2018

I prefer your approach of doing the stacking at the UI level, there are all kinds of issues caused by the fact that we try to stack things at the item level.

@kevingranade

This comment has been minimized.

Copy link
Member

commented Jul 27, 2018

To be more concrete:
Re: #23528 you want to be able to read the different snippets sometimes, but other times it's clutter.
Menus might want to do grouping differently.

@kevingranade

This comment has been minimized.

Copy link
Member

commented Jan 10, 2019

Fixed by #26523

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.