Ark2000 / PankuConsole

Feature-packed real-time debugging toolkit for Godot Engine.
https://k2kra.xyz/PankuConsole/
MIT License
997 stars 38 forks source link

Remove logger? #149

Open Ark2000 opened 9 months ago

Ark2000 commented 9 months ago

the native logger module should be a part of dev console.

TODO:

Ark2000 commented 9 months ago

A new text area node composed of two RichTextLabel nodes will be introduced to cap lines of content.

it is like, an infinite scroll game.

specifically, the first buffer will be cleared and sent to the last when the last buffer is full.

with buffers, we can constanly output lots of stuff while keeping a smooth experience.

toolness commented 2 months ago

Hi! I am really liking Panku Console but the fact that the logger defaults to "Always show" is making things a bit frustrating:

  1. I didn't like this behavior because I wanted my alpha testers to be able to access the console if needed, so I didn't excise it from my build. This behavior is obviously not great for an alpha tester because it clobbers their gameplay with all the logging junk.

  2. I did set a custom Panku config that set the default to "Show if shell visible", BUT I found out after distributing my build that the config file wasn't actually included in the export, so it couldn't actually find it, so it defaulted to "always show" and wrote the configuration to the user's user data dir.

  3. Now I have to give my users instructions on how to go through a bunch of menu options to change the default to "Show if shell visible". (Side note: it would be really cool if I could tell them to download a new build and click an easily-accessible "reset to defaults" button or run a similar command to fix things, as having to write and follow instructions on going through menu hierarchies isn't fun.)

Anyways, this is all to say: maybe I'm in the minority, but I feel like defaulting the logger to "Show if shell visible" would be a much better default.

Regardless, thanks for making this great tool, overall I'm very happy with it!

worron commented 1 month ago

I did set a custom Panku config that set the default to "Show if shell visible", BUT I found out after distributing my build that the config file wasn't actually included in the export

This is confusing part, your custom config file should work in exported build if everything configured right. Please take a look at #175 and #179 for more details.