minetest-mods / inspector

A minetest node inspector tool that shows various properties of nodes.
GNU Lesser General Public License v2.1
9 stars 8 forks source link

Include inherited fields in nodedef listing #5

Closed Treer closed 5 years ago

Treer commented 5 years ago

Includes the values for nodedef's inherited fields, so the inspector will always show things like is_ground_content, and displays them all in a sorted list.

I'm not sure if you want that though, or would rather the two lists be displayed separately, or perhaps the inherited default values should be marked clearly so you know the node hasn't changed that value. So don't feel you ought to pull this - it's useful to me as I don't know all the properties of a nodedef but it might hinder others.

I tried to colourize the text so default values would be grey but still displayed, but I think there are formspec limitations preventing that.

You probably shouldn't pull in the Colorize commit, I failed to make a separate feature fork so it was automatically appended to this PR. I'll replace this PR with a feature branch when I get a moment.