-
Notifications
You must be signed in to change notification settings - Fork 11
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
Possible to Have Even More Lights From Default? #3
Comments
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:
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. |
That's okay, thanks you for letting me aware of this and this is just perfectly typical for you. :)
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. |
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. |
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. |
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!
The text was updated successfully, but these errors were encountered: