Fixes the inventory items not moving properly.

Still needs a bit of tweaking

.gitignore

@@ -32,3 +32,5 @@ bin/
 
 run/
 /logs/
+local.properties
+remappedSrc/

build.gradle

@@ -1,10 +1,10 @@
 plugins {
-	id 'fabric-loom' version '0.5-SNAPSHOT'
+	id 'fabric-loom' version '0.8-SNAPSHOT'
 	id 'maven-publish'
 }
 
-sourceCompatibility = JavaVersion.VERSION_1_8
-targetCompatibility = JavaVersion.VERSION_1_8
+sourceCompatibility = JavaVersion.VERSION_16
+targetCompatibility = JavaVersion.VERSION_16
 
 archivesBaseName = project.archives_base_name
 version = project.mod_version + "-" + project.minecraft_version + "-fabric"
@@ -43,13 +43,8 @@ tasks.withType(JavaCompile).configureEach {
 	// If Javadoc is generated, this must be specified in that task too.
 	it.options.encoding = "UTF-8"
 
-	// The Minecraft launcher currently installs Java 8 for users, so your mod probably wants to target Java 8 too
-	// JDK 9 introduced a new way of specifying this that will make sure no newer classes or methods are used.
-	// We'll use that if it's available, but otherwise we'll use the older option.
-	def targetVersion = 8
-	if (JavaVersion.current().isJava9Compatible()) {
-		 it.options.release = targetVersion
-	}
+	// Minecraft 1.17 (21w19a) upwards uses Java 16.
+	it.options.release = 16
 }
 
 java {

gradle.properties

@@ -3,14 +3,14 @@ org.gradle.jvmargs=-Xmx1G
 
 # Fabric Properties
 	# check these on https://fabricmc.net/use
-	minecraft_version=1.16.4
-	yarn_mappings=1.16.4+build.9
-	loader_version=0.11.1
+	minecraft_version=1.17
+	yarn_mappings=1.17+build.1
+	loader_version=0.11.3
 
-cloth_version=4.11.14
-modmenu_version=1.16.8
+cloth_version=5.0.34
+modmenu_version=2.0.2
 
 # Mod Properties
-	mod_version = 1.2
+	mod_version = 1.1.1
 	maven_group = markil3
 	archives_base_name = immersive_hud

gradle/wrapper/gradle-wrapper.properties

@@ -1,5 +1,5 @@
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-6.8-bin.zip
+distributionUrl=https\://services.gradle.org/distributions/gradle-7.0.2-bin.zip
 zipStoreBase=GRADLE_USER_HOME
 zipStorePath=wrapper/dists

src/main/java/markil3/immersive_hud/ConfigManager.java

@@ -1,19 +1,11 @@
 package markil3.immersive_hud;
 
-import net.minecraft.item.Item;
-import net.minecraft.item.Items;
-import net.minecraft.util.Identifier;
 import net.minecraft.util.math.MathHelper;
-import net.minecraft.util.registry.Registry;
 
 import me.shedaniel.autoconfig.AutoConfig;
 
 import static markil3.immersive_hud.Main.TICKS_PER_SECOND;
 
-import java.util.Arrays;
-import java.util.Set;
-import java.util.stream.Collectors;
-
 /**
  * Configuration manager of this mod, which reads from and writes to this mod's
  * configuration file.
@@ -96,7 +88,6 @@ public class ConfigManager
     private boolean showArmor = true;
     private double minHealth = 0.5;
     private int minHunger = 17;
-    private Identifier[] ignoredItems = new Identifier[]{Registry.ITEM.getId(Items.FIREWORK_ROCKET)};
 
     /**
      * Returns the instance of this class.
@@ -256,26 +247,6 @@ public class ConfigManager
         return this.minHunger;
     }
 
-    /**
-     * Obtains a collection of items that won't cause hands and crosshairs to show when used.
-     *
-     * @return A set of ignored items.
-     */
-    public Set<Item> getIgnoredItems()
-    {
-        return Arrays.stream(ignoredItems).map(Registry.ITEM::get).collect(Collectors.toSet());
-    }
-
-    /**
-     * Obtains a collection of items that won't cause hands and crosshairs to show when used.
-     *
-     * @return An array of identifiers.
-     */
-    public Identifier[] getIgnoredIdentifiers()
-    {
-        return this.ignoredItems;
-    }
-
     /**
      * Sets the time that the hands are allowed to display.
      *
@@ -347,26 +318,6 @@ public class ConfigManager
     {
         this.minHunger = MathHelper.clamp(boundary, 0, 20);
     }
-
-    /**
-     * Sets which items won't cause the hands and crosshairs to show when used.
-     *
-     * @param items - A set of ignored items.
-     */
-    public void setIgnoredItems(Item... items)
-    {
-        this.ignoredItems = Arrays.stream(items).map(Registry.ITEM::getId).toArray(Identifier[]::new);
-    }
-
-    /**
-     * Sets which items won't cause the hands and crosshairs to show when used.
-     *
-     * @param items - A set of ignored items.
-     */
-    public void setIgnoredItems(String... items)
-    {
-        this.ignoredItems = Arrays.stream(items).map(Identifier::new).filter(i -> Registry.ITEM.getOrEmpty(i).isPresent()).toArray(Identifier[]::new);
-    }
 /**
      * Saves changes to this mod's configuration.
      */

src/main/java/markil3/immersive_hud/ModMenu.java

@@ -3,11 +3,7 @@ package markil3.immersive_hud;
 import com.terraformersmc.modmenu.api.ConfigScreenFactory;
 import com.terraformersmc.modmenu.api.ModMenuApi;
 
-import net.minecraft.item.Items;
 import net.minecraft.text.TranslatableText;
-import net.minecraft.util.Identifier;
-import net.minecraft.util.InvalidIdentifierException;
-import net.minecraft.util.registry.Registry;
 
 import me.shedaniel.autoconfig.AutoConfig;
 import me.shedaniel.clothconfig2.api.ConfigBuilder;
@@ -17,11 +13,6 @@ import me.shedaniel.clothconfig2.gui.entries.DoubleListEntry;
 
 import static markil3.immersive_hud.Main.TICKS_PER_SECOND;
 
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.Optional;
-import java.util.stream.Collectors;
-
 public class ModMenu implements ModMenuApi
 {
 
@@ -44,8 +35,7 @@ public class ModMenu implements ModMenuApi
                         .setTooltip(new TranslatableText(
                                 "tooltip.immersive_hud." + name +
                                         "MaxTime"))
-                            .setSaveConsumer(val -> value
-                                    .setMaxTime((int) (val * TICKS_PER_SECOND)))
+                        .setSaveConsumer(val -> value.setMaxTime((int) (val * TICKS_PER_SECOND)))
                         .build();
 
         DoubleListEntry fadeIn =
@@ -59,8 +49,7 @@ public class ModMenu implements ModMenuApi
                         .setTooltip(new TranslatableText(
                                 "tooltip.immersive_hud." + name +
                                         "FadeIn"))
-                            .setSaveConsumer(val -> value
-                                    .setFadeInTime((int) (val * TICKS_PER_SECOND)))
+                        .setSaveConsumer(val -> value.setFadeInTime((int) (val * TICKS_PER_SECOND)))
                         .build();
 
         DoubleListEntry fadeOut =
@@ -74,8 +63,7 @@ public class ModMenu implements ModMenuApi
                         .setTooltip(new TranslatableText(
                                 "tooltip.immersive_hud." + name +
                                         "FadeOut"))
-                            .setSaveConsumer(val -> value
-                                    .setFadeOutTime((int) (val * TICKS_PER_SECOND)))
+                        .setSaveConsumer(val -> value.setFadeOutTime((int) (val * TICKS_PER_SECOND)))
                         .build();
 
         cat.addEntry(maxTime);
@@ -97,11 +85,9 @@ public class ModMenu implements ModMenuApi
                     .setSavingRunnable(() -> {
                         try
                         {
-                                                               if (Class
-                                                                       .forName("me.shedaniel.autoconfig.AutoConfig") != null)
+                            if (Class.forName("me.shedaniel.autoconfig.AutoConfig") != null)
                             {
-                                                                   AutoConfig
-                                                                           .getConfigHolder(ConfigManagerCloth.class)
+                                AutoConfig.getConfigHolder(ConfigManagerCloth.class)
                                         .save();
                             }
                         }
@@ -142,8 +128,7 @@ public class ModMenu implements ModMenuApi
                     .setTooltip(new TranslatableText(
                             "tooltip.immersive_hud" +
                                     ".crosshairTime"))
-                                         .setSaveConsumer(val -> instance
-                                                 .setCrosshairTime((int) (val * TICKS_PER_SECOND)))
+                    .setSaveConsumer(val -> instance.setCrosshairTime((int) (val * TICKS_PER_SECOND)))
                     .build());
 
             general.addEntry(entryBuilder.startBooleanToggle(new TranslatableText(
@@ -161,8 +146,7 @@ public class ModMenu implements ModMenuApi
                     .setDefaultValue(30)
                     .setTooltip(new TranslatableText(
                             "tooltip.immersive_hud.handTime"))
-                                         .setSaveConsumer(val -> instance
-                                                 .setHandTime((int) (val * TICKS_PER_SECOND)))
+                    .setSaveConsumer(val -> instance.setHandTime((int) (val * TICKS_PER_SECOND)))
                     .build());
 
             general.addEntry(entryBuilder.startBooleanToggle(new TranslatableText(
@@ -201,31 +185,6 @@ public class ModMenu implements ModMenuApi
                     .setSaveConsumer(val -> instance.shouldShowArmor(val))
                     .build());
 
-            general.addEntry(entryBuilder
-                    .startStrList(new TranslatableText("option.immersive_hud.ignoreList"), Arrays
-                            .asList(instance
-                                    .getIgnoredIdentifiers()).stream().map(Identifier::toString)
-                            .collect(Collectors.toList()))
-                    .setDefaultValue(Arrays
-                            .asList(Registry.ITEM.getId(Items.FIREWORK_ROCKET).toString()))
-                    .setTooltip(new TranslatableText("tooltip.immersive_hud.ignoreList"))
-                    .setSaveConsumer(val -> instance.setIgnoredItems(val.toArray(new String[0]))).setErrorSupplier(val -> {
-                        try
-                        {
-                            Optional<String> item = val.stream().filter(i -> !Registry.ITEM.getOrEmpty(new Identifier(i)).isPresent()).findFirst();
-                            if (item.isPresent())
-                            {
-                                return Optional.of(new TranslatableText("error.immersive_hud.ignoreList.unknownItem"));
-                            }
-                        }
-                        catch (InvalidIdentifierException e)
-                        {
-                            return Optional.of(new TranslatableText("error.immersive_hud.ignoreList.illegalId"));
-                        }
-                        return Optional.empty();
-                    })
-                    .build());
-
             return builder.build();
         };
     }

src/main/java/markil3/immersive_hud/TimerUtils.java

@@ -49,13 +49,12 @@ import java.util.HashMap;
 import java.util.HashSet;
 import java.util.Iterator;
 import java.util.Optional;
-import java.util.Set;
 
 /**
  * Contains the logic for changing the HUD.
  *
  * @author Markil 3
- * @version 0.1-1.16.4-fabric
+ * @version 1.1.1-1.17-fabric
  */
 public class TimerUtils
 {
@@ -174,7 +173,7 @@ public class TimerUtils
      */
     public static void setAlpha(float alpha)
     {
-        RenderSystem.color4f(1.0F, 1.0F, 1.0F, alpha);
+        RenderSystem.setShaderColor(1.0F, 1.0F, 1.0F, alpha);
     }
 
     /**
@@ -251,8 +250,6 @@ public class TimerUtils
                 ConfigManager.getInstance().getExperienceTime();
         ConfigManager.TimeValues jump =
                 ConfigManager.getInstance().getJumpTime();
-        ConfigManager.TimeValues healthTimes = ConfigManager.getInstance().getHealthTime();
-        ConfigManager.TimeValues hungerTimes = ConfigManager.getInstance().getHungerTime();
 
         float hotbarAlpha = Main.getAlpha(hotbarTime,
                 hotbar.getMaxTime(),
@@ -266,21 +263,8 @@ public class TimerUtils
                 jump.getMaxTime(),
                 jump.getFadeInTime(),
                 jump.getFadeOutTime());
-        float statusAlpha = Main.getAlpha(healthTime,
-                healthTimes.getMaxTime(),
-                healthTimes.getFadeInTime(),
-                healthTimes.getFadeOutTime());
-        statusAlpha = Math.max(statusAlpha, Main.getAlpha(hungerTime,
-                hungerTimes.getMaxTime(),
-                hungerTimes.getFadeInTime(),
-                hungerTimes.getFadeOutTime()));
-        /*
-         * Draw it in its standard location until it is completely transparent, then move it
-         * instantly.
-         */
-        statusAlpha = (float) Math.ceil(statusAlpha);
         return 7 * (1 - Math.max(experienceAlpha,
-                jumpAlpha)) + 22 * (1 - hotbarAlpha) + 42 * (1 - statusAlpha);
+                jumpAlpha)) + 22 * (1 - hotbarAlpha);
     }
 
     /**
@@ -290,6 +274,7 @@ public class TimerUtils
      *
      * @param hand - The hand being triggered
      * @param clear - Whether the hand it being enabled or disabled.
+     *
      * @since 0.1-1.16.4-fabric
      */
     public static void onClick(Hand hand, boolean clear)
@@ -304,7 +289,6 @@ public class TimerUtils
         LivingEntity entity;
         Item item;
         Iterator<ItemStack> hands;
-        Set<Item> ignored;
         if (mc.getCameraEntity() instanceof LivingEntity)
         {
             entity = (LivingEntity) mc.getCameraEntity();
@@ -330,14 +314,6 @@ public class TimerUtils
                             .map(ItemStack::getItem)
                             .orElse(null);
         }
-        if (item != null)
-        {
-            ignored = ConfigManager.getInstance().getIgnoredItems();
-            if (ignored.contains(item))
-            {
-                return;
-            }
-        }
         crosshairTime = ConfigManager.getInstance().getCrosshairTime();
         if (hand == Hand.MAIN_HAND)
         {
@@ -373,7 +349,9 @@ public class TimerUtils
      *
      * @param hand - The hand being rendered.
      * @param matrixStack - The rendering stack.
+     *
      * @return If true, then don't render this hand at all.
+     *
      * @since 0.1-1.16.4-fabric
      */
     public static boolean onRenderHand(Hand hand,
@@ -439,6 +417,7 @@ public class TimerUtils
      * needed.
      *
      * @return If true, then cancel drawing the crosshair.
+     *
      * @since 0.1-1.16.4-fabric
      */
     public static boolean drawCrosshair(float ticks)
@@ -495,6 +474,7 @@ public class TimerUtils
      * as needed.
      *
      * @return If true, then cancel drawing the mount health bar.
+     *
      * @since 0.1-1.16.4-fabric
      */
     public static boolean drawMountHealth(MatrixStack stack, float ticks)
@@ -626,6 +606,7 @@ public class TimerUtils
      * Obtains how transparent
      *
      * @param effectinstance
+     *
      * @return
      */
     public static float getPotionAlpha(StatusEffectInstance effectinstance)
@@ -637,8 +618,7 @@ public class TimerUtils
         if (effectinstance.getDuration() <= BLINK_TIME)
         {
             effectAlpha =
-                    MathHelper.sin(7000F / (effectinstance
-                            .getDuration() + 16F * (float) Math.PI)) * 50F / (effectinstance
+                    MathHelper.sin(7000F / (effectinstance.getDuration() + 16F * (float) Math.PI)) * 50F / (effectinstance
                             .getDuration() + 100F) + 0.5F;
         }
         else if (effectinstance.getDuration() <= BLINK_TIME + potion.getFadeInTime())
@@ -663,6 +643,7 @@ public class TimerUtils
      * needed.
      *
      * @return If true, then cancel drawing the armor bar.
+     *
      * @since 0.1-1.16.4-fabric
      */
     public static boolean drawArmor()
@@ -690,6 +671,7 @@ public class TimerUtils
      * needed.
      *
      * @return If true, then cancel drawing the health.
+     *
      * @since 0.1-1.16.4-fabric
      */
     public static boolean drawHealth(float ticks)
@@ -765,6 +747,7 @@ public class TimerUtils
      * needed.
      *
      * @return If true, then cancel drawing the hunger bar.
+     *
      * @since 0.1-1.16.4-fabric
      */
     public static boolean drawHunger(float ticks)
@@ -843,7 +826,9 @@ public class TimerUtils
      * as needed.
      *
      * @param stack
+     *
      * @return If true, then cancel drawing the jump bar.
+     *
      * @since 0.1-1.16.4-fabric
      */
     public static boolean drawJumpbar(MatrixStack stack, float ticks)
@@ -862,7 +847,7 @@ public class TimerUtils
             return true;
         }
 
-        if (entity.method_3151() > 0)
+        if (entity.getMountJumpStrength() > 0)
         {
             jumpTime = jump.getMaxTime() - (jumpTime > 0 ?
                                             jump.getFadeInTime() :
@@ -890,6 +875,7 @@ public class TimerUtils
      * as needed.
      *
      * @return If true, then cancel drawing the experience bar.
+     *
      * @since 0.1-1.16.4-fabric
      */
     public static boolean drawExperience(MatrixStack stack, float ticks)
@@ -944,6 +930,7 @@ public class TimerUtils
      * or not to draw it.
      *
      * @return If true, then cancel drawing the hotbar.
+     *
      * @see #recolorHotbar(MatrixStack)
      * @since 0.1-1.16.4-fabric
      */
@@ -1138,9 +1125,9 @@ public class TimerUtils
         /*
          * Checks for a change in what slot is used.
          */
-        if (selectedHotbarSlot != entity.inventory.selectedSlot)
+        if (selectedHotbarSlot != entity.getInventory().selectedSlot)
         {
-            selectedHotbarSlot = entity.inventory.selectedSlot;
+            selectedHotbarSlot = entity.getInventory().selectedSlot;
             mainHandTime = handTime;
             changed = true;
         }
@@ -1163,19 +1150,25 @@ public class TimerUtils
      * Determines whether or not to draw the crosshair, adjusting the alpha as
      * needed.
      *
-     * @return If true, then cancel drawing the crosshair.
      * @since 0.1-1.16.4-fabric
      */
     public static void recolorHotbar(MatrixStack stack)
     {
         ConfigManager.TimeValues hotbar =
                 ConfigManager.getInstance().getHotbarTime();
-        RenderSystem.pushMatrix();
+//        RenderSystem.pushMatrix();
         stack.push();
-        RenderSystem.translatef(0F, TimerUtils.getHotbarTranslation(), 0F);
+        stack.translate(0F, TimerUtils.getHotbarTranslation(), 0F);
         setAlpha(Main.getAlpha(hotbarTime,
                 hotbar.getMaxTime(),
                 hotbar.getFadeInTime(),
                 hotbar.getFadeOutTime()));
+        /*
+         * Update the hotbar item matrix
+         */
+        stack = RenderSystem.getModelViewStack();
+        stack.push();
+        stack.translate(0F, TimerUtils.getHotbarTranslation(), 0F);
+        RenderSystem.applyModelViewMatrix();
     }
 }

src/main/java/markil3/immersive_hud/mixin/HudHook.java

@@ -40,7 +40,7 @@ import markil3.immersive_hud.TimerUtils;
  * that most of the logic can be found in {@link TimerUtils}.
  *
  * @author Markil 3
- * @version 0.1-1.16.4-fabric
+ * @version 1.1.1-1.17-fabric
  */
 @Mixin(InGameHud.class)
 public class HudHook
@@ -338,7 +338,7 @@ public class HudHook
     @Inject(method = "renderStatusBars(Lnet/minecraft/client/util" +
             "/math/MatrixStack;)V", at = @At(value = "INVOKE", target = "Lnet" +
             "/minecraft/util/profiler/Profiler;swap(Ljava/lang/String;)V",
-            ordinal = 1, shift = At.Shift.BY, by = 3))
+            ordinal = 1))
     public void renderFood(MatrixStack matrices, CallbackInfo callbackInfo)
     {
         try
@@ -634,7 +634,7 @@ public class HudHook
      * @since 0.1-1.16.4-fabric
      */
     @Inject(method = "renderHotbar", at = @At(value = "INVOKE", target =
-            "Lcom/mojang/blaze3d/systems/RenderSystem;color4f(FFFF)V", shift
+            "Lcom/mojang/blaze3d/systems/RenderSystem;setShaderColor(FFFF)V", shift
             = At.Shift.AFTER))
     public void adjustHotbar(float tickDelta,
                              MatrixStack matrices,
@@ -661,7 +661,7 @@ public class HudHook
      * @param stack - The matrix drawing stack.
      * @param callbackInfo
      *
-     * @version 0.2-1.16.4-fabric
+     * @version 1.1.1-1.17-fabric
      * @since 0.1-1.16.4-fabric
      */
     @Inject(method = "renderHotbar", at = @At(value = "TAIL"))
@@ -671,8 +671,10 @@ public class HudHook
     {
         try
         {
-            RenderSystem.popMatrix();
             stack.pop();
+            stack = RenderSystem.getModelViewStack();
+            stack.pop();
+            RenderSystem.applyModelViewMatrix();
             /*
              * Resets the color so that nothing else is bothered.
              */

src/main/resources/assets/immersive_hud/lang/en_us.json

@@ -25,7 +25,6 @@
   "option.immersive_hud.minHealth": "Minimum Health Fade",
   "option.immersive_hud.minHunger": "Minimum Hunger Fade",
   "option.immersive_hud.showArmor": "Show Armor",
-  "option.immersive_hud.ignoreList": "Ignored Items",
   "tooltip.immersive_hud.hotbarMaxTime": "How long the hotbar can be displayed on screen in seconds",
   "tooltip.immersive_hud.hotbarFadeIn": "How many seconds it takes for the hotbar to fade in",
   "tooltip.immersive_hud.hotbarFadeOut": "How many seconds it takes for the hotbar to move out",
@@ -50,8 +49,5 @@
   "tooltip.immersive_hud.hideHands": "Whether or not hands should be hidden after a period of time",
   "tooltip.immersive_hud.minHealth": "When the percentage of health goes below this level, the health bar won't fade away",
   "tooltip.immersive_hud.minHunger": "When the hunger level goes below this level, the hunger bar won't fade away",
-  "tooltip.immersive_hud.showArmor": "Whether or not the armor bar should display on the HUD",
-  "tooltip.immersive_hud.ignoreList": "Which items will not trigger the arm and crosshairs when used",
-  "error.immersive_hud.ignoreList.unknownItem": "Unknown item %s",
-  "error.immersive_hud.ignoreList.illegalId": "%s is not a legal item identifier"
+  "tooltip.immersive_hud.showArmor": "Whether or not the armor bar should display on the HUD"
 }

src/main/resources/fabric.mod.json

@@ -34,8 +34,9 @@
   ],
 
   "depends": {
-    "fabricloader": ">=0.7.4",
-    "minecraft": "1.16.x"
+    "fabricloader": ">=0.11.3",
+    "minecraft": "1.17.x",
+    "java": ">=16"
   },
   "suggests": {
     "modmenu": "*",

src/main/resources/immersive_hud.mixins.json

@@ -2,7 +2,7 @@
   "required": true,
   "minVersion": "0.8",
   "package": "markil3.immersive_hud.mixin",
-  "compatibilityLevel": "JAVA_8",
+  "compatibilityLevel": "JAVA_16",
   "mixins": [
   ],
   "client": [