search

MrCrayfish / GoblinTraders

Adds goblins that trade speical items with the player!
MIT License
56 stars 39 forks source link

Interface Bug (probably compatability) #36

Closed Foozey closed 3 years ago

Foozey commented 3 years ago

Issue is shown here. This is most likely a compatibility issue with AppleSkin, which is the mod that adds the food info tooltips. Either way, definitely worth investigating due to how commonly the AppleSkin mod is used.

MrCrayfish commented 3 years ago

I would fix if I could but I am using the villager trading GUI, not a custom GUI. Problems lies more with AppleSkin.

MrCrayfish commented 3 years ago

Can you test this with a villager with custom trades? It's possible the issue hasn't been seen before because Villagers by default don't accept food as a buy item. If this is an issue, please create an issue on AppleSkin and reference this thread. /summon villager ~ ~1 ~ {VillagerData:{profession:farmer,level:2,type:plains},Offers:{Recipes:[{buy:{id:apple,Count:1},sell:{id:apple,Count:1},rewardExp:0b,maxUses:9999999}]}}

MrCrayfish commented 3 years ago

Okay, so I tested myself. No issues... image

Foozey commented 3 years ago

Hrm, did you have OptiFine installed as well during your test?

MrCrayfish commented 3 years ago

There's your problem

MrCrayfish commented 3 years ago

Regardless of the issue, nothing I can do because I simply use the Villager trade GUI. Follow up with the AppleSkin dev.

Another commands for Villager with more trades if you need to test if the issue happens with villagers too. /summon villager ~ ~1 ~ {VillagerData:{profession:farmer,level:2,type:plains},Offers:{Recipes:[{buy:{id:apple,Count:1},sell:{id:apple,Count:1},rewardExp:0b,maxUses:9999999},{buy:{id:bread,Count:1},sell:{id:bread,Count:1},rewardExp:0b,maxUses:9999999}]}}

Foozey commented 3 years ago

Ah yep, happening with villagers as well. I'll notify the AppleSkin dev.