diff options
| author | Mitch Curtis <mitch.curtis@qt.io> | 2024-09-24 14:44:25 +0800 |
|---|---|---|
| committer | Mitch Curtis <mitch.curtis@qt.io> | 2024-10-03 18:10:04 +0800 |
| commit | 43c2c4e9facf4e0cb134de1a51b8e64c7ac01563 (patch) | |
| tree | cc6c3895ebe732940ad867cf329effd21f95f7d1 /src/qml/jsruntime/qv4arraybuffer.cpp | |
| parent | 57e91102bcadabf3ffcefede27190e1c1272fab4 (diff) | |
ListView: fix countChanged not being emitted in certain cases
This broke after 477c15def834bd49553c00b90f3a2006456ea931.
countChanged would have normally been emitted in the call to
applyModelChanges that we skipped after adding the q->size().isNull()
check.
We can check hasPendingChanges() to know if a count change
is pending (which is also the first thing applyModelChanges checks
before doing its work), and if so, emit countChanged so that
any bindings that are relying on it can function as expected.
Fixes: QTBUG-129165
Change-Id: Ic045f1870b39d192f6880e23daab03fd73a16d58
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
(cherry picked from commit 0a1d8fb26d1e89c82a0d35dd728d7e928a94d197)
Reviewed-by: Tarja Sundqvist <tarja.sundqvist@qt.io>
Diffstat (limited to 'src/qml/jsruntime/qv4arraybuffer.cpp')
0 files changed, 0 insertions, 0 deletions