brnbrd / Delightful

Farmer's Delight addon with immersive new content.
https://www.curseforge.com/minecraft/mc-mods/delightful
MIT License
6 stars 14 forks source link

Placeable pies still show food tooltips for Diet mod #86

Open danaYatsuta opened 2 months ago

danaYatsuta commented 2 months ago

Describe the bug: This is minor, but still thought it was worth reporting: when pumpkin pie overhaul is enabled, pumpkin pie tooltip still has nutrition values, which is confusing because it is not an edible item anymore. Made worse if playing with AppleSkin and/or Diet: tooltips will show that sum of nutrients from 4 pumpkin slices is more than nutrients of pumpkin pie, which makes no sense.

To Reproduce: N/A

Expected behavior: No nutrition tooltip on pumpkin pie

Screenshots: N/A

Versions: Minecraft 1.20.1, Delightful 3.5.6, Farmer's Delight 1.2.4, NeoForge 47.1.106.

danaYatsuta commented 2 months ago

Also applies to Source Berry Pie, but I don't know how much control over its nutritional values Delightful has.

brnbrd commented 1 month ago

AppleSkin support added in v3.5.7 (and applies to compat pies such as Source Berry). Food tooltips added by Quark should be disabled in their config in favor of using AppleSkin. I will look into supporting Diet, but it is not a priority.