-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add Scene View Overlay Toolbar #6
Conversation
Added: + New overlay toolbar + About window + New scriptable object class for editor resources + Icons required for about window Modified: * Made DynamicLightManager.EditorDeleteLightmaps() public * Commented out old menu items for support links * Added reference to RealtimeCSG in the editor asmdef * Added version define for RealtimeCSG to the editor asmdef
Hey Fluffles, thank you for your contribution to the project! I appreciate the effort and time you've put into this. I did notice a couple things. On monitors without UI scaling the about dialog is very small and hard to read: When you open the menu for other light types multiple times, it continues adding separator bars at the bottom: The light creation options in the create menu have additional settings to set up the light sources. These options are not being applied. For example the Discoball light: I am using CodeMaid with default Visual Studio formatting. Please use the same coding style as the rest of the project: Regarding the attribution comments in the code, I completely understand the desire to receive credit for your work. However, having the following comment at the top of every file: // * * * * * * * * * * * * * * * * * * * * * *
// Author: Lindsey Keene (nukeandbeans)
// Contact: Twitter @nukeandbeans, Discord @nukeandbeans
//
// Description:
//
// * * * * * * * * * * * * * * * * * * * * * * Feels a bit out of place, especially on smaller files like editor resources. I'm sure we can find a way to properly acknowledge your contributions, I like having you in the about dialog and I can even add a dedicated section in the documentation. This way, your work is recognized without the need for extensive comments throughout the codebase. What do you think? Looking forward to your thoughts! |
This code does not compile on Unity 2020 because it's using C# 9.0 features. LinkButton discordButton = new(); // Feature 'target-typed object creation' is not available in C# 8.0. Please use language version 9.0 or greater. But also because it doesn't yet know Also Unity objects should not use coalescing assignment, the ??= operator. It will throw warning UNT0023 and does not work the way you expect ( |
* Adjusted scaling for about window UI * Fixed disco ball lights not having proper settings when created * Fixed spacers being added to the bake dropdown endlessly * Fixed null coalescing issue (thanks again, Unity 😦) * Fixed compilation issues on older versions of Unity that doesn't support overlays
Thank you so much Fluffles! This is a big quality of life improvement! 😁 |
What this does:
How do I use?
Press space bar while the scene view is focused, and toggle the "Dynamic Lighting" overlay to show and hide it.
What can I do with it?
Extras: