https://bugs.kde.org/show_bug.cgi?id=406295

Nate Graham <n...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED
      Latest Commit|                            |https://invent.kde.org/plas
                   |                            |ma/discover/commit/4c7c4036
                   |                            |0ecc639e140629e4adf55b4b014
                   |                            |e6460

--- Comment #9 from Nate Graham <n...@kde.org> ---
Git commit 4c7c40360ecc639e140629e4adf55b4b014e6460 by Nate Graham, on behalf
of Aleix Pol.
Committed on 18/10/2021 at 15:40.
Pushed by ngraham into branch 'master'.

SourcesPage: Address how we show the checked state

Do not use data() as it won't refresh as the model changes, we never
actually set the checked property anyway. Just call set and let the
checked property be updated by the model.

Alternatively we had 2 actors updating the value and it didn't work
well.

M  +1    -1    discover/qml/SourcesPage.qml
M  +1    -0    libdiscover/resources/SourcesModel.cpp

https://invent.kde.org/plasma/discover/commit/4c7c40360ecc639e140629e4adf55b4b014e6460

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to