Ohhh, okay, thanks anyway😢

Right, but they can be cherry-picked to the Plasma 5.19. In truth, I'm tired of editing these files after every Plasma update☹️

In D29818#672942, @ngraham wrote:

was landed on master, not 5.19

Can it be moved to Plasma 5.19?

What about something like this?

You're right. Fixed.

Currently, the Notifications applet closes itself when the last element (notification) is removed (or if all elements are removed by pressing "Clear History" button in the top-right corner). But also it closes itself, even if the applet is pinned, and I guess that this behaviour isn't correct. D29651 fixes this issue.

Fixed.

I believe that there is a better solution, but I did not come up with it.

Is it normal that I can close pinned applet by pressing ESC?

Thanks!

Rebase on master

The previous patch updates the pinned state only for the active applet. For instance, if you open the Networks applet and pin it, then switch to the Clipboard applet and click on the history item, then the Clipboard applet closes. This is because we need to use binding instead of assignment.

Unfortunately, the previous patch doesn't solve the problem completely, but this one should. Can you check it and if everything works well, then replace the previous one with it?

Rebase on master, second edition. Can you try this patch?

I have no idea how to test code from master branch

Rebase on master

In Plasma/5.18 branch there was a Containment.onAppletAdded handler that took the applet as a parameter, but I can't find this handler in master branch. Unfortunately, I don’t have enough knowledge of Plasma API to know if I can add this handler manually.

Well then I have no idea how to fix it

Can't be rebased on master

There's no easy way to do this. I can use something like this:

return Number.fromLocaleString(locale, text.match(new RegExp("[" + locale.groupSeparator + "\\d]+")))

But currently, the number of rows is a positive integer in the range [1..20], so we don't need to take into account the locale when parsing it.

Rebase to master

Use hideOnWindowDeactivate property instead of configuration.pin

In D28408#637476, @davidedmundson wrote:

As a question, rather than a researched proposal, is it better to check for the pin in?

systemtray / PlasmoidItem.qml

Connections {
    target: applet
    onExpandedChanged: {

We should be able to hide the pinned applet by clicking in the system tray (at least this can be done now).

Put the dateEquals function outside the onAgendaUpdated handler, so it can be used elsewhere.

Make a function that compares dates inline.

Thank you, guys!

Fix hover state for pinned launchers

Use ES6 spread syntax instead of Array.concat()

Create a new function in TasksTools to make the code more readable.