Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Mod seemingly fails to reimplement official hardcoded behavior in its own system #19

Open
LPFan12 opened this issue Sep 22, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@LPFan12
Copy link

LPFan12 commented Sep 22, 2024

Mod Loader

Fabric

Minecraft Version

1.21.1

What Happened?

The splash that is under the translationkey ID "222" in the textfile gets used by default, when it isn't supposed to be, and there does not seem to be a relevant config toggle to disable it.

In the vanillagame, since translationkeys for splashes don't exist there, there is instead a hardcoded function that stops a splash that matches "This message will never appear on the splash screen, isn't that weird?" from being printed onscreen, being skipped in the random selection. It makes sense if this was simply missed when making the mod, since it looks like any other splash besides its self-explaining message.

I noticed the inconsistency when I started work on a resource pack to translate/localize the splashes into german.

What Should Happen?

Any splash under the translationkey "splash.minecraft.222" does not render onscreen unless the user actually wants it(requires new config toggle to seperately handle the intentionally hidden Splash text with default value false).

Steps to Reproduce

1: Use the mod
2: The relevant Splash is in the pool of messages to show on the title screen
3: Discover the splash actually exists to begin with(I honestly don't think I knew beforehand)

Crash Log

No response

Anything Else?

This is an amazingly cool mod. I wish a feature like this was available for every version of MC by default, but alas...

@LPFan12 LPFan12 added the bug Something isn't working label Sep 22, 2024
@KrLite
Copy link
Member

KrLite commented Sep 22, 2024

That's such an interesting feature! I didn't notice that when I read the vanilla code. It's definitely ok to add a feature like that, but since I'm not currently working on Java for a period of time, I'm sorry that updates may delay.

Thanks for proposing and I'll work on it once I come back to write Java again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants