Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

EMI doesn't display fluid temperatures anymore #2748

Open
2 tasks done
DSantiagoBC opened this issue Jan 15, 2025 · 2 comments · May be fixed by #2753
Open
2 tasks done

EMI doesn't display fluid temperatures anymore #2748

DSantiagoBC opened this issue Jan 15, 2025 · 2 comments · May be fixed by #2753
Labels
type: bug Something isn't working

Comments

@DSantiagoBC
Copy link

Checked for existing issues

  • I have checked for existing issues, and have found none.

Tested latest version

  • I have checked that this occurs on the latest version.

GregTech CEu Version

0.11.3

Minecraft Version

1.20.1 Forge

Recipe Viewer Installed

EMI

Environment

Singleplayer

Cross-Mod Interaction

Yes

Other Installed Mods

Monifactory

Expected Behavior

Fluid tooltips on EMI's item list and recipe preview should display the temperature of the liquid, so you can prepare your setup accordingly

Actual Behavior

Temperature is only displayed when the fluid is in a GTCEu machine

Steps to Reproduce

Just lookup any fluid on EMI with the default configuration

Additional Information

No response
@DSantiagoBC DSantiagoBC added the type: bug Something isn't working label Jan 15, 2025
@DiFFoZ
Copy link

DiFFoZ commented Jan 15, 2025 •
edited
Loading

Can confirm, 1.6.3 GTceu version doesn't show tooltip of the fluid in the EMI, but fluid in the buckets it shows the tooltip in EMI.

Probably regression from #2609

@krossgg
Copy link
Contributor

krossgg commented Jan 15, 2025

Not a regression - EMI's latest update changed some API things, which unfortunately broke how we render those tooltips. We are now aware of this and will work on figuring it out.
If it inconveniences you greatly, downgrading EMI by one minor version will fix it for now.

@omergunr100 omergunr100 linked a pull request Jan 15, 2025 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type: bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Refactor the EMI mixin omergunr100/GregTech-Modern
3 participants
@DSantiagoBC @DiFFoZ @krossgg