Skip to content

Commit 5ab3620

Browse files
committed
Address reservoir hatch review
1 parent 76e1c1f commit 5ab3620

File tree

1 file changed

+21
-26
lines changed

1 file changed

+21
-26
lines changed

src/main/java/gregtech/common/metatileentities/multi/multiblockpart/MetaTileEntityReservoirHatch.java

Lines changed: 21 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
import gregtech.api.mui.GTGuiTextures;
1212
import gregtech.api.mui.GTGuis;
1313
import gregtech.api.mui.sync.GTFluidSyncHandler;
14-
import gregtech.api.util.TextFormattingUtil;
1514
import gregtech.client.renderer.texture.Textures;
1615
import gregtech.common.mui.widget.GTFluidSlot;
1716

@@ -37,9 +36,12 @@
3736
import com.cleanroommc.modularui.api.drawable.IKey;
3837
import com.cleanroommc.modularui.factory.PosGuiData;
3938
import com.cleanroommc.modularui.screen.ModularPanel;
39+
import com.cleanroommc.modularui.utils.Alignment;
40+
import com.cleanroommc.modularui.utils.Color;
4041
import com.cleanroommc.modularui.value.sync.PanelSyncManager;
4142
import com.cleanroommc.modularui.value.sync.SyncHandlers;
4243
import com.cleanroommc.modularui.widgets.ItemSlot;
44+
import com.cleanroommc.modularui.widgets.RichTextWidget;
4345
import com.cleanroommc.modularui.widgets.SlotGroupWidget;
4446
import org.jetbrains.annotations.NotNull;
4547
import org.jetbrains.annotations.Nullable;
@@ -130,7 +132,8 @@ public boolean usesMui2() {
130132
public ModularPanel buildUI(PosGuiData guiData, PanelSyncManager guiSyncManager) {
131133
guiSyncManager.registerSlotGroup("item_inv", 2);
132134

133-
GTFluidSyncHandler tankSyncHandler = new GTFluidSyncHandler(this.fluidTank)
135+
GTFluidSyncHandler tankSyncHandler = GTFluidSlot.sync(this.fluidTank)
136+
.showAmount(false)
134137
.accessibility(true, false);
135138

136139
// TODO: Change the position of the name when it's standardized.
@@ -143,15 +146,23 @@ public ModularPanel buildUI(PosGuiData guiData, PanelSyncManager guiSyncManager)
143146
.child(GTGuiTextures.TANK_ICON.asWidget()
144147
.left(92).top(36)
145148
.size(14, 15))
146-
.child(IKey.lang("gregtech.gui.fluid_amount").color(0xFFFFFF).asWidget().pos(11, 20))
147-
.child(IKey.dynamic(() -> getFluidAmountFormatted(tankSyncHandler))
148-
.color(0xFFFFFF)
149-
.asWidget().pos(11, 30))
150-
.child(IKey.dynamic(() -> getFluidNameTranslated(tankSyncHandler))
151-
.color(0xFFFFFF)
152-
.asWidget().pos(11, 40))
149+
.child(new RichTextWidget()
150+
.size(75, 47)
151+
.pos(10, 20)
152+
.textColor(Color.WHITE.main)
153+
.alignment(Alignment.TopLeft)
154+
.autoUpdate(true)
155+
.textBuilder(richText -> {
156+
richText.addLine(IKey.lang("gregtech.gui.fluid_amount"));
157+
String name = tankSyncHandler.getFluidLocalizedName();
158+
if (name == null) return;
159+
160+
richText.addLine(IKey.str(name));
161+
richText.addLine(IKey.str(tankSyncHandler.getFormattedFluidAmount()));
162+
}))
153163
.child(new GTFluidSlot().syncHandler(tankSyncHandler)
154-
.pos(69, 52))
164+
.pos(69, 52)
165+
.disableBackground())
155166
.child(new ItemSlot().slot(SyncHandlers.itemSlot(this.importItems, 0)
156167
.slotGroup("item_inv")
157168
.filter(itemStack -> FluidUtil.getFluidHandler(itemStack) != null))
@@ -164,22 +175,6 @@ public ModularPanel buildUI(PosGuiData guiData, PanelSyncManager guiSyncManager)
164175
.pos(90, 53));
165176
}
166177

167-
private String getFluidNameTranslated(GTFluidSyncHandler tankSyncHandler) {
168-
if (tankSyncHandler.getFluid() == null) {
169-
return "";
170-
} else {
171-
return tankSyncHandler.getFluid().getLocalizedName();
172-
}
173-
}
174-
175-
private String getFluidAmountFormatted(GTFluidSyncHandler tankSyncHandler) {
176-
if (tankSyncHandler.getFluid() == null) {
177-
return "0";
178-
} else {
179-
return TextFormattingUtil.formatNumbers(tankSyncHandler.getFluid().amount);
180-
}
181-
}
182-
183178
@Override
184179
public void addInformation(ItemStack stack, @Nullable World player, @NotNull List<String> tooltip,
185180
boolean advanced) {

0 commit comments

Comments
 (0)