Skip to content

[Feature] Selectable Containers#80

Open
Dirius77 wants to merge 3 commits intoTheDenSS14:masterfrom
Dirius77:selectable-lockers
Open

[Feature] Selectable Containers#80
Dirius77 wants to merge 3 commits intoTheDenSS14:masterfrom
Dirius77:selectable-lockers

Conversation

@Dirius77
Copy link

@Dirius77 Dirius77 commented Mar 20, 2026

Allows the creation of containers which cause the user to select their contents when first used.

Why / Balance

This will be useful for things later (MODsuits), but is also generic enough to be a separate system if anyone has a use for it.

Technical details

Creates a new component which opens a UI when a container is interacted with, once a selection is made the component makes the UI impossible to open, allowing the container to be opened and interacted with as normal.

Media

2026-03-20_03-08-24.mp4

Requirements

  • I have read and am following the Pull Request and Changelog Guidelines.
  • I have added media to this PR or it does not require an in-game showcase.
  • I have tested my changes and additions in-game.

Licensing

  • All code in this pull request can be licensed to MIT.
  • (OPTIONAL) I give permission to seeing this feature upstreamed to Macrocosm in the future.

Breaking changes

Enforces that ActivatableUI handles interaction events before EntityStorageSystem and StorageSystem. This was already the case for at least EntityStorageSystem, but it WASN'T the case on prebase, so I thought it best to ensure that order was strictly defined.

Changelog
This isn't user facing.

@Dirius77 Dirius77 changed the title Selectable Containers [Feature] Selectable Containers Mar 20, 2026
@Dirius77 Dirius77 force-pushed the selectable-lockers branch from e2693e4 to 6081e69 Compare March 20, 2026 07:45
Copy link
Contributor

@portfiend portfiend left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeappp 👍

@Dirius77
Copy link
Author

Did a pass of adding comments and cleaning up the UI code. Also made a tiny tweak to EntityTableSelector so that ListSpawns doesn't lie about entries that are using the amount field. I'll probably push that little tweak upstream as well in a bit since the function just DoesNotDoWhatItSaysItDoes without it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants