FirstPersonModel icon indicating copy to clipboard operation
FirstPersonModel copied to clipboard

Incompatibility/Suggestion in relation with Freecam Mod

Open AzrannAVANA opened this issue 1 year ago • 5 comments

Hello! I have a problem where when using Freecam, the a default player is blocking any view.. image

which is how Freecam works, but I think something could be done for this.

If Freecam is activated, never render the player model.. This sounds easy to me, but I'm not sure how modding goes.

This is but a simple suggestion, hope it finds you well. Good day to everyone-

AzrannAVANA avatar Apr 29 '24 00:04 AzrannAVANA

Oh, I see that it was fixed before, but it's back now in 1.20.1, mod version 2.3.4.

AzrannAVANA avatar Apr 29 '24 00:04 AzrannAVANA

Original issue #291, fixed in this commit.

Current Freecam issue https://github.com/MinecraftFreecam/Freecam/issues/185, for reference.

Not sure yet if the issue is with FPM or Freecam, won't be able to investigate for at least a few days either.

MattSturgeon avatar Apr 29 '24 00:04 MattSturgeon

Issue is present on all 1.20.1 versions of FPM and Freecam, My guess is that the fix was lost in 1.20+ maybe...

I've tested from FPM 2.2.3 to 2.3.4, and none work correctly alongside Freecam I'm afraid

AzrannAVANA avatar Apr 29 '24 01:04 AzrannAVANA

I figured it out! The commit was only for the fabric version of the mod, as the file address stated, and it was never present in the forge port.

Thank you @MattSturgeon for pointing out the commit, I wouldn't have found that otherwise.

So, it's fixed for fabric, but not forge.

AzrannAVANA avatar Apr 29 '24 01:04 AzrannAVANA

I noticed FPM has a compile-only dependency on Freecam's fabric build. This is used to check Freecam.isEnabled().

That dependency should also work fine for forge/neoforge, so long as you don't reference any non-common classes, such as FreecamFabric or ModMenuIntegration, and you don't need a runtime dependency. You could use the forge version if you prefer though.

@tr7zw Hopefully enabling your existing fix in Forge builds will be fairly simple and resolves the issue. I don't currently have time to do a PR though.

MattSturgeon avatar Apr 29 '24 03:04 MattSturgeon

This still hasn't been fixed sadly, and newer version of this mod entirely bans Sinytra, telling me to switch to forge- Cannot however because it still has the compatibility issue..

AzrannAVANA avatar May 31 '24 23:05 AzrannAVANA