run gradlew migrateMappings --mappings "1.17+build.10"
update to gradle 7.0.2, remove deprecated jcenter()
update to loom 0.8, Java 16 (now required for building)
update dependencies
mixin into new renderTooltipFromComponents() taking List<TooltipComponent> instead of renderOrderedTooltip() taking List<? extends OrderedText> because the latter directly calls the former
Note: This could have unintended side-effects I'm not aware of.
fix drawTooltipIcons(): RenderSystem.scalef() doesn't exist anymore, replaced by matrices.scale(). RenderSystem.enableRescaleNormal() seems to have no replacement at all, so I just left it out - let's hope this is fine.
fix drawScaledItem():
Here, matrices.scale() doesn't work because renderGuiItemIcon() uses a different MatrixStack, namely RenderSystem.getModelViewStack(). It also calls RenderSystem.applyModelViewMatrix(); after it changing it - I have no idea what the latter does, but I applied it anyways.
I didn't do a ton of testing but so far everything seems fine.
You might wanna pull this and push it to a separate branch because GitHub doesn't allow PRs to new branches
Thanks for the pr, I'll do the basic testing I usually do when I make big changes and I'll just release it as a beta for now just in case something is in fact broken.
The process looked like this:
gradlew migrateMappings --mappings "1.17+build.10"
jcenter()
renderTooltipFromComponents()
takingList<TooltipComponent>
instead ofrenderOrderedTooltip()
takingList<? extends OrderedText>
because the latter directly calls the formerNote: This could have unintended side-effects I'm not aware of.
fix
drawTooltipIcons()
:RenderSystem.scalef()
doesn't exist anymore, replaced bymatrices.scale()
.RenderSystem.enableRescaleNormal()
seems to have no replacement at all, so I just left it out - let's hope this is fine.fix
drawScaledItem()
:Here,
matrices.scale()
doesn't work becauserenderGuiItemIcon()
uses a differentMatrixStack
, namelyRenderSystem.getModelViewStack()
. It also callsRenderSystem.applyModelViewMatrix();
after it changing it - I have no idea what the latter does, but I applied it anyways.I didn't do a ton of testing but so far everything seems fine. You might wanna pull this and push it to a separate branch because GitHub doesn't allow PRs to new branches