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).
This commit is contained in:
Clement Tsang 2021-02-19 01:22:19 -05:00 committed by GitHub
parent e6c9187928
commit f2975c3a7c
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 4 deletions

View File

@ -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.

View File

@ -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();
}
}
}