-
-
Notifications
You must be signed in to change notification settings - Fork 991
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
Expose unit pick dialog to Lua as a simple API call #8829
Comments
If we're consolidating those dialogs, I'd suggest the following two things :
|
I'm not entirely sure what you mean by this, but I think it should be obvious that opening a modal dialog in response to a mouse hover is a very bad idea. Maybe that's not what you meant though. Also, I'm not quite sure but it sounds like your point 2 is likely out of scope for this issue. |
If thats what you want to do, you'd need to first add a way for lua to hook in before the original dialog is shown (to be replaced with your custom one), which is imo the more complicated part |
What's complicated about that, exactly…? Just find the place that currently calls the dialog and do something else instead, right? Not entirely sure that's actually necessary though. |
That's right. Is it possible with a modeless dialog (assuming we can get it working)?
Could be, perhaps there should be another issue on consolidating the three dialogs the you mentioned? |
Maybe, but that sounds like a strange way to do it… |
This refers to the recruit, recall, and debug unit create dialogs – I believe it would make sense to merge them into single dialog that supports various options to tweak it into being identical to any of the current dialogs. If it turns out I'm wrong, however, the main thing would be to expose the recruit dialog, so that SotA can handle its corpse recruits in a more reasonable manner (and automatically get all the features of the recruit dialog, like filtering).
The text was updated successfully, but these errors were encountered: