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
Debugconsole and input history improvements #5944
Conversation
* fix debug console menu entry tooltip
Code LGTM, tested and working. |
Yes, of course, that was my intention too. |
@Noordfrees Regarding #1668, IMO this PR already does what's reasonable. Any more separation would require jumping through hoops back and forth between |
…ton is used in any window
* re-add translators comment mistakenly deleted in widelands#6059
I believe the last commit solves the remaining part of #2066. |
OK, this is big enough now, and the important parts are in place. No more new features for this PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested this again. I was a bit surprised that saving the script console history to file had to be explicitly enabled, IMO we could remove the option and do that always.
I also don't see the value of letting the user configure the history size, why not just hardcode this with a size of 64 of both (or even more, maybe 256, this shouldn't take up that much memory?).
Other than that, working well, great feature :) Code LGTM, one nit.
OK, though I was afraid that it might be a privacy issue for some, like the chat history.
OK. Then I'll revert it to 64. If somebody needs more, then they should probably create helper scripts that they can use with |
(sorry for the incomplete pushes) There's one more thing I'm thinking about: should the histories be saved regularly to keep more of them on crashes? (the script console in particular is supposed to be used in conditions where crashes are more likely) One obvious place for saving would be after each autosave, but it would bring in a new dependency on a file from Or I could keep a counter of new entries, and save e.g. on every 10th, but that's probably too much overhead. |
Mostly i get Lua errors and no complete crashes when working with the lua console. So i am not sure if a regular file save is needed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks :)
Thanks for the reviews. :) |
Type of change
New feature / Refactoring
Issue(s) closed
Fixes #5793
Fixes #1668
Fixes #2066
New behavior
Possible regressions
EditBox and chat/script console behaviour and history handling
Additional context
Postponed:
dofile()
Scrapped: