yasirkula / UnityIngameDebugConsole

A uGUI based console to see debug messages and execute commands during gameplay in Unity
MIT License
2.11k stars 221 forks source link

Start hidden #2

Closed lucasmontec closed 7 years ago

lucasmontec commented 7 years ago

Your console is a great debugging tool! It would be great if there was a simple option to start hidden or an alternative prefab to use in this manner.

yasirkula commented 7 years ago

Good thinking. Will implement an exposed variable for it shortly.

yasirkula commented 7 years ago

Can you check the latest version? There is a new variable called "Launch In Popup Mode" in DebugLogCanvas prefab.

lucasmontec commented 7 years ago

I'm getting a Null Reference Exception on the method call for OnSetVisible() because OnEnable for DebugLogManager is called before the awake for the DebugLogPopup. To fix this I moved the entire 'if (launchInPopupMode)' to Start on DebugLogManager class. This solved the issue.

tl;dr canvasGroup is not set when onEnable is called in DebugLogManager, this results in a NRE.

yasirkula commented 7 years ago

I did not encounter that error while testing miself, strangely, but I will apply your solution to the main branch soon, nevertheless.

lucasmontec commented 7 years ago

Probably an error with unet.I have also changed your LogFilter name since unet uses an identical name.

Em 11/10/2016 02:21, "Süleyman Yasir KULA" notifications@github.com escreveu:

I did not encounter that error while testing miself, strangely, but I will apply your solution to the main branch soon.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/yasirkula/UnityIngameDebugConsole/issues/2#issuecomment-252816473, or mute the thread https://github.com/notifications/unsubscribe-auth/ABUb_SKqK2_IKIDS6nlUC8ziEfeSUbD2ks5qyxytgaJpZM4KSGpi .

yasirkula commented 7 years ago

I have moved the launchInPopupMode code to Start function and changed the name of LogFilter to DebugLogFilter. Thanks for all the info.

lucasmontec commented 7 years ago

Thanks for the awesome asset!

Em 11/10/2016 07:47, "Süleyman Yasir KULA" notifications@github.com escreveu:

I have moved the launchInPopupMode code to Start function and changed the name of LogFilter to DebugLogFilter. Thanks for all the info.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/yasirkula/UnityIngameDebugConsole/issues/2#issuecomment-252878580, or mute the thread https://github.com/notifications/unsubscribe-auth/ABUb_Xyvef99dSj9od9q6xV4H2Va00Qnks5qy2kGgaJpZM4KSGpi .