I tested. It's doesn't solve the mentioned issue because here it's not the compute of the remaining use but the total uses, which is displayed right.
Based on our discussion MineClone2/MineClone2#4196 (comment) I removed the notion of "Mining Durability", now it's only "Durability". So it's not used anymore
I don't think it's a good solution to use a different method for the first case. It's like hiding a bug.
and otherwise using math.ceil with (use - 1) to correctly show 59/60 after the first…
I think your use of math.round() instead of math.ceil() allows tooltips that read "Durability: 0/60" to happen. Changing this in 3 spots should fix this.
This is true. But please note that by…
why is this PR closed please ? It was so close to be finalized
I mean, durability does affect both, only speed doesn't, so revert speed but not durability.
Done, I remove "Mining" for the durability
![image](/attachments/0977890e-d587-4cd8-aef5-980c0c2…
@the-real-herowl I knew it could lead to debate, this is why I did it in a separated commit. So if I understand well, you suggest to revert the last commit ?
In the last commit, I removed the notion of "Mining" for Durability & Speed in tooltip and factoid. It's to avoid having "mining speed" for sword for example
NOW : stack overflow
stack traceback:
[C]: in function 'get_name'
.../games/minec…