Skip to content
This repository was archived by the owner on Dec 5, 2024. It is now read-only.

Conversation

@StanleyGoldman
Copy link
Contributor

@StanleyGoldman StanleyGoldman commented Nov 29, 2017

Note: This branch targets enhancements/branches-view-rollup #464

  • Learned from @CapnRat that it is problematic to attempt to serialize Texture2D objects and it is better to provide them OnEnable. A large component to this fix is the usage of HideFlags.HideAndDontSave which prevents Unity from garbage collecting icons after they load during "game play".
  • We also subclassed Tree into BranchesTree in order to provide custom logic to set the icons for trees that represent a branch

# Conflicts:
#	src/UnityExtension/Assets/Editor/GitHub.Unity/UI/TreeControl.cs
I learned that is how we can tell Unity that Texture2D objects that were loaded during "game time" need to be retained after "game time" ends
@StanleyGoldman StanleyGoldman force-pushed the fixes/branches-view-remove-serialized-texture2d branch from c295c37 to f98db05 Compare November 29, 2017 22:31
@StanleyGoldman StanleyGoldman mentioned this pull request Nov 29, 2017
15 tasks
@StanleyGoldman StanleyGoldman force-pushed the fixes/branches-view-remove-serialized-texture2d branch from a7fbc49 to 03d85a8 Compare November 29, 2017 23:28
@StanleyGoldman StanleyGoldman force-pushed the fixes/branches-view-remove-serialized-texture2d branch from 9cfa797 to 9dc931a Compare November 29, 2017 23:33
Copy link

@drguthals drguthals left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

…ew-rollup' into fixes/branches-view-remove-serialized-texture2d

# Conflicts:
#	src/UnityExtension/Assets/Editor/GitHub.Unity/UI/TreeControl.cs
@StanleyGoldman StanleyGoldman merged commit 9f23c84 into enhancements/branches-view-rollup Nov 30, 2017
@StanleyGoldman StanleyGoldman deleted the fixes/branches-view-remove-serialized-texture2d branch November 30, 2017 00:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants