[PATCH datacenter-manager v5 09/10] ui: pve: node selector: sort memory columns by usage not by memory size

[Dominik Csapak <d.csapak@proxmox.com>]

We only show the usage, so it makes sense to sort by that instead of the
raw memory size value.

Use total_cmp for floats, which sorts (+/-)NaN/Infinity separate from
finite values.

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---
 ui/src/widget/pve_node_selector.rs | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/ui/src/widget/pve_node_selector.rs b/ui/src/widget/pve_node_selector.rs
index d471780e..5882f532 100644
--- a/ui/src/widget/pve_node_selector.rs
+++ b/ui/src/widget/pve_node_selector.rs
@@ -261,7 +261,11 @@ fn columns(show_memory: bool) -> Rc<Vec<DataTableHeader<PveNodeResource>>> {
                     }
                     html! {format!("{:.2}%", 100.0 * entry.mem as f64 / entry.maxmem as f64)}
                 })
-                .sorter(|a: &PveNodeResource, b: &PveNodeResource| a.mem.cmp(&b.mem))
+                .sorter(|a: &PveNodeResource, b: &PveNodeResource| {
+                    let a = (a.mem as f64) / (a.maxmem as f64);
+                    let b = (b.mem as f64) / (b.maxmem as f64);
+                    a.total_cmp(&b)
+                })
                 .into(),
         );
     }
-- 
2.47.3