diff options
| author | Jens Bache-Wiig <jens.bache-wiig@digia.com> | 2014-02-20 17:10:05 +0100 |
|---|---|---|
| committer | The Qt Project <gerrit-noreply@qt-project.org> | 2014-02-20 17:54:25 +0100 |
| commit | c29877850f44d0fc4e2164295beb2bded5f4aa30 (patch) | |
| tree | b01bb638ab329377c96ee1d562195cfab01210fc /src/controls/TableView.qml | |
| parent | 73ac28cba6d9344b866788443a5ff753d15f6606 (diff) | |
Speed up binding comparisons for TableView
Change-Id: I0bf4db5ff91fea4927e32127ba22094d57caeaf5
Reviewed-by: Gabriel de Dietrich <gabriel.dedietrich@digia.com>
Diffstat (limited to 'src/controls/TableView.qml')
| -rw-r--r-- | src/controls/TableView.qml | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/src/controls/TableView.qml b/src/controls/TableView.qml index f72b429ad..4f0f7fbb8 100644 --- a/src/controls/TableView.qml +++ b/src/controls/TableView.qml @@ -610,7 +610,7 @@ ScrollView { if (pressed && !Settings.hasTouchScreen) { var newIndex = Math.max(0, listView.indexAt(0, mouseY + listView.contentY)) - if (newIndex >= 0 && newIndex != currentRow) { + if (newIndex >= 0 && newIndex !== currentRow) { listView.currentIndex = newIndex; if (selectionMode === SelectionMode.SingleSelection) { selection.__selectOne(newIndex) @@ -799,8 +799,8 @@ ScrollView { // Reassign row-specific bindings rowItem.rowIndex = model.index; - rowItem.alternate = Qt.binding( function() { return alternatingRowColors && model.index % 2 == 1 }); - rowItem.itemModelData = Qt.binding( function() { return typeof modelData == "undefined" ? null : modelData }); + rowItem.alternate = Qt.binding( function() { return alternatingRowColors && model.index % 2 === 1 }); + rowItem.itemModelData = Qt.binding( function() { return typeof modelData === "undefined" ? null : modelData }); rowItem.itemModel = Qt.binding( function() { return model }); rowItem.parent = rowItemContainer; rowItem.visible = true; @@ -931,7 +931,7 @@ ScrollView { delegate: Item { z:-index - width: columnCount == 1 ? viewport.width + __verticalScrollBar.width : modelData.width + width: columnCount === 1 ? viewport.width + __verticalScrollBar.width : modelData.width implicitWidth: headerStyle.implicitWidth visible: modelData.visible height: headerVisible ? headerStyle.height : 0 @@ -953,7 +953,7 @@ ScrollView { id: targetmark width: parent.width height:parent.height - opacity: (index == repeater.targetIndex && repeater.targetIndex != repeater.dragIndex) ? 0.5 : 0 + opacity: (index === repeater.targetIndex && repeater.targetIndex !== repeater.dragIndex) ? 0.5 : 0 Behavior on opacity { NumberAnimation{duration:160}} color: palette.highlight visible: modelData.movable @@ -965,8 +965,8 @@ ScrollView { hoverEnabled: true anchors.fill: parent onClicked: { - if (sortIndicatorColumn == index) - sortIndicatorOrder = sortIndicatorOrder == Qt.AscendingOrder ? Qt.DescendingOrder : Qt.AscendingOrder + if (sortIndicatorColumn === index) + sortIndicatorOrder = sortIndicatorOrder === Qt.AscendingOrder ? Qt.DescendingOrder : Qt.AscendingOrder sortIndicatorColumn = index } // Here we handle moving header sections @@ -989,11 +989,11 @@ ScrollView { } onReleased: { - if (repeater.targetIndex >= 0 && repeater.targetIndex != index ) { + if (repeater.targetIndex >= 0 && repeater.targetIndex !== index ) { var targetColumn = columnModel.get(repeater.targetIndex).columnItem if (targetColumn.movable) { columnModel.move(index, repeater.targetIndex, 1) - if (sortIndicatorColumn == index) + if (sortIndicatorColumn === index) sortIndicatorColumn = repeater.targetIndex } } |