Skip to content

Conversation

@ghzdude
Copy link
Contributor

@ghzdude ghzdude commented May 19, 2025

What

Ports every singleblock Recipemap machine and generator, along with steam machines/boilers, to MUI2
Built off of the sb-recipe-map-mui branch

Implementation Details

adds usesMui2() to SimpleGeneratorMTE, SimpleMachineMTE, and SteamMTE to defer to recipe map ui's usesMui2()
steam boilers ui is manually constructed, and therefore ported directly to Mui2
slot overlays for mui2 are stored in an Int2Obj map, with the key being the slot index
adds RecipeProgressWidget to allow for JEI searching
adds RecipeMapUIBuilder for building the mui2 slot overlays
deprecate old mui code and methods

Outcome

the last set of uis should be ported to mui2

@ghzdude ghzdude requested a review from a team as a code owner May 19, 2025 03:54
@ghzdude ghzdude added the type: refactor Suggestion to refactor a section of code label May 19, 2025
@ghzdude ghzdude force-pushed the gh/sb-recipe-map-mui branch from d417cfc to c35a8d2 Compare May 19, 2025 03:56
Copy link
Contributor

@M-W-K M-W-K left a comment

Choose a reason for hiding this comment

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

Did a first look through the code, did not test ingame.

* @return this
*/
@Deprecated
@ApiStatus.ScheduledForRemoval(inVersion = "2.9")
Copy link
Contributor

Choose a reason for hiding this comment

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

Isn't our next release going to be 2.9? Probably ought to schedule removal for 2.10 or so.

@ghzdude ghzdude moved this to Hard in MUI2 Port May 19, 2025
Copy link
Contributor

@ALongStringOfNumbers ALongStringOfNumbers left a comment

Choose a reason for hiding this comment

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

The fluid amount overlay on slots seems to be slightly adjusted from what it was before. Was that something touched in this PR, or am I just crazy?

@Zorbatron
Copy link
Member

Painting a machine doesn't change its UI color and the defaultUIColor config doesn't change anything (it appears there is something wrong with the GT UI theme).

@ALongStringOfNumbers
Copy link
Contributor

Also, we think that we should implement the side column, like in multiblocks and hatches, for clearing out some space in the GUI. See the discussion starting here: https://discord.com/channels/701354865217110096/904846945095385138/1408285204795359292

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: refactor Suggestion to refactor a section of code

Projects

Status: Hard

Development

Successfully merging this pull request may close these issues.

6 participants