mesecons
mesecons copied to clipboard
minetest-mods
Changes to torch and delayer appearance
Changed torch to be plantlike, x-shaped instead of just a sprite and only use 1 texture instead of 3 and changed delayer to require only 3 textures from texturepacks instead of 13 changing the model to be a simple slab. Yes, I'm sad for whoever put in the work into the delayer model but it clashes with texturepacks and requires way to much work. Especially if they are not 16x16.
Would you approve of this? Now it only requires 1 additional texture for texture pack creators to support the delayer after supporting the rest of the mod.
Before anyone asks: License of my arrow of slowness and code changes is CC0
License of my arrow of slowness and code changes is CC0
Not sure how the license situation here is, acording to the readme all media files are cc-by-sa 3.0.
I said CC0 meaning I don't give a shit what you do with it or how you relicense it. I would like to know what you think of the new delayer and what still needs to be done until it can be merged.
I would like to know what you think of the new delayer and what still needs to be done
I like the "old" delayer design as it looks (more) like an arrow, this is pretty subjective.
why not just merge?
Because no one approved your changes, I recommend to move the uncontroversial stuff to a second PR.
If no one cares anyway
Probably due to your previous behavior...
This PR has been up for a month now. I asked if anybody would like to volunteer reviewing this on IRC two days ago but nobody even replied. https://irc.minetest.net/minetest/2024-05-10
I really think the people who are active don't care and those who aren't active I can't ask either way. I don't see how it's a controversial change. It doesn't break any other texture packs or mods to my knowledge except for the top of delayer in existing texture packs. However the old delayer is an ugly piece of shit, that requires 13 texture that are variations and the elevated voxels of the circle assume a 16x16 texture pack and look shit with all other resolutions like 32x32, 64x64, 128x128. Here is a picture from mesecons.net. I think it's the hdx-128 texture pack. It's objectively ugly. The other mesecons textures in the pack look better. I think the creator of the pack gave up on it and this monstrosity was born.
Of course I'm biased because I'm maintaining a 128x texture pack too. https://content.minetest.net/packages/shaft/hand_painted_expanded/
I can change the symbol to a simple arrow if that pleases you.
Thank you so much for your review.
Things where I don't agree:
- I will add the dependency of gates on delayer back and put them in delayer. That seems like a good solution and you're probably right about that.
- They were already there and I see no reason to move them. The jeija_luacontroller_LED_*.png in luacontroller were removed, to reduce the amount of textures and then the jeija_microcontroller_LED_*.png were renamed to jeija_luacontroller_LED_*.png because digistuff depends on the luacontroller leds and I don't want to break it.
- I can fix that too.
All done and I've changed the symbol to the bold arrow which you clearly both prefer.
It doesn't actually indicate whether the output really active because of the delay though which is why I originally didn't do it. Just points to where the output is. (The power out of the old delayer doesn't either, so it's probably fine.)