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

Direct row and column Parameters for ItemGroup in Fabric API #4281

Open
FrQuentin opened this issue Dec 7, 2024 · 1 comment
Open

Direct row and column Parameters for ItemGroup in Fabric API #4281

FrQuentin opened this issue Dec 7, 2024 · 1 comment

Comments

@FrQuentin
Copy link

Currently, the ItemGroup in Fabric API does not provide row and column parameters, which limits our ability to position custom item groups effectively on the inventory bar. This request is for the direct addition of these parameters to the ItemGroup constructor or methods, allowing modders to easily specify their desired positions on the inventory screen.

Hi request the addition of row and column parameters directly to the ItemGroup class or constructor in Fabric API. This feature would provide a straightforward way to place custom item groups on the inventory bar, enhancing modders' ability to organize inventory content more effectively without relying on workarounds such as mixin usage.

As you can see, it's private.
private

When you try the logical
privateÉ
private3

Benefits:

  • Simplified Item Group Placement: By directly setting row and column values, developers can more easily manage the layout of item groups on the inventory screen.

  • Consistency with Vanilla Behavior: Aligning with vanilla item group positioning would improve the user experience, making it easier for players to navigate custom content in mods.

  • Enhanced Modding Flexibility: This change would reduce the need for complex, non-standard solutions and make modding more intuitive.

Thank you for considering this enhancement request.

@modmuss50
Copy link
Member

Hi, I'm not sure this is possible as we currently dyanmicaly assign the page, row and column here: https://github.com/FabricMC/fabric/blob/1.21.4/fabric-item-group-api-v1/src/main/java/net/fabricmc/fabric/mixin/itemgroup/ItemGroupsMixin.java#L101-L102

Im not sure how this could ever work alongside other mods.

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

2 participants