GregTechCEu / GregTech

GregTech CE 1.12 fork continuing progression and development
GNU Lesser General Public License v3.0
227 stars 174 forks source link

[BUG] Terminal Console App rendering issue #1012

Closed lordclips closed 1 year ago

lordclips commented 2 years ago

Describe the bug Solar Panel is an attention craving motherfucker in the Console app

2022-06-05_01 22 49 2022-06-05_01 22 53

Versions Forge: Whatever 2.3.1 uses GTCEu: 2.3.1

Steps To Reproduce

  1. Make machine
  2. Put solar panel on machine
  3. ???
  4. Profit
  5. But also use Console App
  6. Open Machine gui
  7. See solar panel indicator peeking through into the GUI of the machine.

Expected behavior The solar panel is put in its proper place.

Additional context Just a fun fact, discovered this fixing something else.

brachy84 commented 2 years ago

Will probably be fixed with the ModularUI rework in 4 years

lordclips commented 2 years ago

Considering I have like, most applicable things open in my IDE atm ill try to see if I cant find it anyway, but if not then yeah, its a fairly niche issue to have

Console app is just kind of a messy one

brachy84 commented 2 years ago

Everything ui related is "kind of" messy

lordclips commented 2 years ago

Fair 'nuf. Figure I might as well just do it since I'm here and in waiting for the UI rework, just as a nicety

lordclips commented 2 years ago

Turns out it is generically a cover issue

lordclips commented 2 years ago

I checked and it looks like SlotWidget being a Native Widget and using vanilla Slots is probably what it is but I am unlikely to reasonably fix that, so this'll just sit open while the ModularUI rework works its way through

Syrcan commented 1 year ago

Terminal rendering issues are known as in shown in #446. So closing this specific issue.

ALongStringOfNumbers commented 1 year ago

This is a different issue than the one in 446, so reopening.

Syrcan commented 1 year ago

Alright, time to actually close it since I transferred it to #446, so it can rest in peace with other terminal issues.