From 54363194069ee74e972e6559998b68ae9599e17a Mon Sep 17 00:00:00 2001 From: Doruk Turak Date: Sun, 27 Dec 2015 16:32:32 +0200 Subject: [PATCH] Observer Shroud Options listing polish #10263 Pretty self explanatory: Now "All Players" and "Disable Shroud" options come before --- .../Logic/Ingame/ObserverShroudSelectorLogic.cs | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/OpenRA.Mods.Common/Widgets/Logic/Ingame/ObserverShroudSelectorLogic.cs b/OpenRA.Mods.Common/Widgets/Logic/Ingame/ObserverShroudSelectorLogic.cs index f2dd3d40e7..eec6ab49e5 100644 --- a/OpenRA.Mods.Common/Widgets/Logic/Ingame/ObserverShroudSelectorLogic.cs +++ b/OpenRA.Mods.Common/Widgets/Logic/Ingame/ObserverShroudSelectorLogic.cs @@ -62,6 +62,11 @@ namespace OpenRA.Mods.Common.Widgets.Logic var groups = new Dictionary>(); + combined = new CameraOption(this, world, "All Players", world.Players.First(p => p.InternalName == "Everyone")); + disableShroud = new CameraOption(this, world, "Disable Shroud", null); + if (!limitViews) + groups.Add("Other", new List() { combined, disableShroud }); + teams = world.Players.Where(p => !p.NonCombatant && p.Playable) .Select(p => new CameraOption(this, p)) .GroupBy(p => (world.LobbyInfo.ClientWithIndex(p.Player.ClientIndex) ?? new Session.Client()).Team) @@ -74,11 +79,6 @@ namespace OpenRA.Mods.Common.Widgets.Logic groups.Add(label, t); } - combined = new CameraOption(this, world, "All Players", world.Players.First(p => p.InternalName == "Everyone")); - disableShroud = new CameraOption(this, world, "Disable Shroud", null); - if (!limitViews) - groups.Add("Other", new List() { combined, disableShroud }); - var shroudSelector = widget.Get("SHROUD_SELECTOR"); shroudSelector.OnMouseDown = _ => {