From 3fa945d97a1e4f71dce5afe9237831f81fa5ec9a Mon Sep 17 00:00:00 2001 From: Sollace Date: Sat, 1 Jan 2022 17:23:28 +0200 Subject: [PATCH] More work on the hud and spell dismissal dialogue --- .../client/gui/DismissSpellScreen.java | 104 +++++++----------- .../unicopia/client/gui/DrawableUtil.java | 18 +++ .../unicopia/client/gui/UHud.java | 30 +++-- .../unicopia/client/particle/SphereModel.java | 28 +++-- 4 files changed, 90 insertions(+), 90 deletions(-) diff --git a/src/main/java/com/minelittlepony/unicopia/client/gui/DismissSpellScreen.java b/src/main/java/com/minelittlepony/unicopia/client/gui/DismissSpellScreen.java index bc8d6629..a8d8c5f8 100644 --- a/src/main/java/com/minelittlepony/unicopia/client/gui/DismissSpellScreen.java +++ b/src/main/java/com/minelittlepony/unicopia/client/gui/DismissSpellScreen.java @@ -11,7 +11,6 @@ import com.minelittlepony.unicopia.client.particle.SphereModel; import com.minelittlepony.unicopia.entity.player.Pony; import com.minelittlepony.unicopia.network.Channel; import com.minelittlepony.unicopia.network.MsgRemoveSpell; -import com.mojang.blaze3d.systems.RenderSystem; import net.minecraft.client.MinecraftClient; import net.minecraft.client.gui.Drawable; @@ -30,35 +29,22 @@ import net.minecraft.util.math.Vector4f; public class DismissSpellScreen extends GameGui { private final Pony pony = Pony.of(MinecraftClient.getInstance().player); - private final List spells; + private int relativeMouseX; + private int relativeMouseY; public DismissSpellScreen() { super(new TranslatableText("gui.unicopia.dismiss_spell")); - - spells = pony.getSpellSlot().stream(true).toList(); } @Override protected void init() { double azimuth = 0; - int ring = 1; + double ring = 2; - for (Spell spell : spells) { - - double increment = Math.PI / (8 * ring); - - azimuth += increment; - if (azimuth > Math.PI) { - azimuth = 0; - ring++; - } - - double radius = 75 * ring - 25; - - Vector4f pos = SphereModel.convertToCartesianCoord(radius, azimuth, azimuth); - pos.add(0, -(float)radius / 2F, 0, 0); - - addDrawableChild(new Entry(spell, pos)); + for (Spell spell : pony.getSpellSlot().stream(true).toList()) { + addDrawableChild(new Entry(spell, 75 * ring - 25, azimuth)); + ring *= 1 + 0.03 / ring; + azimuth += Math.PI / (8 * ring); } } @@ -67,26 +53,22 @@ public class DismissSpellScreen extends GameGui { fillGradient(matrices, 0, 0, width, height / 2, 0xF0101010, 0x80101010); fillGradient(matrices, 0, height / 2, width, height, 0x80101010, 0xF0101010); + relativeMouseX = -width + mouseX * 2; + relativeMouseY = -height + mouseY * 2; + matrices.push(); matrices.translate(width - mouseX, height - mouseY, 0); - - mouseX = -width + mouseX * 2; - mouseY = -height + mouseY * 2; - - DrawableUtil.drawLine(matrices, 0, 0, mouseX, mouseY, 0xFFFFFF88); - DrawableUtil.drawArc(matrices, 40, 80, 0, Math.PI * 2, 0x00000010, false); - DrawableUtil.drawArc(matrices, 160, 1600, 0, Math.PI * 2, 0x00000020, false); + DrawableUtil.drawLine(matrices, 0, 0, relativeMouseX, relativeMouseY, 0xFFFFFF88); + DrawableUtil.drawArc(matrices, 40, 80, 0, DrawableUtil.TWO_PI, 0x00000010, false); + DrawableUtil.drawArc(matrices, 160, 1600, 0, DrawableUtil.TWO_PI, 0x00000020, false); super.render(matrices, mouseX, mouseY, delta); - matrices.push(); - DrawableUtil.drawCircle(matrices, 2, 0, Math.PI * 2, 0xFFAAFF99, false); - matrices.translate(mouseX, mouseY, 0); - DrawableUtil.drawLine(matrices, 0, -4, 0, 4, 0xFFAAFF99); - DrawableUtil.drawLine(matrices, -4, 0, 4, 0, 0xFFAAFF99); + DrawableUtil.drawCircle(matrices, 2, 0, DrawableUtil.TWO_PI, 0xFFAAFF99, false); matrices.pop(); - matrices.pop(); + DrawableUtil.drawLine(matrices, mouseX, mouseY - 4, mouseX, mouseY + 4, 0xFFAAFF99); + DrawableUtil.drawLine(matrices, mouseX - 4, mouseY, mouseX + 4, mouseY, 0xFFAAFF99); matrices.push(); matrices.translate(0, 0, 300); @@ -105,21 +87,24 @@ public class DismissSpellScreen extends GameGui { return true; } - class Entry implements Element, Drawable, Selectable { + class Entry extends Vector4f implements Element, Drawable, Selectable { private final List tooltip = new ArrayList<>(); private final Spell spell; private final Spell actualSpell; - private final Vector4f pos; private boolean lastMouseOver; - public Entry(Spell spell, Vector4f pos) { + private final Vector4f copy = new Vector4f(); + + public Entry(Spell spell, double radius, double azimuth) { this.spell = spell; - this.pos = pos; this.actualSpell = getActualSpell(); + SphereModel.convertToCartesianCoord(this, radius, azimuth, azimuth); + add(0, -(float)radius / 2F, 0, 0); + MutableText name = actualSpell.getType().getName().shallowCopy(); int color = actualSpell.getType().getColor(); name.setStyle(name.getStyle().withColor(color == 0 ? 0xFFAAAAAA : color)); @@ -143,46 +128,41 @@ public class DismissSpellScreen extends GameGui { @Override public boolean mouseClicked(double mouseX, double mouseY, int button) { - remove(this); - pony.getSpellSlot().removeIf(spell -> spell == this.spell, true); - Channel.REMOVE_SPELL.send(new MsgRemoveSpell(spell)); - playClickEffect(); - return true; + if (isMouseOver(relativeMouseX, relativeMouseY)) { + remove(this); + pony.getSpellSlot().removeIf(spell -> spell == this.spell, true); + Channel.REMOVE_SPELL.send(new MsgRemoveSpell(spell)); + playClickEffect(); + return true; + } + return false; } @Override public boolean isMouseOver(double mouseX, double mouseY) { - return squareDistance(mouseX, mouseY, pos.getX(), pos.getY()) < 75; + return squareDistance(mouseX, mouseY, getX(), getY()) < 75; } @Override public void render(MatrixStack matrices, int mouseX, int mouseY, float tickDelta) { - Vector4f copy = new Vector4f(pos.getX(), pos.getY(), pos.getZ(), pos.getW()); + copy.set(getX(), getY(), getZ(), getW()); copy.transform(matrices.peek().getPositionMatrix()); - boolean hovered = isMouseOver(mouseX, mouseY); - - MatrixStack modelStack = RenderSystem.getModelViewStack(); - modelStack.push(); - float scale = 1.1F; - modelStack.translate(-21, -14, 0); - modelStack.scale(scale, scale, 1); - RenderSystem.applyModelViewMatrix(); - - client.getItemRenderer().renderGuiItemIcon(actualSpell.getType().getDefualtStack(), (int)copy.getX() - 8, (int)copy.getY() - 8); - - modelStack.pop(); - RenderSystem.applyModelViewMatrix(); + DrawableUtil.renderItemIcon(actualSpell.getType().getDefualtStack(), + copy.getX() - 8 + (copy.getX() - mouseX - 5) / 60D, + copy.getY() - 8 + (copy.getY() - mouseY - 5) / 60D, + 1 + ); matrices.push(); - matrices.translate(pos.getX(), pos.getY(), 0); + matrices.translate(getX(), getY(), 0); int color = actualSpell.getType().getColor() << 2; - DrawableUtil.drawArc(matrices, 7, 8, 0, Math.PI * 2, color | 0x00000088, false); + DrawableUtil.drawArc(matrices, 7, 8, 0, DrawableUtil.TWO_PI, color | 0x00000088, false); - if (hovered) { - DrawableUtil.drawArc(matrices, 0, 8, 0, Math.PI * 2, color | 0x000000FF, false); + if (isMouseOver(relativeMouseX, relativeMouseY)) { + DrawableUtil.drawArc(matrices, 0, 8, 0, DrawableUtil.TWO_PI, color | 0x000000FF, false); renderTooltip(matrices, tooltip, 0, 0); if (!lastMouseOver) { diff --git a/src/main/java/com/minelittlepony/unicopia/client/gui/DrawableUtil.java b/src/main/java/com/minelittlepony/unicopia/client/gui/DrawableUtil.java index 13dc3a9a..32284e7f 100644 --- a/src/main/java/com/minelittlepony/unicopia/client/gui/DrawableUtil.java +++ b/src/main/java/com/minelittlepony/unicopia/client/gui/DrawableUtil.java @@ -2,6 +2,7 @@ package com.minelittlepony.unicopia.client.gui; import com.mojang.blaze3d.systems.RenderSystem; +import net.minecraft.client.MinecraftClient; import net.minecraft.client.render.BufferBuilder; import net.minecraft.client.render.BufferRenderer; import net.minecraft.client.render.GameRenderer; @@ -9,6 +10,7 @@ import net.minecraft.client.render.Tessellator; import net.minecraft.client.render.VertexFormat; import net.minecraft.client.render.VertexFormats; import net.minecraft.client.util.math.MatrixStack; +import net.minecraft.item.ItemStack; import net.minecraft.util.math.MathHelper; import net.minecraft.util.math.Matrix4f; @@ -17,6 +19,22 @@ public class DrawableUtil { private static final double NUM_RINGS = 300; private static final double INCREMENT = TWO_PI / NUM_RINGS; + + public static void renderItemIcon(ItemStack stack, double x, double y, float scale) { + MatrixStack modelStack = RenderSystem.getModelViewStack(); + modelStack.push(); + modelStack.translate(x, y, 0); + if (scale != 1) { + modelStack.scale(scale, scale, 1); + } + RenderSystem.applyModelViewMatrix(); + + MinecraftClient.getInstance().getItemRenderer().renderGuiItemIcon(stack, 0, 0); + + modelStack.pop(); + RenderSystem.applyModelViewMatrix(); + } + public static void drawLine(MatrixStack matrices, int x1, int y1, int x2, int y2, int color) { RenderSystem.enableBlend(); RenderSystem.disableTexture(); diff --git a/src/main/java/com/minelittlepony/unicopia/client/gui/UHud.java b/src/main/java/com/minelittlepony/unicopia/client/gui/UHud.java index 58e2dad7..6a71dda3 100644 --- a/src/main/java/com/minelittlepony/unicopia/client/gui/UHud.java +++ b/src/main/java/com/minelittlepony/unicopia/client/gui/UHud.java @@ -156,26 +156,24 @@ public class UHud extends DrawableHelper { public void renderSpell(CustomisedSpellType spell, double x, double y) { if (!spell.isEmpty()) { - MatrixStack modelStack = RenderSystem.getModelViewStack(); - - modelStack.push(); - modelStack.translate(x, y, 0); - modelStack.scale(EQUIPPED_GEMSTONE_SCALE, EQUIPPED_GEMSTONE_SCALE, 1); - - RenderSystem.applyModelViewMatrix(); - client.getItemRenderer().renderGuiItemIcon(spell.getDefaultStack(), 0, 0); - - modelStack.loadIdentity(); - modelStack.translate(7.5, 8.5, 0); - Pony pony = Pony.of(client.player); if (spell.isOn(pony)) { - DrawableUtil.drawArc(modelStack, 5, 7, 0, Math.PI * 2, (spell.type().getColor() | 0x000000FF) & 0xFFFFFFAF, false); - DrawableUtil.drawArc(modelStack, 7, 8, 0, Math.PI * 2, 0xAAAAFFFF, false); + MatrixStack modelStack = new MatrixStack(); + + modelStack.push(); + modelStack.translate(x + 5.5, y + 5.5, 0); + + int color = spell.type().getColor() | 0x000000FF; + double radius = 2 + Math.sin(client.player.age / 9D) / 4; + + DrawableUtil.drawArc(modelStack, radius, radius + 3, 0, DrawableUtil.TWO_PI, color & 0xFFFFFF2F, false); + DrawableUtil.drawArc(modelStack, radius + 3, radius + 4, 0, DrawableUtil.TWO_PI, color & 0xFFFFFFAF, false); + + modelStack.pop(); } - modelStack.pop(); - RenderSystem.applyModelViewMatrix(); + + DrawableUtil.renderItemIcon(spell.getDefaultStack(), x, y, EQUIPPED_GEMSTONE_SCALE); } } diff --git a/src/main/java/com/minelittlepony/unicopia/client/particle/SphereModel.java b/src/main/java/com/minelittlepony/unicopia/client/particle/SphereModel.java index 93270f6f..017a5dcf 100644 --- a/src/main/java/com/minelittlepony/unicopia/client/particle/SphereModel.java +++ b/src/main/java/com/minelittlepony/unicopia/client/particle/SphereModel.java @@ -30,21 +30,22 @@ public class SphereModel { } Matrix4f position = matrices.peek().getPositionMatrix(); + Vector4f vec = new Vector4f(); for (double zenith = -PI; zenith < PI; zenith += zenithIncrement) { for (double azimuth = -azimuthRange; azimuth < azimuthRange; azimuth += azimuthIncrement) { - drawQuad(position, vertexWriter, radius, zenith, azimuth, light, overlay, r, g, b, a); + drawQuad(position, vec, vertexWriter, radius, zenith, azimuth, light, overlay, r, g, b, a); } } } - private void drawQuad(Matrix4f model, VertexConsumer vertexWriter, + private void drawQuad(Matrix4f model, Vector4f vec, VertexConsumer vertexWriter, double radius, double zenith, double azimuth, int light, int overlay, float r, float g, float b, float a) { - drawVertex(model, vertexWriter, convertToCartesianCoord(radius, zenith, azimuth), light, overlay, r, g, b, a); - drawVertex(model, vertexWriter, convertToCartesianCoord(radius, zenith + zenithIncrement, azimuth), light, overlay, r, g, b, a); - drawVertex(model, vertexWriter, convertToCartesianCoord(radius, zenith + zenithIncrement, azimuth + azimuthIncrement), light, overlay, r, g, b, a); - drawVertex(model, vertexWriter, convertToCartesianCoord(radius, zenith, azimuth + azimuthIncrement), light, overlay, r, g, b, a); + drawVertex(model, vertexWriter, convertToCartesianCoord(vec, radius, zenith, azimuth), light, overlay, r, g, b, a); + drawVertex(model, vertexWriter, convertToCartesianCoord(vec, radius, zenith + zenithIncrement, azimuth), light, overlay, r, g, b, a); + drawVertex(model, vertexWriter, convertToCartesianCoord(vec, radius, zenith + zenithIncrement, azimuth + azimuthIncrement), light, overlay, r, g, b, a); + drawVertex(model, vertexWriter, convertToCartesianCoord(vec, radius, zenith, azimuth + azimuthIncrement), light, overlay, r, g, b, a); } private static void drawVertex(Matrix4f model, VertexConsumer vertexWriter, Vector4f position, int light, int overlay, float r, float g, float b, float a) { @@ -52,11 +53,14 @@ public class SphereModel { vertexWriter.vertex(position.getX(), position.getY(), position.getZ(), r, g, b, a, 0, 0, overlay, light, 0, 0, 0); } - public static Vector4f convertToCartesianCoord(double r, double theta, double phi) { - double x = r * Math.sin(theta) * Math.cos(phi); - double y = r * Math.sin(theta) * Math.sin(phi); - double z = r * Math.cos(theta); - - return new Vector4f((float)x, (float)y, (float)z, 1); + public static Vector4f convertToCartesianCoord(Vector4f output, double r, double theta, double phi) { + double st = Math.sin(theta); + output.set( + (float)(r * st * Math.cos(phi)), + (float)(r * st * Math.sin(phi)), + (float)(r * Math.cos(theta)), + 1 + ); + return output; } }