From 0bd3c9cda678b7e158c6142c1567ac927fde18b9 Mon Sep 17 00:00:00 2001 From: Richard Moe Gustavsen Date: Fri, 10 Aug 2018 14:25:22 +0200 Subject: QQuickTableView: remove cacheBuffer from the public API As discussed during API review, remove cacheBuffer from the public API. The cache buffer was a feature inherited from ListView to avoid loading a lot of items (and affect performance) when the user started to flick. But now that TableView has support for reusing items, the point of the cache buffer is more or less gone. At least we choose to remove it from the public API until we have better understanding if this is really needed. Note that the cacheBuffer still plays a small role internally, so we don't remove it from the implementation. We want to preload an extra row and column for reuse at start-up, since you often cannot reuse the first row and column during the first flick (they will still be visible on the screen). Change-Id: Ie62835a04ac29a84c9a76151b73fe2f75d9ae844 Reviewed-by: Mitch Curtis --- tests/manual/tableview/abstracttablemodel/main.qml | 1 - 1 file changed, 1 deletion(-) (limited to 'tests/manual') diff --git a/tests/manual/tableview/abstracttablemodel/main.qml b/tests/manual/tableview/abstracttablemodel/main.qml index 52541657df..a9b8da94c6 100644 --- a/tests/manual/tableview/abstracttablemodel/main.qml +++ b/tests/manual/tableview/abstracttablemodel/main.qml @@ -68,7 +68,6 @@ Window { model: tableModel delegate: tableViewDelegate - cacheBuffer: 500 columnSpacing: 1 rowSpacing: 1 } -- cgit v1.2.3