diff options
| -rw-r--r-- | src/quick/items/qquicktableview.cpp | 4 | ||||
| -rw-r--r-- | tests/auto/quick/qquicktableview/tst_qquicktableview.cpp | 15 |
2 files changed, 13 insertions, 6 deletions
diff --git a/src/quick/items/qquicktableview.cpp b/src/quick/items/qquicktableview.cpp index d830f5a66d..29c6f91216 100644 --- a/src/quick/items/qquicktableview.cpp +++ b/src/quick/items/qquicktableview.cpp @@ -2972,7 +2972,7 @@ qreal QQuickTableView::rowSpacing() const void QQuickTableView::setRowSpacing(qreal spacing) { Q_D(QQuickTableView); - if (qt_is_nan(spacing) || !qt_is_finite(spacing) || spacing < 0) + if (qt_is_nan(spacing) || !qt_is_finite(spacing)) return; if (qFuzzyCompare(d->cellSpacing.height(), spacing)) return; @@ -2990,7 +2990,7 @@ qreal QQuickTableView::columnSpacing() const void QQuickTableView::setColumnSpacing(qreal spacing) { Q_D(QQuickTableView); - if (qt_is_nan(spacing) || !qt_is_finite(spacing) || spacing < 0) + if (qt_is_nan(spacing) || !qt_is_finite(spacing)) return; if (qFuzzyCompare(d->cellSpacing.width(), spacing)) return; diff --git a/tests/auto/quick/qquicktableview/tst_qquicktableview.cpp b/tests/auto/quick/qquicktableview/tst_qquicktableview.cpp index 6e13044390..26360c3443 100644 --- a/tests/auto/quick/qquicktableview/tst_qquicktableview.cpp +++ b/tests/auto/quick/qquicktableview/tst_qquicktableview.cpp @@ -1326,15 +1326,22 @@ void tst_QQuickTableView::checkSpacingValues() QCOMPARE(tableView->contentWidth(), expectedInitialContentWidth); QCOMPARE(tableView->contentHeight(), expectedInitialContentHeight); - // Invalid assignments (should ignore) + // Negative spacing is allowed, and can be used to eliminate double edges + // in the grid if the delegate is a rectangle with a border. tableView->setRowSpacing(-1); - tableView->setColumnSpacing(-5); + tableView->setColumnSpacing(-1); + QCOMPARE(tableView->rowSpacing(), -1); + QCOMPARE(tableView->columnSpacing(), -1); + + tableView->setRowSpacing(10); + tableView->setColumnSpacing(10); + // Invalid assignments (should ignore) tableView->setRowSpacing(INFINITY); tableView->setColumnSpacing(INFINITY); tableView->setRowSpacing(NAN); tableView->setColumnSpacing(NAN); - QCOMPARE(tableView->rowSpacing(), 42); - QCOMPARE(tableView->columnSpacing(), 12); + QCOMPARE(tableView->rowSpacing(), 10); + QCOMPARE(tableView->columnSpacing(), 10); } void tst_QQuickTableView::checkDelegateParent() |