Skip to content

Feature InventoryCleaner module - #229

Open
IceTank wants to merge 11 commits into
lambda-client:1.21.11from
IceTank:feature/module/easy-trash
Open

Feature InventoryCleaner module#229
IceTank wants to merge 11 commits into
lambda-client:1.21.11from
IceTank:feature/module/easy-trash

Conversation

@IceTank

@IceTankIceTank commented Jan 26, 2026

Copy link
Copy Markdown
Contributor

This module adds a feature to auto drop items when you don't have space in your inventory

@IceTank

IceTank commented Jan 26, 2026

Copy link
Copy Markdown
ContributorAuthor
Auto.drop.mp4
Auto.inv.drop.mp4

@beanbag44
beanbag44 self-requested a review January 26, 2026 23:30
beanbag44
beanbag44 requested changes Jan 26, 2026
View reviewed changes

@beanbag44beanbag44 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Really handy module, just some small nit picks and changes

Comment threadsrc/main/java/com/lambda/mixin/render/HandledScreenMixin.java Outdated
Comment threadsrc/main/kotlin/com/lambda/module/modules/player/EasyTrash.kt Outdated
Comment threadsrc/main/kotlin/com/lambda/module/modules/player/EasyTrash.kt Outdated
Comment threadsrc/main/kotlin/com/lambda/module/modules/player/EasyTrash.kt Outdated
Comment threadsrc/main/kotlin/com/lambda/module/modules/player/EasyTrash.kt Outdated
Comment threadsrc/main/kotlin/com/lambda/module/modules/player/EasyTrash.kt Outdated
emyfops
emyfops suggested changes Jan 27, 2026
View reviewed changes
Comment threadsrc/main/java/com/lambda/mixin/render/HandledScreenMixin.java Outdated
Comment threadsrc/main/kotlin/com/lambda/module/modules/player/EasyTrash.kt Outdated
@IceTankIceTank mentioned this pull request Jan 27, 2026
Closed
@IceTankIceTank changed the title feature easy-trash moduleFeature InventoryCleaner moduleFeb 21, 2026
beanbag44
beanbag44 requested changes Feb 23, 2026
View reviewed changes

@beanbag44beanbag44 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

when clicking an empty slot, it still throws a stack to make space

Comment threadsrc/main/kotlin/com/lambda/module/modules/player/InventoryCleaner.kt Outdated
Comment threadsrc/main/kotlin/com/lambda/module/modules/player/InventoryCleaner.kt Outdated
@IceTank
IceTank requested a review from beanbag44March 7, 2026 15:30
@github-project-automationgithub-project-automationBot moved this to Backlog in KanbanMar 11, 2026
beanbag44
beanbag44 requested changes Mar 12, 2026
View reviewed changes
Comment threadsrc/main/kotlin/com/lambda/module/modules/player/InventoryCleaner.kt Outdated
Comment threadsrc/main/kotlin/com/lambda/module/modules/player/InventoryCleaner.kt
@IceTank
Optimize some code and made the ultra-wide monitor investment worth i…
8921b84
…t by putting multi line description strings for settings into one line
beanbag44
beanbag44 requested changes Mar 17, 2026
View reviewed changes
Comment threadsrc/main/kotlin/com/lambda/module/modules/player/InventoryCleaner.kt Outdated
emyfops
emyfops reviewed Mar 17, 2026
View reviewed changes
Comment threadsrc/main/kotlin/com/lambda/event/events/InventoryEvent.kt Outdated
/**
* Represents an action performed on an inventory slot, such as clicking or interacting with it.
*/
abstract class SlotAction : Event, ICancellable {

@emyfopsemyfopsMar 17, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Should be open, also what is up with all this ktdoc

@IceTankIceTankMar 18, 2026

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Just copied what the constructor did in the rest of the file

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

Reviewers

@beanbag44beanbag44beanbag44 requested changes
+1 more reviewer
@emyfopsemyfopsemyfops requested changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

Kanban
Status: Backlog

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@IceTank@emyfops@beanbag44@Avanatiker