diff options
author | Richard Moe Gustavsen <richard.gustavsen@qt.io> | 2018-08-10 14:25:22 +0200 |
---|---|---|
committer | Richard Moe Gustavsen <richard.gustavsen@qt.io> | 2018-08-10 15:00:19 +0000 |
commit | 0bd3c9cda678b7e158c6142c1567ac927fde18b9 (patch) | |
tree | 551b1e91b2365c1674f14fdec4bc7a2df7cde4c9 /tests/auto/quick/qquicktableview/data | |
parent | 481ea25bc8aa64e7131edd85a6c871b2a3193f58 (diff) |
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 <mitch.curtis@qt.io>
Diffstat (limited to 'tests/auto/quick/qquicktableview/data')
9 files changed, 0 insertions, 9 deletions
diff --git a/tests/auto/quick/qquicktableview/data/alternatingrowheightcolumnwidth.qml b/tests/auto/quick/qquicktableview/data/alternatingrowheightcolumnwidth.qml index c9fb1031dc..4126dcf405 100644 --- a/tests/auto/quick/qquicktableview/data/alternatingrowheightcolumnwidth.qml +++ b/tests/auto/quick/qquicktableview/data/alternatingrowheightcolumnwidth.qml @@ -55,7 +55,6 @@ Item { anchors.margins: 1 clip: true delegate: tableViewDelegate - cacheBuffer: 0 columnSpacing: 1 rowSpacing: 1 } diff --git a/tests/auto/quick/qquicktableview/data/contentwidthheight.qml b/tests/auto/quick/qquicktableview/data/contentwidthheight.qml index 2c9d64c20e..7beab21009 100644 --- a/tests/auto/quick/qquicktableview/data/contentwidthheight.qml +++ b/tests/auto/quick/qquicktableview/data/contentwidthheight.qml @@ -55,7 +55,6 @@ Item { anchors.margins: 1 clip: true delegate: tableViewDelegate - cacheBuffer: 0 columnSpacing: 1 rowSpacing: 1 leftMargin: 10 diff --git a/tests/auto/quick/qquicktableview/data/countingtableview.qml b/tests/auto/quick/qquicktableview/data/countingtableview.qml index 73fe4a80da..dd15442a01 100644 --- a/tests/auto/quick/qquicktableview/data/countingtableview.qml +++ b/tests/auto/quick/qquicktableview/data/countingtableview.qml @@ -65,7 +65,6 @@ Item { anchors.margins: 1 clip: true delegate: tableViewDelegate - cacheBuffer: 0 } Component { diff --git a/tests/auto/quick/qquicktableview/data/forcelayout.qml b/tests/auto/quick/qquicktableview/data/forcelayout.qml index 118abf51ee..af00d06837 100644 --- a/tests/auto/quick/qquicktableview/data/forcelayout.qml +++ b/tests/auto/quick/qquicktableview/data/forcelayout.qml @@ -54,7 +54,6 @@ Item { height: 400 anchors.margins: 1 delegate: tableViewDelegate - cacheBuffer: 0 columnSpacing: 1 rowSpacing: 1 diff --git a/tests/auto/quick/qquicktableview/data/plaintableview.qml b/tests/auto/quick/qquicktableview/data/plaintableview.qml index 21373e6732..730d779896 100644 --- a/tests/auto/quick/qquicktableview/data/plaintableview.qml +++ b/tests/auto/quick/qquicktableview/data/plaintableview.qml @@ -58,7 +58,6 @@ Item { anchors.margins: 1 clip: true delegate: tableViewDelegate - cacheBuffer: 0 columnSpacing: 1 rowSpacing: 1 } diff --git a/tests/auto/quick/qquicktableview/data/qqmllistpropertymodel.qml b/tests/auto/quick/qquicktableview/data/qqmllistpropertymodel.qml index c866af8526..fb0b7516bf 100644 --- a/tests/auto/quick/qquicktableview/data/qqmllistpropertymodel.qml +++ b/tests/auto/quick/qquicktableview/data/qqmllistpropertymodel.qml @@ -55,7 +55,6 @@ Item { anchors.margins: 1 clip: true delegate: tableViewDelegate - cacheBuffer: 0 } Item { diff --git a/tests/auto/quick/qquicktableview/data/tableviewimplicitsize.qml b/tests/auto/quick/qquicktableview/data/tableviewimplicitsize.qml index 65e4d0861c..13b866f8d2 100644 --- a/tests/auto/quick/qquicktableview/data/tableviewimplicitsize.qml +++ b/tests/auto/quick/qquicktableview/data/tableviewimplicitsize.qml @@ -43,7 +43,6 @@ Item { anchors.margins: 1 clip: true delegate: tableViewDelegate - cacheBuffer: 0 columnSpacing: 1 rowSpacing: 1 } diff --git a/tests/auto/quick/qquicktableview/data/usefaultyrowcolumnprovider.qml b/tests/auto/quick/qquicktableview/data/usefaultyrowcolumnprovider.qml index e7a83a5d8b..d4d218f5db 100644 --- a/tests/auto/quick/qquicktableview/data/usefaultyrowcolumnprovider.qml +++ b/tests/auto/quick/qquicktableview/data/usefaultyrowcolumnprovider.qml @@ -55,7 +55,6 @@ Item { anchors.margins: 1 clip: true delegate: tableViewDelegate - cacheBuffer: 0 columnSpacing: 1 rowSpacing: 1 columnWidthProvider: function(column) { } diff --git a/tests/auto/quick/qquicktableview/data/userowcolumnprovider.qml b/tests/auto/quick/qquicktableview/data/userowcolumnprovider.qml index e781eb8a7c..bc219e0d78 100644 --- a/tests/auto/quick/qquicktableview/data/userowcolumnprovider.qml +++ b/tests/auto/quick/qquicktableview/data/userowcolumnprovider.qml @@ -55,7 +55,6 @@ Item { anchors.margins: 1 clip: true delegate: tableViewDelegate - cacheBuffer: 0 columnSpacing: 1 rowSpacing: 1 columnWidthProvider: function(column) { return column + 10 } |