Translation for advancement criteria names

[Strubbelkopp]

It would be nice to have the ability to change the internal names of advancement criteria thorugh a config or a language file as suggested in this issue from AdvancementInfo

[jhmaster2000]

+1, I've just closed my linked issue on AdvancementInfo after finding out the author of that mod is not interested in adding new features so it'd be nice to have this mod replace it.

[Leronus]

+1

[WondaMegapon]

+1

I'm a complete novice to Fabric, but it seems like just getting lines 99, 107, and 117 in CriterionGrid.java to use translatable tokens would be more than enough to achieve this effect. The only issue would be getting those tokens in the first place. 😮

[Darkosto]

Hey @way2muchnoise ! Is there any chance that you'd be willing to look into this? I'd love to have more accessibility for players by allowing translatable criteria. Thank you for the mod and keep it updated! <3

[way2muchnoise]

Thanks to @GoryMoon you can now add translations keys as follows: betteradvancements.criterion.<advancement id>.<criterion name> Full details in the PR: https://github.com/way2muchnoise/BetterAdvancements/pull/165#issue-2352170289

[Darkosto]

Sorry to be a bother @way2muchnoise but is there any way you'd be able to push that update to 1.20.1? I'm unable to move my project up to 1.20.4 without starting over on most of it. If not, no worries and thanks for your help!

[way2muchnoise]

Will backport it 😃 just give me some time. Hope to get to it in the weekend

[Darkosto]

Thank you so much! You're the best! ❤️

[way2muchnoise]

@Darkosto Thanks to @GoryMoon, we have full back port to 1.20.1 (#167)