I'm sorry. But how did you make "Unit group - Pick every units in..." is that an action?
ok thank you very much
You do not even need to store players, there is a Convert Player Index To Player function and another Get Player Color function you can use unless you change them.
What does he mean by that?
Actions
For each (Integer A) from 1 to i, do (Actions)
Loop - Actions
If (All Conditions are True) then do (Then Actions) else do (Else Actions)
If - Conditions
(Entered chat string) Equal to ColorString[(Integer A)]
Then - Actions
Unit Group - Pick every unit in (Units currently selected by (Triggering player)) and do (Actions)
Loop - Actions
Unit - Change color of (Picked unit) to (Color of (Player((Integer A))))
Else - Actions
Surely. This is what TC wanted, only the selected units shall change color, not the player or his/her other units. It does leak a group but it's marginally, the event is not frequent and GUI users do not get it anyway.