From 3859871e717e74bbe6659c7dd47af2f9787cccb3 Mon Sep 17 00:00:00 2001 From: hacklex Date: Thu, 8 Oct 2015 04:58:11 +0300 Subject: [PATCH] fixed misuse of OrderBy To order further inside equivalent groups, one should use ThenBy, as subsequent OrderBy calls are not guaranteed to preserve the original elements order. --- src/Perspex.Diagnostics/ViewModels/ControlDetailsViewModel.cs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/Perspex.Diagnostics/ViewModels/ControlDetailsViewModel.cs b/src/Perspex.Diagnostics/ViewModels/ControlDetailsViewModel.cs index 462cf3a1f7..8a45ec8244 100644 --- a/src/Perspex.Diagnostics/ViewModels/ControlDetailsViewModel.cs +++ b/src/Perspex.Diagnostics/ViewModels/ControlDetailsViewModel.cs @@ -16,8 +16,8 @@ namespace Perspex.Diagnostics.ViewModels { Properties = control.GetRegisteredProperties() .Select(x => new PropertyDetails(control, x)) - .OrderBy(x => x.Name) - .OrderBy(x => x.IsAttached); + .OrderBy(x => x.IsAttached) + .ThenBy(x => x.Name); } }