From 66a8e151fb88f410e0ecc706a6444ba35669f0e7 Mon Sep 17 00:00:00 2001 From: Andreas Kling Date: Fri, 30 Oct 2020 23:49:24 +0100 Subject: [PATCH] LibGUI: Remove redundant scroll_into_view() calls in TableView The calls to set_cursor() already take care of scrolling the new cursor into view if needed. --- Libraries/LibGUI/TableView.cpp | 2 -- 1 file changed, 2 deletions(-) diff --git a/Libraries/LibGUI/TableView.cpp b/Libraries/LibGUI/TableView.cpp index ff0b3459b8..f65ace1925 100644 --- a/Libraries/LibGUI/TableView.cpp +++ b/Libraries/LibGUI/TableView.cpp @@ -209,13 +209,11 @@ void TableView::move_cursor(CursorMovement movement, SelectionUpdate selection_u case CursorMovement::Home: { auto index = model.index(0, 0); set_cursor(index, selection_update); - scroll_into_view(index, false, true); break; } case CursorMovement::End: { auto index = model.index(model.row_count() - 1, 0); set_cursor(index, selection_update); - scroll_into_view(index, false, true); break; } case CursorMovement::PageUp: {