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.

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

[BUG] Terminal Console App rendering issue #1012

Closed
lordclips opened this issue Jun 5, 2022 · 9 comments
Closed

[BUG] Terminal Console App rendering issue #1012

lordclips opened this issue Jun 5, 2022 · 9 comments
Labels
status: unverified Issue is unverified type: bug Something isn't working

Comments

@lordclips
Copy link
Contributor

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.

@lordclips lordclips added status: unverified Issue is unverified type: bug Something isn't working labels Jun 5, 2022
@brachy84 brachy84 changed the title [BUG] The Solar Panel must be put into its proper place in the Console App [BUG] Terminal Console App rendering issue Jun 5, 2022
@brachy84
Copy link
Contributor

brachy84 commented Jun 5, 2022

Will probably be fixed with the ModularUI rework in 4 years

@lordclips
Copy link
Contributor Author

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
Copy link
Contributor

brachy84 commented Jun 5, 2022

Everything ui related is "kind of" messy

@lordclips
Copy link
Contributor Author

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
Copy link
Contributor Author

Turns out it is generically a cover issue

@lordclips
Copy link
Contributor Author

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
Copy link
Contributor

Syrcan commented Sep 6, 2022

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

@Syrcan Syrcan closed this as completed Sep 6, 2022
@ALongStringOfNumbers
Copy link
Contributor

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

@Syrcan
Copy link
Contributor

Syrcan commented Sep 8, 2022

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

@Syrcan Syrcan closed this as completed Sep 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: unverified Issue is unverified type: bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants