-
Bug
-
Resolution: Done
-
P2: Important
-
6.2.3, 6.3.0
-
None
-
a08eb13e7a (qt/qtdeclarative/dev) a08eb13e7a (qt/tqtc-qtdeclarative/dev) b6b07fefe1 (qt/qtdeclarative/6.3) b6b07fefe1 (qt/tqtc-qtdeclarative/6.3) ad892e424d (qt/tqtc-qtdeclarative/6.2) f6c7ee264b (qt/qtdeclarative/6.4) b6b07fefe1 (qt/qtdeclarative/6.3.2)
In `QQuickPaddedRectangle::setPadding`, it emits the padding changed signal for each edge that doesn't have a specific value set for that edge, except for the top edge. There is a typo in the evaluation, where it is missing the `!` like the other edges.
void QQuickPaddedRectangle::setPadding(qreal padding)
{
if (!qFuzzyCompare(m_padding, padding)) {
m_padding = padding;
update();
emit paddingChanged();
if (m_hasTopPadding) <=== missing `!`
emit topPaddingChanged();
if (!m_hasLeftPadding)
emit leftPaddingChanged();
if (!m_hasRightPadding)
emit rightPaddingChanged();
if (!m_hasBottomPadding)
emit bottomPaddingChanged();
}
}
| For Gerrit Dashboard: QTBUG-104865 | ||||||
|---|---|---|---|---|---|---|
| # | Subject | Branch | Project | Status | CR | V |
| 421254,3 | Fix emission of QQuickPaddedRectangle::topPaddingChanged | dev | qt/qtdeclarative | Status: MERGED | +2 | 0 |
| 421736,2 | Fix emission of QQuickPaddedRectangle::topPaddingChanged | 6.4 | qt/qtdeclarative | Status: MERGED | +2 | 0 |
| 421737,2 | Fix emission of QQuickPaddedRectangle::topPaddingChanged | 6.3 | qt/qtdeclarative | Status: MERGED | +2 | 0 |
| 421743,2 | Fix emission of QQuickPaddedRectangle::topPaddingChanged | tqtc/lts-6.2 | qt/tqtc-qtdeclarative | Status: MERGED | +2 | 0 |