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

ratijas <m...@ratijas.tk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
      Latest Commit|                            |https://invent.kde.org/plas
                   |                            |ma/discover/-/commit/576e46
                   |                            |0ade383da789727066929d14baa
                   |                            |248aa02
             Status|ASSIGNED                    |RESOLVED

--- Comment #5 from ratijas <m...@ratijas.tk> ---
Git commit 576e460ade383da789727066929d14baa248aa02 by ivan tkachenko.
Committed on 24/11/2023 at 18:25.
Pushed by ratijas into branch 'master'.

Refactor roughCount into an atomic QGadget, fix usage of i18np

i18np's first argument should be a valid number. Meanwhile roughCount
was a string. Instead we should pass the original number as the
plural-deciding argument, while using the formatted string only for the
actual substitution.

It is only natural to factor out the counter into a gadget class of its
own, since all of its properties (and their semantics like the
`roughCount.length > 0` special case) change atomically and thus do not
require separate connections.

Also, let's store the Timer inline, instead of allocating dynamically.

M  +1    -0    discover/DiscoverDeclarativePlugin.cpp
M  +7    -8    discover/qml/ApplicationsListPage.qml
M  +34   -13   libdiscover/resources/ResourcesProxyModel.cpp
M  +27   -4    libdiscover/resources/ResourcesProxyModel.h

https://invent.kde.org/plasma/discover/-/commit/576e460ade383da789727066929d14baa248aa02

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

Reply via email to