search

minetest-mods / craftguide

:book: The most comprehensive Crafting Guide on Minetest
Other
43 stars 20 forks source link

[WIP] Move fuel time, burn time tooltip to recipe type icon #70

Closed Wuzzy2 closed 5 years ago

Wuzzy2 commented 5 years ago

Currently, fuel and burn time is added to the item tooltip. This is a bit suboptimal IMO. The fuel and cooktime is not an inherent part of the item, however, it is part of the recipe.

This PR wants to move the burn/cooktime away from the item icon to the appropriate recipe type icon (furnace/flame).

This PR is incomplete. For the flame icon, only the “small flame” is supported. The big flame only shows the word “Fuel” but not the burntime (I have not figured out how to get the burntime here).

Wuzzy2 commented 5 years ago

Oh, the word “Fuel” can be translated with “Brennstoff” in German.

kilbith commented 5 years ago

So I tested your PR and this making things less informative. Why does the output item should tell about its {burn,cook}ing time but not the input items? This is a bit arbitrary in my opinion.