Closed Estecka closed 3 months ago
thank you for looking into this. i was aware but it being a non curcial tiny issue never looked much into it since people could just use the other configs to tweak the attachment accordingly. Ill see if that code fixes
Just realized you use Mojmap. The snippet I gave uses Yarn mappings.
Confirmed that fix is working. This is its equivalent in mojmap.
double guiScale = Minecraft.getInstance().getWindow().getGuiScale();
so weirdly this is only needed on fabric version. even tho they use the same exact code and im not sure why
this was actual fix. they were using getGuiWidth() instead
Probably related to #105
The minimap anchor points only work properly when the vanilla GUI scale is set to 1. At any other GUI scale, the effective anchor point ends up being located farther outside of the screen, varying greatly depending on the GUI scale and the screen resolution.
I believe the faulty code is here. The correct version would look something like this:
I would have made a PR for this, but I was unable to test this code due to being unable to build the project.
Repro:
Versions: