Fix Toolscreen Background #1979
Merged
+5
−12
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The RenderTarget was drawn on top of itself for some reason. It caused the background to be pure black on Windows. The Linux-only fix is also gone, removed
MATERIAL_VAR_VERTEXALPHAflag instead. I added clearing the RT on creation just to avoid potential black-background on start.I also tweaked the colors and trail a bit, but nothing major. Feel free to modify it if you think it looks bad.
Before and after (pic on the right is how it always looked on Linux and how it was meant to look like from the beginning):
Tested both Linux and Windows (Wine), 64 and 32. But it shouldn't matter now anyway.