Remove known incompatibilities list from README and sources

Both DashLoader and ArmorSkin are quite inactive and don't work with the latest game version, in addition to that, I don't think many people will try to combine viafabric and viafabricplus since the mod pages already say it's an independent mod, so I decided to just yeet the list away ™️
This commit is contained in:
FlorianMichael 2024-03-20 21:03:56 +01:00
parent 309c9f37fb
commit 096afc54b6
No known key found for this signature in database
GPG Key ID: C2FB87E71C425126
4 changed files with 4 additions and 22 deletions

View File

@ -23,11 +23,6 @@ At the time of writing, VFP is the only protocol translation platform for the cl
- Snapshots (3D Shareware, 20w14infinite, Combat Test 8C)
- Bedrock Edition 1.20.50 (Experimental)
## Known incompatibilities
- ***[ViaFabric](https://github.com/ViaVersion/ViaFabric)*** - Both mods are doing the same thing and conflict
- ***[DashLoader](https://github.com/alphaqu/DashLoader/tree/fabric-1.20)*** - Font rendering related fixes aren't working
- ***[ArmorSkin](https://github.com/IzzyDotExe/ArmorSkin)*** - Beta HUD changes aren't working
## For players
- Tutorial for installing and using the mod can be found [here](docs/USAGE.md)
- If you encounter any issues, please report them on the [issue tracker](https://github.com/ViaVersion/ViaFabricPlus/issues) or on the ViaVersion [Discord](https://discord.gg/viaversion)

View File

@ -84,12 +84,10 @@ public class ClientsideFixes {
// Reloads some clientside stuff when the protocol version changes
ChangeProtocolVersionCallback.EVENT.register((oldVersion, newVersion) -> MinecraftClient.getInstance().execute(() -> {
if (!ViaFabricPlusMixinPlugin.DASH_LOADER_PRESENT) {
// Clear all font caches to enforce a reload of all fonts (this is needed because we change the font renderer behavior)
for (FontStorage storage : MinecraftClient.getInstance().fontManager.fontStorages.values()) {
storage.glyphRendererCache.clear();
storage.glyphCache.clear();
}
// Clear all font caches to enforce a reload of all fonts (this is needed because we change the font renderer behavior)
for (FontStorage storage : MinecraftClient.getInstance().fontManager.fontStorages.values()) {
storage.glyphRendererCache.clear();
storage.glyphCache.clear();
}
// Reloads all bounding boxes of the blocks that we changed

View File

@ -32,13 +32,10 @@ public class ViaFabricPlusMixinPlugin implements IMixinConfigPlugin {
public static final String INJECTOR_PACKAGE = "de.florianmichael.viafabricplus.injection.mixin.";
private static final String MC_FIXES_PACKAGE = "fixes.minecraft.";
private static final String COMPAT_PACKAGE = "compat.";
public static String VFP_VERSION;
public static boolean DASH_LOADER_PRESENT;
public static boolean ARMOR_SKIN_PRESENT;
public static boolean IPNEXT_PRESENT;
public static boolean MORE_CULLING_PRESENT;
public static boolean LITHIUM_PRESENT;
@ -49,8 +46,6 @@ public class ViaFabricPlusMixinPlugin implements IMixinConfigPlugin {
VFP_VERSION = loader.getModContainer("viafabricplus").get().getMetadata().getVersion().getFriendlyString();
DASH_LOADER_PRESENT = loader.isModLoaded("dashloader");
ARMOR_SKIN_PRESENT = loader.isModLoaded("armorskin");
IPNEXT_PRESENT = loader.isModLoaded("inventoryprofilesnext");
MORE_CULLING_PRESENT = loader.isModLoaded("moreculling");
LITHIUM_PRESENT = loader.isModLoaded("lithium");
@ -70,8 +65,6 @@ public class ViaFabricPlusMixinPlugin implements IMixinConfigPlugin {
@Override
public boolean shouldApplyMixin(String targetClassName, String mixinClassName) {
return switch (mixinClassName) {
case INJECTOR_PACKAGE + MC_FIXES_PACKAGE + "MixinFontStorage" -> !DASH_LOADER_PRESENT;
case INJECTOR_PACKAGE + MC_FIXES_PACKAGE + "MixinInGameHud" -> !ARMOR_SKIN_PRESENT;
case INJECTOR_PACKAGE + COMPAT_PACKAGE + "ipnext.MixinAutoRefillHandler_ItemSlotMonitor" -> IPNEXT_PRESENT;
case INJECTOR_PACKAGE + COMPAT_PACKAGE + "lithium.MixinEntity" -> LITHIUM_PRESENT;
default -> true;

View File

@ -58,9 +58,5 @@
},
"breaks": {
"viafabric": "*"
},
"conflicts": {
"dashloader": "*",
"armorskin": "*"
}
}