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

[still cursed] Volumetric Storage #2377

Merged
merged 30 commits into from
May 20, 2024

Conversation

Sun-Soaked
Copy link
Member

@Sun-Soaked Sun-Soaked commented Sep 27, 2023

About The Pull Request

Revival of This PR

Back storage is now organized visually using volumetric objects of different sizes, like bay.
This will require testing and balancing, as well as potential re-weighting, since the default code allows much larger objects to be placed in storage(albiet taking up a whole lot of space.)
So this is a draft for now

Also ports unique trash bag behavior

May implement unique behavior for different storage objects, also like on bay.

Why It's Good For The Game

Sudie wanted me to revive this

Changelog

🆑
add: Volumetric storage on backpacks.
add: trash bags now have more storage space, but only make a small amount of their contents accessible at any given time.
/:cl:

@Sun-Soaked Sun-Soaked requested a review from a team as a code owner September 27, 2023 15:15
@github-actions github-actions bot added DME Edit Sprites A bikeshed full of soulless bikes. Code change Watch something violently break. labels Sep 27, 2023
@Sun-Soaked Sun-Soaked marked this pull request as draft September 27, 2023 15:16
-makes ui less bad
-makes bigger objects fit in trash bags(it's a trash bag goofus)
@Sun-Soaked
Copy link
Member Author

shut up linters

I am the trash man
huh??? huh? huh???
these were here already. I cut you in half with a huge sword
@tmtmtl30
Copy link
Contributor

Is there much reason to make "volume" separate from "weight class"?

@thgvr
Copy link
Member

thgvr commented Oct 13, 2023

is this in a tm ready state or is it silly

@github-actions github-actions bot added the Merge Conflict Use Git Hooks, you're welcome. label Oct 17, 2023
@github-actions
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@thgvr
Copy link
Member

thgvr commented Oct 29, 2023

ooooooo you want to worko n this

@Sun-Soaked
Copy link
Member Author

ooooooo you want to worko n this

I grab the papers in front of me with my teeth and shake them around really fast
(ughhh ouugghhh aoooouggghhh)

Okay but seriously these are the two issues:

-I wanted to do the "backpacks can only be opened inhand"/"duffels can only be opened from floor" thing, and the code I made for it is jank/doesn't work properly

-The ui is still janky and flickers/overlaps. This is how it is on Cit as well, so I need to do a deep dive into how the UI works to see even what the problem is, and then start to think of a solution

@thgvr
Copy link
Member

thgvr commented Nov 1, 2023

I'd leave the inventory access stuff for another pr, it's important enough to warrant it

@github-actions github-actions bot removed the Merge Conflict Use Git Hooks, you're welcome. label Nov 6, 2023
OOPSIES
@thgvr
Copy link
Member

thgvr commented Nov 7, 2023

Fix your lints

@rye-rice
Copy link
Member

Fix your lints

linters were fixed lizard boy

@thgvr
Copy link
Member

thgvr commented Feb 4, 2024

maybe we can tm it........

@Sun-Soaked Sun-Soaked marked this pull request as ready for review February 19, 2024 16:51
@Sun-Soaked
Copy link
Member Author

confetti (tired voice)

@meemofcourse
Copy link
Contributor

god

@Sun-Soaked Sun-Soaked changed the title [still cursed] [prototype] Volumetric Storage [still cursed] Volumetric Storage Feb 19, 2024
@github-actions github-actions bot added the Merge Conflict Use Git Hooks, you're welcome. label Feb 28, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@thgvr thgvr added the Test Merge Required They'll have to do it for free. label Mar 12, 2024
@github-actions github-actions bot removed the Merge Conflict Use Git Hooks, you're welcome. label Mar 12, 2024
@Gorionus Gorionus mentioned this pull request Mar 27, 2024
@Bimbly
Copy link

Bimbly commented Mar 30, 2024

Backpacks and satchels have different storage space amounts despite them having the same move speed still, which I assume is an oversight. We tested this specifically with the captains satchel and explorer bag.

@rye-rice rye-rice added this pull request to the merge queue May 20, 2024
Merged via the queue into shiptest-ss13:master with commit 8f86c62 May 20, 2024
14 checks passed
@rye-rice
Copy link
Member

Oops hand slipped and clicked merge on 20 prs my bad

MysticalFaceLesS pushed a commit to CeladonSS13/Shiptest that referenced this pull request May 25, 2024
<!-- Write **BELOW** The Headers and **ABOVE** The comments else it may
not be viewable. -->
<!-- You can view Contributing.MD for a detailed description of the pull
request process. -->

## About The Pull Request
Revival of [This PR](shiptest-ss13#850)

Back storage is now organized visually using volumetric objects of
different sizes, like bay.
This will require testing and balancing, as well as potential
re-weighting, since the default code allows much larger objects to be
placed in storage(albiet taking up a whole lot of space.)
So this is a draft for now

Also ports unique trash bag behavior

May implement unique behavior for different storage objects, also like
on bay.
<!-- Describe The Pull Request. Please be sure every change is
documented or this can delay review and even discourage maintainers from
merging your PR! -->

## Why It's Good For The Game
Sudie wanted me to revive this
<!-- Please add a short description of why you think these changes would
benefit the game. If you can't justify it in words, it might not be
worth adding. -->

## Changelog

:cl:
add: Volumetric storage on backpacks.
add: trash bags now have more storage space, but only make a small
amount of their contents accessible at any given time.
/:cl:

<!-- Both :cl:'s are required for the changelog to work! You can put
your name to the right of the first :cl: if you want to overwrite your
GitHub username as author ingame. -->
<!-- You can use multiple of the same prefix (they're only used for the
icon ingame) and delete the unneeded ones. Despite some of the tags,
changelogs should generally represent how a player might be affected by
the changes rather than a summary of the PR's contents. -->

---------

Co-authored-by: Sun-Soaked <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code change Watch something violently break. DME Edit Sprites A bikeshed full of soulless bikes. Test Merge Required They'll have to do it for free.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants