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

[Suggestion] Right click in inventory to Hot Swap. #7

Open
TheZennou opened this issue Aug 9, 2021 · 5 comments
Open

[Suggestion] Right click in inventory to Hot Swap. #7

TheZennou opened this issue Aug 9, 2021 · 5 comments

Comments

@TheZennou
Copy link

TheZennou commented Aug 9, 2021

I think it'd be cool to add this either to default or as a config, so if you right clicked a piece of armor from your inventory it would swap the your currently equipped armor with the one in the slot you right clicked.
If this conflicts with the vanilla controls, maybe a shift click, or something else?

@MAGGen-hub
Copy link

I can try ot fix it and make pull request... See this mod: https://github.com/StephenBarnes/ArmorExchanger
It has more bugs than this mod but some code in this modification can be useful, I will try to change event bus subscriber to make tis mod work more corect: final PlayerInteractEvent.RightClickItem event this might be a solution...

@MAGGen-hub
Copy link

I will post a fix to my git page soon...

@MAGGen-hub
Copy link

Well... Maybe not as soon as I wanted...

@MAGGen-hub
Copy link

Well... I guess that I need to make my own mod... Because I have changed a lot... REALLY a lot...

@kregerl
Copy link
Owner

kregerl commented Apr 5, 2022

Great suggestion, I've implemented it in the latest commit to the 1.16 version. 8e4bec9
Should be added to the other versions and released on curseforge within a day or two. It swaps armor when someone holds CTRL and right clicks the item.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants