RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-24 23:17:44 +00:00
Code Issues Activity Actions

LibGUI: Remove redundant scroll_into_view() calls in TableView

Browse source 
The calls to set_cursor() already take care of scrolling the new cursor
into view if needed.
This commit is contained in:
Andreas Kling 2020-10-30 23:49:24 +01:00
parent 563d3c8055
commit 66a8e151fb
1 changed files with 0 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

2
Libraries/LibGUI/TableView.cpp
View file

@ -209,13 +209,11 @@ void TableView::move_cursor(CursorMovement movement, SelectionUpdate selection_u
case CursorMovement::Home: { case CursorMovement::Home: {
auto index = model.index(0, 0); auto index = model.index(0, 0);
set_cursor(index, selection_update); set_cursor(index, selection_update);
scroll_into_view(index, false, true);
break; break;
} }
case CursorMovement::End: { case CursorMovement::End: {
auto index = model.index(model.row_count() - 1, 0); auto index = model.index(model.row_count() - 1, 0);
set_cursor(index, selection_update); set_cursor(index, selection_update);
scroll_into_view(index, false, true);
break; break;
} }
case CursorMovement::PageUp: { case CursorMovement::PageUp: {