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

homedecor stairs compatibility #90

Open
bell07 opened this issue May 5, 2022 · 4 comments
Open

homedecor stairs compatibility #90

bell07 opened this issue May 5, 2022 · 4 comments
Labels
bug Something isn't working Wait for upstream fix Mod needs a fix in upsteam

Comments

@bell07
Copy link
Collaborator

bell07 commented May 5, 2022

In #49 we get workarounded the stairs support removal.

Now the support is on the way back in mt-mods/homedecor_modpack@2e3a826 thanks to @wsor4035

We should test the change and support it to reach upstream. Then we can remove the workaround frim whynot_compat at
https://github.com/minetest-whynot/whynot-game/blob/main/mods/libs/whynot_compat/init.lua#L132

@Lazerbeak12345
Copy link
Collaborator

A cursory review of the code without running it looks like it will work.

It's composed of an array of what looks like hitbox data, along with that loop that more or less matches the loop they gave us earlier.

@Lazerbeak12345
Copy link
Collaborator

I've play-tested it, and as-is, I think it'd be fine.

There's only one problem with it, which I think should only be fixed if it's easy, and that's that some items like stairs:slab_fake_grass have incorrect side-textures.

My reasoning for saying this is probably fine is that we are relying on a legacy codepath, and usually I figure that the developer would want it to be functional but just barely. This is just past the "just barely" mark, in my mind.

What are your thoughts @bell07 ?

@bell07
Copy link
Collaborator Author

bell07 commented May 9, 2022

The change is in homedecor's stair_complaining branch and not merged into master yet.

Unsure if the change will be merged into master. Currently the change is not proposed.

@wsor4035 what is the planned feature for the change?

Since whynot is not a new home for mods, the best case is the change is merged in upstream. Otherwise we need an maintained own fork that we try to avoid.

@Lazerbeak12345
Copy link
Collaborator

To clarify (mostly to myself in the future) we have not switched to the proposed "stairs_complaining" branch.

@Lazerbeak12345 Lazerbeak12345 added bug Something isn't working Wait for upstream fix Mod needs a fix in upsteam labels Aug 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Wait for upstream fix Mod needs a fix in upsteam
Projects
None yet
Development

No branches or pull requests

2 participants