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

Update controller inventory to be compatible with mcl2 #89

Merged
merged 1 commit into from
Feb 21, 2024

Conversation

Aracaa
Copy link
Contributor

@Aracaa Aracaa commented Feb 19, 2024

Use the same inventory layout in controller.lua than in api.lua for drawer inventory.

The inventory in mcl2 is 9 slots per row and the hotbar is displayed at the bottom.
The default game inventory is 8 slots per row and the hotbar is displayed at the top.

For consistency and make the mcl2 inventory full visible, I made larger the controller formspec size.

Screenshot of how it's now

default game

image
image

Mineclone2

image
image

Copy link
Member

@Niklp09 Niklp09 left a comment

Choose a reason for hiding this comment

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

LGTM. Will merge in a few days unless there are objections.

@Niklp09 Niklp09 merged commit 70c3090 into minetest-mods:master Feb 21, 2024
1 check passed
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