- User Since
- Sep 18 2018, 7:19 PM (48 w, 1 d)
in that case I should rather leave this diff. I agree this would be messy for the history.
Tue, Aug 20
changed line endings to Unix style
removed actualSize in favour of always passing an icon of size 1024x1024 to SnoreToast
had some non-unix line endings, so updated the diff.
Mon, Aug 19
the builds are successfully being built on the CI now. We just need to iron out the kinks so it actually works like it should, on all systems.
reference patch : https://phabricator.kde.org/D23209 (merged already)
can I merge this one? 👀
may I land this now? 👀
Sun, Aug 18
Sat, Aug 17
Fri, Aug 16
so we have Jenkins ready to build .Appx packages for us now, thanks to @bcooksley : https://invent.kde.org/sysadmin/binary-factory-tooling/commit/aeddc47b0ca4dcdef64044226b99d9b78a0156a7
took inspiration from NotifyByAndroid. Now I can see all types of notifications perfectly well!
Wed, Aug 14
the duplicate notification always has id= -1, so I think it's more an issue of KNotifications, I think
Tue, Aug 13
Mon, Aug 12
Sun, Aug 11
landing this one. 🎉
waiting for some green light(s) for now...
- removed the duplicate remover.
Thu, Aug 1
Sat, Jul 27
Fri, Jul 26
Please do a last minute review, I believe this patch is ready to merge now. 🎉
Wed, Jul 24
Tue, Jul 23
Jul 23 2019
let's put a pin in this for a while until I get a better solution for DBus on Windows
got it 😥🔫
nvm, I'll try to find a better solution instead.
sorry about that. I'll find a better solution that doesn't break builds 🙈
For KStatusNotifierItem, KDE Connect uses QSystemTrayIcon on windows. See https://github.com/KDE/kdeconnect-kde/blob/master/indicator/main.cpp#L169