Skip to content

[MUI2] Battery Buffer#4281

Merged
TarLaboratories merged 14 commits into
mui2-refactorfrom
tar/mui-battery-buffer
Dec 13, 2025
Merged

[MUI2] Battery Buffer#4281
TarLaboratories merged 14 commits into
mui2-refactorfrom
tar/mui-battery-buffer

Conversation

@TarLaboratories

@TarLaboratories TarLaboratories commented Dec 6, 2025

Copy link
Copy Markdown
Contributor

What

Battery buffer UI

Implementation details

Someone will need to fix the dupe later (originating from slots not being properly synced in GTMuiMachineUtil

Outcome

image image image

@TarLaboratories
TarLaboratories requested a review from a team as a code owner December 6, 2025 15:48
@TarLaboratories TarLaboratories mentioned this pull request Dec 6, 2025
@TarLaboratories TarLaboratories added type: feature New feature or request bundled for a 0.X.0 Update ignore changelog PR should not be added to the changelog. 1.20.1 Release: Patch - 0.0.X Smaller changes that either are bug fixes or very minor tweaks. labels Dec 6, 2025
@github-actions github-actions Bot added the Tests: Passed Game Tests have passed on this PR label Dec 6, 2025

@jurrejelle jurrejelle left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some questions, looks good for the rest! Maybe include a "before" screenshot so people can compare?

@jurrejelle jurrejelle left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, would like a review from someone with more experience with MUI if possible :hapybapycat:

@jurrejelle

Copy link
Copy Markdown
Contributor

Before pics:
image
image
image

@TarLaboratories

TarLaboratories commented Dec 6, 2025

Copy link
Copy Markdown
Contributor Author

Added power button
image

@screret

screret commented Dec 6, 2025

Copy link
Copy Markdown
Contributor

IMO the 2nd screenshot with the 2x4 grid should be laid out as 4x2 instead

@jurrejelle

Copy link
Copy Markdown
Contributor

Agreed since that would follow the previous LDLib impl

@jurrejelle

Copy link
Copy Markdown
Contributor
java_5TFh4WG2PX.mp4

LGTM

@TarLaboratories
TarLaboratories added this pull request to the merge queue Dec 13, 2025
Merged via the queue into mui2-refactor with commit 01a95bd Dec 13, 2025
4 checks passed
@TarLaboratories
TarLaboratories deleted the tar/mui-battery-buffer branch December 13, 2025 12:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.20.1 ignore changelog PR should not be added to the changelog. Release: Patch - 0.0.X Smaller changes that either are bug fixes or very minor tweaks. Tests: Passed Game Tests have passed on this PR type: feature New feature or request bundled for a 0.X.0 Update

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants