ui: Only shows port number if protocol allows changing it

joaofl · Dec 30, 2024 · fd6ad70 · fd6ad70
1 parent 81d3500
commit fd6ad70
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/ui/window.rs b/src/ui/window.rs
@@ -99,7 +99,10 @@ impl eframe::App for UI {
                     ui.group(|ui| {
                         ui.add(Label::new(format!("{}", p.protocol.to_string())));
 
-                        if p.protocol != Protocol::Dhcp {
+
+                        // Some protocols do not allow changing ports (and may be set to 0)
+                        // so we only show the port field if it is not 0
+                        if p.port != 0 {
                             ui.add(DragValue::new(&mut p.port).range(1..=50000));
                         }