random-geek / morelights

Minetest mod adding additional lighting nodes.
GNU Lesser General Public License v3.0
11 stars 11 forks source link

Possible to Have Even More Lights From Default? #3

Open FriendlyGamer opened 3 years ago

FriendlyGamer commented 3 years ago

Hiya there! I hope this finds you well!

But I am pondering, is it possible for us to have other full block lighting options, like for other bricks, planks and such from the game's Default "mod"?

So that buildings can be more ingrained and such?

Do let me know if this is possible and thanks in advance!

random-geek commented 3 years ago

Hi! Sorry for the belated response (as is typical for me :P).

Adding more lights based on existing nodes is certainly possible, but I'm reluctant to do it for a few reasons:

  1. Inventory clutter/loading times. One of the main goals of Morelights is to be fairly lightweight and fast. The more nodes I add, the longer it will take for clients to load the mod. I also don't want to gum up the crafting guide/creative menu with a lot of similar lighting nodes.
  2. Support for a variety of games. The Minetest developers want users to explore games beyond Minetest Game, and I have tried to assist that effort by supporting MineClone 2 and making it easier to add more games in the future. Grass, stone block and sandstone block lights all use the current game's native nodes/textures. Other games may not have these nodes, though, which would make crafting recipes and textures for these nodes more challenging.
  3. The "slippery slope." Once I add a wood plank light, then the mod would really need lights for all the types of wood planks, etc.

There are a couple other options. One is to add a "light placer" tool, allowing the player to add lights to any supported node; again, this would require registering more nodes and could create confusion. Another option is to add a similar lighting node that would be fairly flush with the node and could be placed anywhere. Unfortunately, this would make it impossible to place another node on the same surface.

Perhaps I could add a couple more nodes, but I'm very hesitant at the moment.

FriendlyGamer commented 3 years ago

Hi! Sorry for the belated response (as is typical for me :P).

That's okay, thanks you for letting me aware of this and this is just perfectly typical for you. :)

Perhaps I could add a couple more nodes, but I'm very hesitant at the moment.

I understand, thank you for caring to explain to me on why you are very hesitant to do it, at least at this point in time.

doxygen-spammer commented 3 years ago

The LinuxForks fork of this mod (https://github.com/h-v-smacker/morelights) has additional block lights similar to the existing grass light and stone tile light.

To keep this mod lean, would you accept if I add a function that returns a node definition for arbitrary block lights? Then anyone can add their own block lights easily.

random-geek commented 3 years ago

To keep this mod lean, would you accept if I add a function that returns a node definition for arbitrary block lights? Then anyone can add their own block lights easily.

I don't see a reason not to, but it might not save that many lines of code. I'd at least consider such a function.