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

Incompatibility between papertweaks with regions UI #236

Open
bradgillap opened this issue Jan 27, 2024 · 4 comments
Open

Incompatibility between papertweaks with regions UI #236

bradgillap opened this issue Jan 27, 2024 · 4 comments

Comments

@bradgillap
Copy link

Hello,

I've identified that papertweaks causes some strange UI compatibility issues with regions on vane. I think regions still works but the menus display really weird.

image

I haven't identified if it's a single papertweaks module or if it's just the entire plugin at this time. I'll just remove paper tweaks as a solution for myself but I didn't see this documented anywhere so far in my google searches.

Anyway, hope this helps someone. Vane is really cool, thanks for your hard work.

@oddlama
Copy link
Owner

oddlama commented Jan 27, 2024

This looks like paper tweaks modifies the lore of that item and in doing so they strip the special "translated text components" from the lore, so your client doesn't know that it has to look up a translation anymore.

If the problem really goes away when removing papertweaks, then it's likely that they have to make a fix for this.

@Machine-Maker
Copy link

To add some clarity here, I'm guessing this is just a result of conflicting resource packs? I haven't yet adopted the new pack behavior where clients can have multiple applied from the server. So if you have a module in PaperTweaks enabled which sends a resource pack, it replaces the pack from vane and so you are missing translations.

I will update my plugin to use the new methods to not remove all packs first, I think vane should do the same (tho I think PaperTweaks should load after vane, probably good to update it anyways).

@oddlama
Copy link
Owner

oddlama commented Jan 28, 2024

So if you have a module in PaperTweaks enabled which sends a resource pack, it replaces the pack from vane and so you are missing translations.

That makes sense, thanks for figuring it out!

where clients can have multiple applied from the server

Oh my, this exists now? Finally! Do you happen to know when this was added? Anyway, I'll definitely have to switch to that, been waiting for this forever. Vane can probably be quite low-priority in the chain of packs.

@Machine-Maker
Copy link

Oh my, this exists now? Finally! Do you happen to know when this was added?

Yeah, new functionality was added in the 1.20.3/4 update for this. There are new methods in the API sendResourcePacks on adventure's Audience interface for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants