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

Shift-clicking in the crafting terminal can put items in off-hand slot #2611

Closed
GuntherDW opened this Issue Nov 13, 2016 · 2 comments

Comments

Projects
None yet
3 participants
@GuntherDW
Contributor

GuntherDW commented Nov 13, 2016

Description

When you shift-click a crafting recipe when your inventory is full it will put the resulting item in your offhand slot (if that slot is empty).
The Crafting or terminal interface doesn't give you access to that slot which makes it so you have to exit out and remove it manually if you want to do something with it.

I don't know whether or not this is an intended change but it's kind of something that can take you by surprise if you didn't know about it, and can be irritating because the UI doesn't have a slot for that particular inventory slot.

  • Minecraft Version: 1.10.2
  • AE2 Version: fc83403
  • Forge Version: 12.18.2.2122
@yueh

This comment has been minimized.

Show comment
Hide comment
@yueh

yueh Nov 13, 2016

Member

This stupid inventory handling of minecraft (read there is none)...

Member

yueh commented Nov 13, 2016

This stupid inventory handling of minecraft (read there is none)...

@yueh yueh added the type-bug label Nov 13, 2016

@yueh yueh added this to the rv4 beta - 1.10 milestone Nov 13, 2016

@fscan

This comment has been minimized.

Show comment
Hide comment
@fscan

fscan Oct 8, 2017

Member

It would be easy to fix this, but i think the proper way to solve this would be to show the armor/offhand slots in the GUI.

Member

fscan commented Oct 8, 2017

It would be easy to fix this, but i think the proper way to solve this would be to show the armor/offhand slots in the GUI.

@yueh yueh modified the milestones: rv5.alpha - 1.12, rv5.beta- 1.12 Oct 24, 2017

yueh added a commit that referenced this issue Oct 31, 2017

Fixes #2611: Limit the adaptor to the main player inv.
This prevents the result from various inventory actions to be placed into
the offhand or armor slots.

@yueh yueh added the state-fixed label Oct 31, 2017

@yueh yueh closed this in 725fa49 Oct 31, 2017

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