change: Don't jump to top when using sort shortcuts (#418)

For consistency, we now don't automatically jump to the top of the list when using a sort shortcut. This behaviour already occurred with the sort menu and sorting by mouse clicks, so this is just now more consistent (and IMO less annoying, you can also always jump to the top via gg).

2 files changed, 2 insertions, 4 deletions

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 0030c1a1..3c04c584 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -31,6 +31,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 - [#391](https://github.com/ClementTsang/bottom/pull/391): Show degree symbol on Celsius and Fahrenheit.
 
+- [#418](https://github.com/ClementTsang/bottom/pull/418): Removed automatically jumping to the top of the list for process sort shortcuts. The standard behaviour is to now stay in place.
+
 ## Bug Fixes
 
 - [#416](https://github.com/ClementTsang/bottom/pull/416): Fixes grouped vs ungrouped modes in the processes widget having inconsistent spacing.
diff --git a/src/app.rs b/src/app.rs
index 019720f0..ba06a28a 100644
--- a/src/app.rs
+++ b/src/app.rs
@@ -1482,7 +1482,6 @@ impl App {
                             .set_to_sorted_index_from_type(&processes::ProcessSorting::CpuPercent);
                         proc_widget_state.update_sorting_with_columns();
                         self.proc_state.force_update = Some(self.current_widget.widget_id);
-                        self.skip_to_first();
                     }
                 }
             }
@@ -1504,7 +1503,6 @@ impl App {
                         );
                         proc_widget_state.update_sorting_with_columns();
                         self.proc_state.force_update = Some(self.current_widget.widget_id);
-                        self.skip_to_first();
                     }
                 }
             }
@@ -1521,7 +1519,6 @@ impl App {
                                 .set_to_sorted_index_from_type(&processes::ProcessSorting::Pid);
                             proc_widget_state.update_sorting_with_columns();
                             self.proc_state.force_update = Some(self.current_widget.widget_id);
-                            self.skip_to_first();
                         }
                     }
                 }
@@ -1569,7 +1566,6 @@ impl App {
                         );
                         proc_widget_state.update_sorting_with_columns();
                         self.proc_state.force_update = Some(self.current_widget.widget_id);
-                        self.skip_to_first();
                     }
                 }
             }