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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas
                   |ma/discover/commit/4c7c4036 |ma/discover/commit/e9f3a33f
                   |0ecc639e140629e4adf55b4b014 |eb24093f5f40e565a16b20e8133
                   |e6460                       |4b0ee

--- Comment #10 from Nate Graham <n...@kde.org> ---
Git commit e9f3a33feb24093f5f40e565a16b20e81334b0ee by Nate Graham, on behalf
of Aleix Pol.
Committed on 18/10/2021 at 15:43.
Pushed by ngraham into branch 'Plasma/5.23'.

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.


(cherry picked from commit 4c7c40360ecc639e140629e4adf55b4b014e6460)

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

https://invent.kde.org/plasma/discover/commit/e9f3a33feb24093f5f40e565a16b20e81334b0ee

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

Reply via email to