R3944Realms/ModernFix
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Only preserve window position if a non-default width/height was given

Browse Source 
Fixes #103 (in next release)
This commit is contained in:
embeddedt 2023-05-20 16:35:41 -04:00
parent 962b843199
commit 0d901c6281
No known key found for this signature in database
GPG Key ID: A69433EC199B5613
1 changed files with 13 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
forge/src/main/java/org/embeddedt/modernfix/forge/mixin/bugfix/preserve_early_window_pos/WindowMixin.java
View File

@ -1,8 +1,7 @@
package org.embeddedt.modernfix.forge.mixin.bugfix.preserve_early_window_pos;
import com.mojang.blaze3d.platform.Monitor;
import com.mojang.blaze3d.platform.ScreenManager;
import com.mojang.blaze3d.platform.Window;
import com.mojang.blaze3d.platform.*;
import net.minecraft.client.Minecraft;
import net.minecraftforge.fml.common.ObfuscationReflectionHelper;
import net.minecraftforge.fml.loading.progress.EarlyProgressVisualization;
import org.embeddedt.modernfix.annotation.ClientOnlyMixin;
@ -35,19 +34,26 @@ public class WindowMixin {
}
private Object getEarlyProgressVisualizer() {
Minecraft client = Minecraft.getInstance();
if(VISUALIZER == null || this.fullscreen)
return null;
Object o = ObfuscationReflectionHelper.getPrivateValue(EarlyProgressVisualization.class, EarlyProgressVisualization.INSTANCE, "visualization");
return VISUALIZER.isAssignableFrom(o.getClass()) ? o : null;
}
private static boolean defaultDisplayData(DisplayData arg3) {
return arg3.width == 854 && arg3.height == 480 && !arg3.isFullscreen;
}
/**
* Return a null monitor if not in fullscreen and the visualizer is present, so that the code grabs the
* original X/Y position of the window.
*/
@Redirect(method = "<init>", at = @At(value = "INVOKE", target = "Lcom/mojang/blaze3d/platform/ScreenManager;getMonitor(J)Lcom/mojang/blaze3d/platform/Monitor;"))
private Monitor getMonitor(ScreenManager manager, long id) {
return getEarlyProgressVisualizer() != null ? null : manager.getMonitor(id);
private Monitor getMonitor(ScreenManager manager, long id, WindowEventHandler arg, ScreenManager arg2, DisplayData arg3) {
if(defaultDisplayData(arg3) && getEarlyProgressVisualizer() != null)
return null;
return manager.getMonitor(id);
}
/**
@ -55,9 +61,9 @@ public class WindowMixin {
*/
@SuppressWarnings("unchecked")
@Redirect(method = "<init>", at = @At(value = "INVOKE", target = "Lnet/minecraftforge/fml/loading/progress/EarlyProgressVisualization;handOffWindow(Ljava/util/function/IntSupplier;Ljava/util/function/IntSupplier;Ljava/util/function/Supplier;Ljava/util/function/LongSupplier;)J"), require = 0)
private long performHandoff(EarlyProgressVisualization instance, IntSupplier width, IntSupplier height, Supplier<String> title, LongSupplier monitor) {
private long performHandoff(EarlyProgressVisualization instance, IntSupplier width, IntSupplier height, Supplier<String> title, LongSupplier monitor, WindowEventHandler arg, ScreenManager arg2, DisplayData arg3) {
Object visualizer = getEarlyProgressVisualizer();
if(visualizer != null) {
if(visualizer != null && defaultDisplayData(arg3)) {
long windowId = ObfuscationReflectionHelper.<Long, Object>getPrivateValue((Class<? super Object>)visualizer.getClass(), visualizer, "window");
int[] w = new int[1];
int[] h = new int[1];