Follow-up Comment #3, bug #24683 (project wesnoth):

> The text needs to be tweaked in any case, as currently it doesn't mention
:idle at all

Hmm yes that shouldn't bee too hard.

>  but on top of that :control should still be able to handle it too 

Shouldn't be too hard eigher, the code that does the :control is here:
https://github.com/wesnoth/wesnoth/blob/ca382018b7a9b5b744f0669bf1442a30ead63c15/src/menu_events.cpp#L2594
and the code that handles :idle is here:
https://github.com/wesnoth/wesnoth/blob/ca382018b7a9b5b744f0669bf1442a30ead63c15/src/menu_events.cpp#L2542

Mostlikeley it is enough to juste copy some code from the :idle implementation
to the :control implementation

> and :give_control too, if it currently can't


Didn't even know there was such a thing, i doubt that it currently can. I am
also unsure whether it actually makes sense to have such a dialog, specially
since it's only avaialbe via commandmode and people who use commandmode will
most likeley just use :control.

I think it'd be better if there were dropboxes to change the sides controller
in the side overview dialog (openes with s hotkey)

    _______________________________________________________

Reply to this item at:

  <http://gna.org/bugs/?24683>

_______________________________________________
  Nachricht gesendet von/durch Gna!
  http://gna.org/


_______________________________________________
Wesnoth-bugs mailing list
Wesnoth-bugs@gna.org
https://mail.gna.org/listinfo/wesnoth-bugs

Reply via email to