Meeting minutes and log...
--
sebas
http://www.kde.org | http://vizZzion.org
Plasma Team Meeting, 22 january 2018
Present: Sho, notmart, d_ed, kbroulik, bshah, romangg, sebas
Sho:
* [Kicker] Fixed the search result that Enter/Return will run while still in
the search field not being highlighted, giving no visual feedback you can just
slam the key
* [Kicker] Fixed menu closing unexpectedly when moving mouse cursor away from
an item that opened a subdialog (two seperate bugs, fixed in one approach)
* [Dashboard] Fixed a small regression from a recent refactoring breaking "Add
to Favorites"
* [FV] Almost ready to add requested unit test to
https://codereview.qt-project.org/#/c/215939/ - was painful to figure out how
to write
* [Simple Menu] Released v1.0.8:
** Added user-requested option to only change categories on click (default:
hover switch)
** Synced custom panel button image handling to latest Kicker
** Synced config dialog UI to latest Kicker
** Mouse-hover-vs-keyboard-nav fix a la previous Dashboard fix
** Quite nifty speed optimizations a la previous Dashboard work
* [Simple Menu] Released v1.0.9 with a follow-up fix for a bug that causes
system action buttons to stay highlighted when the mouse pointer left the area
* [Simple Menu] Have another bugfix release planned soon ...
* [Other] Did some GIO/GVFS remote filesystem tests to evaluate their approach
to mounting vs. ours, as users seem to prefer it
notmart:
* first rough working kernel driver for dell tablet mode (it turns out it's a
standard-ish feature of intel acpi, so may work on more laptops than dells), in
talk with some kernel people to make it into mainline
* big kirigami branch about static plugins has been merged, deployment story
perhaps a bit easier
* fixes build issues on Android
* started patch to expose to kwin wether we are in tablet mode D9944 need help
how to correctly wire things together
Diff 9944 "[WIP]expose if the tablet mode switch is available" [Needs Review]
https://phabricator.kde.org/D9944
* plasma: tablet mode experiments: show/hide panels and show/hide plasmoids
* pushed slimmer scrollbars in breeze just on master, will be in 5.13
d_ed:
I ported the minimize all applet to libtaskmanager to work on wayland
and get rid of an import whilst we're at it
kbroulik:
* Fixed memory leak in notifications history using a Repeater and creating all
delegates at once (now uses ListView, it's rather ugly but the least invasive
(read: not a total rewrite) patch I could come up with that could still go into
5.12)
* Tried to fix Kickoff user avatar not reliably updating, D9950 and D9951 but
it still breaks in certain cases
Diff 9950 "[KUser] Check whether .face.icon is actually readable before
returning it" [Accepted] https://phabricator.kde.org/D9950
Diff 9951 "[KUserProxy] Adjust to accounts service" [Needs Review]
https://phabricator.kde.org/D9951
* Appmenu applet now underlines mnemonics only when Alt pressed to be in line
with Breeze style
* Fixed deleting and creating files ("New File" menu) in Folder View taking
forever (it didn't signal KDirNotify properly), D9912 still open. Haven't
figured out a fix for DropJob yet
Diff 9912 "[KNewFileMenu] KDirNotify::emitFilesAdded after storedPut" [Needs
Review] https://phabricator.kde.org/D9912
* Looked into getting rid of those stupid "Loop device" we get in places for
e.g. snappy mounts: D9895
Diff 9895 "[UDisks] Ignore non-user mounts" [Needs Review]
https://phabricator.kde.org/D9895
* Since we're getting quite a few crashes in libsqlite in the Bookmarksrunner
lately, which pokes Firefox' database file to extract bookmarks (speaking of
sqlite, could that be related to kactivities also breaking? that qsqlite has a
regression?), I was looking into providing the bookmarks runner through
plasma-browser-integration. It can just query the browser instead. However,
after I did all of that for D9984 I figured that it obviously won't work if the
browser isn't running. So I don't know how to procede. We could also have
p-b-i extract all bookmarks, save them into a json file somewhere and the
search there if not running but that wouldn't make it much better than what we
currently have, except for that we don't need SQL.
Diff 9984 "WIP: Add bookmarks runner" [Needs Review]
https://phabricator.kde.org/D9984
bshah:
- Mostly working on same stuff as last week
- Which includes testing the fixes for udev-android-networkmanager-simplelogin
startup race
- On popular request worked on Plasma Mobile ISO based on neon :
http://images.plasma-mobile.org/iso/
- Need help with that as plasmashell doesn't start on first start
- Will work on packaging newer libinput in neon
- will work on packaging qt 5.10 on neon
romangg:
well, currently (re)writing my Xwayland patches as I said.
d_ed and I added two KWin ideas for GSoC
https://community.kde.org/GSoC/2018/Ideas#KWin
there are still not many Plasma ideas.
Could there be a plasma workspace idea?
sebas:
* Testing kscreen osd branch by dvratil D9414 (X works nicely with a few
patches on top, but
* wayland hotplugging is broken on my system, so looking into that
* More kscreen bug triaging done and to do
* semi-related: Working on privacy goal stuff, collecting info, rallying
people, PR stuff around it
* want to get back onto the Plasma Mobile state of the art
[12:00:36] <sebas> Meeting time! Who's here?
[12:02:35] <bshah> I am here, but will take bit to preparte notes
[12:02:37] <Sho_> here
[12:02:43] <notmart> here
[12:03:06] <d_ed> I've not had any time for Plasma, so not really here.
[12:03:21] <notmart> the schroedinger david :)
[12:03:44] <kbroulik> jeez. bug against kdialog. I trace it, went from kdialog
to plasma-integration, then went to KIO and now it's back in plasma-integration
again
[12:03:48] <kbroulik> here
[12:04:59] <sebas> Alright ... let's start with Sho, notmart, d_ed, kbroulik,
bshah, /me
[12:05:00] <romangg> here, though it's like David (working on my Xwayland
patches)
[12:05:01] <sebas> GO!
[12:05:04] <sebas> +romangg :)
[12:05:18] <Sho_> * [Kicker] Fixed the search result that Enter/Return will run
while still in the search field not being highlighted, giving no visual
feedback you can just slam the key
[12:05:19] <Sho_> * [Kicker] Fixed menu closing unexpectedly when moving mouse
cursor away from an item that opened a subdialog (two seperate bugs, fixed in
one approach)
[12:05:21] <Sho_> * [Dashboard] Fixed a small regression from a recent
refactoring breaking "Add to Favorites"
[12:05:22] <Sho_> * [FV] Almost ready to add requested unit test to
https://codereview.qt-project.org/#/c/215939/ - was painful to figure out how
to write
[12:05:24] <Sho_> * [Simple Menu] Released v1.0.8:
[12:05:25] <Sho_> ** Added user-requested option to only change categories on
click (default: hover switch)
[12:05:27] <Sho_> ** Synced custom panel button image handling to latest Kicker
[12:05:28] <Sho_> ** Synced config dialog UI to latest Kicker
[12:05:30] <Sho_> ** Mouse-hover-vs-keyboard-nav fix a la previous Dashboard fix
[12:05:31] <Sho_> ** Quite nifty speed optimizations a la previous Dashboard
work
[12:05:33] <Sho_> * [Simple Menu] Released v1.0.9 with a follow-up fix for a
bug that causes system action buttons to stay highlighted when the mouse
pointer left the area
[12:05:34] <Sho_> * [Simple Menu] Have another bugfix release planned soon ...
[12:05:36] <Sho_> * [Other] Did some GIO/GVFS remote filesystem tests to
evaluate their approach to mounting vs. ours, as users seem to prefer it
[12:06:19] <Sho_> next()
[12:06:43] <romangg> re Simple Menu: I can't click on "recent documents" from
the context menu of a favorite
[12:06:56] --> llucas (d4f4b91c@gateway/web/freenode/ip.212.244.185.28) has
joined #plasma
[12:07:03] <kbroulik> Sho_: did you look at kio fuse?
[12:07:04] <Sho_> that's the * [Simple Menu] Have another bugfix release
planned soon ...
[12:07:08] <Sho_> kbroulik: no
[12:07:15] <kbroulik> https://techbase.kde.org/Projects/KioFuse you should
[12:07:25] <kbroulik> if you're into "remote filesystem tests"
[12:07:36] <Sho_> kbroulik: I didn't know someone resurrected it
[12:07:40] <Sho_> You have it working?
[12:07:40] <kbroulik> it wasn't
[12:07:51] <kbroulik> at one point I checked out the svn and tried to port it
to qt5 and ecm but gave up
[12:08:03] <kbroulik> but might be worth resurrecting it and completing it
[12:08:21] <Sho_> kbroulik: My employer asked me to look into it so I asked
David Faure to summarize issues with the GVFS approach and he had me run some
testrs
[12:08:21] <sebas> I once tried to mount a nepomuk kio slave, that was pretty
hilariously broken :D
[12:08:32] <Sho_> Which are inconclusive so far because I couldn't actually get
anything to use a mount yet
[12:08:40] <Sho_> apparently most things now just GIO directly, which is
basically like our stuff
[12:08:43] <kbroulik> not being able to easily browse smb shares in libreoffice
and $third-party-IDE is a major pain point with plasma, same for not being able
to stream video from a NAS easily
[12:08:45] <sebas> idea was to do a fs-based backup tool of semantic data,
crack smoking is awesome
[12:08:53] <kbroulik> do a GIO-KIO bridge \o/
[12:09:07] <Sho_> So the question would be why does LibreOffice have GIO spport
but not KIO support
[12:09:19] <kbroulik> we should bester LiMux into having them do that, they
sponsored the Qt 5 port after all
[12:09:21] <kbroulik> *pester
[12:09:27] <sebas> b/c Libreoffice is run by GNOME fansboys
[12:09:31] <kbroulik> it would be in their interest, too
[12:09:47] <Sho_> sebas: well they have a project to do a qt port and we have
our icons upstream etc.
[12:09:48] <notmart> well, limux...
[12:09:50] <kbroulik> (though, that seems to be a common thing. Chromium also
has more Gnome-specific codepaths than KDE stuff)
[12:09:55] <notmart> is, you know...
[12:09:57] <Sho_> there's a history of successful upstreaming kde stuff with
LO, I think, so ...
[12:09:57] --> oc2k1__ (~oc...@62-193-39-77.as16211.net) has joined #plasma
[12:10:04] <kbroulik> notmart: not dead until End of 2020!
[12:10:12] <kbroulik> yeah we should look into LO KIO :)
[12:10:29] <Sho_> kbroulik: but otoh I remember talking to a Chrome dev who
explicitly reached out to us to add KDE codepaths, etc.
[12:10:29] <notmart> if there is still room for meneuvering there, sure
[12:10:30] --> Son_Goku (~King_InuY@fedora/ngompa) has joined #plasma
[12:10:41] <Sho_> let's not always blame the others - are we screwing up
somehow, too?
[12:10:45] <-- oc2k1_ (~oc...@62-193-39-77.as16211.net) has quit (Ping timeout:
276 seconds)
[12:10:48] <romangg> kbroulik: What is dead may never die! XD
[12:10:51] <Sho_> food for thought
[12:10:59] <kbroulik> it was just an unbiased observation
[12:11:46] <romangg> well, the question is why app devs need to write two
different backends on Linux Desktop at all.
[12:11:49] <kbroulik> I wanted to read into how to contribute to chromium,
though, to improve the kde integration there. Often times they have a
gnome-specific codepath when there's actually a FDO standard we both support
[12:12:00] --> s8321414 (~s8321414@150.117.137.90) has joined #plasma
[12:12:19] <kbroulik> (like the "open containnig folder" stuff, they check for
nautilus and otherwise just open parent folder, rather than using the
FileManager1 dbus iface we both supprot, etc)
[12:12:19] <kbroulik> but let's not digress too much
[12:12:33] <sebas> notmart's turn!
[12:12:35] <Sho_> I think KDE often does adopt a mindset of "we're fixing all
of our issues inside of our bubble and if others want stuff it's up to them to
adopt it", but we should probably be more aggressive about writing patches and
submitting them
[12:12:41] <Sho_> and with that, notmart take it away indeed
[12:12:50] <sebas> FWIW, I agree with Sho
[12:13:02] <notmart> yeah, i kind of agree with that
[12:13:03] <notmart> aanyways
[12:13:42] <notmart> * first rough working kernel driver for dell tablet mode
(it turns out it's a standard-ish feature of intel acpi, so may work on more
laptops than dells), in talk with some kernel people to make it into mainline
[12:13:44] <notmart> * big kirigami branch about static plugins has been
merged, deployment story perhaps a bit easier
[12:13:45] <notmart> * fixes build issues on Android
[12:13:47] <notmart> * started patch to expose to kwin wether we are in tablet
mode D9944 need help how to correctly wire things together
[12:13:48] <sKreamer> Diff 9944 "[WIP]expose if the tablet mode switch is
available" [Needs Review] https://phabricator.kde.org/D9944
[12:13:48] <notmart> * plasma: tablet mode experiments: show/hide panels and
show/hide plasmoids
[12:13:50] <notmart> * pushed slimmer scrollbars in breeze just on master, will
be in 5.13
[12:14:17] <notmart> (so, pushing a kernel patch to go outside our bubble, yay
\o/)
[12:14:33] <sebas> cool stuff, and awesome you're getting into kernel
development
[12:14:40] <sebas> (You know you'll regret that)
[12:14:43] <sebas> d_ed's turn
[12:15:11] <d_ed> I ported the minimize all applet to libtaskmanager to work on
wayland
[12:15:13] <notmart> luckily it's a driver of a laptop thing, so at least i
know it won't compromise half of world's servers anytime soon :p
[12:15:22] <d_ed> and get rid of an import whilst we're at it
[12:15:40] <d_ed> will be on review at lunch
[12:15:47] <notmart> nice :)
[12:15:53] <d_ed> that's pretty much it on plasma
[12:15:59] <sebas> d_ed deletes all my code
[12:16:01] <sebas> awesome
[12:16:23] <sebas> kbroulik!
[12:16:30] <Sho_> "the process of getting Plasma to LTS involved rewriting
sebas' code"
[12:16:34] <Sho_> :P <3
[12:16:38] <sebas> I love you, too!
[12:16:44] <kbroulik> * Fixed memory leak in notifications history using a
Repeater and creating all delegates at once (now uses ListView, it's rather
ugly but the least invasive (read: not a total rewrite) patch I could come up
with that could still go into 5.12)
[12:16:45] <Sho_> :)
[12:16:54] <kbroulik> * Tried to fix Kickoff user avatar not reliably updating,
D9950 and D9951 but it still breaks in certain cases
[12:16:55] <sKreamer> Diff 9950 "[KUser] Check whether .face.icon is actually
readable before returning it" [Accepted] https://phabricator.kde.org/D9950
[12:16:55] <sKreamer> Diff 9951 "[KUserProxy] Adjust to accounts service"
[Needs Review] https://phabricator.kde.org/D9951
[12:17:02] <kbroulik> * Appmenu applet now underlines mnemonics only when Alt
pressed to be in line with Breeze style
[12:17:07] <kbroulik> * Fixed deleting and creating files ("New File" menu) in
Folder View taking forever (it didn't signal KDirNotify properly), D9912 still
open. Haven't figured out a fix for DropJob yet
[12:17:07] <sKreamer> Diff 9912 "[KNewFileMenu] KDirNotify::emitFilesAdded
after storedPut" [Needs Review] https://phabricator.kde.org/D9912
[12:17:18] <kbroulik> * Looked into getting rid of those stupid "Loop device"
we get in places for e.g. snappy mounts: D9895
[12:17:19] <sKreamer> Diff 9895 "[UDisks] Ignore non-user mounts" [Needs
Review] https://phabricator.kde.org/D9895
[12:17:25] <kbroulik> * Since we're getting quite a few crashes in libsqlite in
the Bookmarksrunner lately, which pokes Firefox' database file to extract
bookmarks (speaking of sqlite, could that be related to kactivities also
breaking? that qsqlite has a regression?), I was looking into providing the
bookmarks runner through plasma-browser-integration. It can just query the
browser instead. However, after I did all of that for D9984 I figured that it
obviously won't work if the
[12:17:25] <kbroulik> browser isn't running. So I don't know how to procede.
We could also have p-b-i extract all bookmarks, save them into a json file
somewhere and the search there if not running but that wouldn't make it much
better than what we currently have, except for that we don't need SQL.
[12:17:25] <sKreamer> Diff 9984 "WIP: Add bookmarks runner" [Needs Review]
https://phabricator.kde.org/D9984
[12:18:30] <notmart> kbroulik: maybe is thread safeness and could be made an
out of process runner like baloo?
[12:18:30] <kbroulik> next
[12:18:54] <kbroulik> yeah, was thinking about that too, but then had the
brilliant idea with p-b-i which turned out to be not as brilliant after all :)
[12:19:02] <sebas> bshah is next up
[12:19:11] <bshah> okay
[12:19:31] <bshah> - Mostly working on same stuff as last week
[12:19:33] <bshah> - Which includes testing the fixes for
udev-android-networkmanager-simplelogin startup race
[12:19:35] <bshah> - On popular request worked on Plasma Mobile ISO based on
neon : http://images.plasma-mobile.org/iso/
[12:19:37] <bshah> - Need help with that as plasmashell doesn't start on first
start
[12:19:39] <bshah> - Will work on packaging newer libinput in neon
[12:19:41] <bshah> - will work on packaging qt 5.10 on neon
[12:20:05] <-- ceed^ (~viva...@c-73-16-30-241.hsd1.ma.comcast.net) has quit
(Ping timeout: 240 seconds)
[12:20:43] <bshah> notmart: can you take a look at the plasmashell not starting
on iso bug? it seems panels show up but not containment
[12:20:52] <bshah> if I restart session everything is fine then
[12:21:09] <notmart> bshah: on mobile? pine? normal neon?
[12:21:24] <bshah> notmart: plasma mobile iso
[12:21:27] --> Jucato (~jucato@kde/developer/jucato) has joined #plasma
[12:21:33] <notmart> so the panels are there, not the desktop... pretty much
the opposite of what used to happen
[12:21:39] <bshah> :D
[12:21:56] <bshah> anyway.. breakout I suppose :)
[12:21:57] <-- Jucato (~jucato@kde/developer/jucato) has quit (Client Quit)
[12:21:58] <bshah> next
[12:22:05] <notmart> i'll give a try
[12:24:21] <romangg> shall I go?
[12:24:23] <bshah> *awkward silence*
[12:24:33] <bshah> romangg: sure :)
[12:24:46] <romangg> well, currently (re)writing my Xwayland patches as I said.
[12:24:58] <romangg> d_ed and I added two KWin ideas for GSoC
[12:25:05] <romangg> https://community.kde.org/GSoC/2018/Ideas#KWin
[12:25:20] <romangg> there are still not many Plasma ideas.
[12:25:37] <romangg> Could there be a plasma workspace idea?
[12:25:51] <romangg> next
[12:28:02] --> Jucato (~jucato@kde/developer/jucato) has joined #plasma
[12:28:06] <romangg> notmart: Do you have a repo with your kernel work?
[12:28:07] <ilya_b> Could I break in with Plasma Mobile UX?
[12:28:11] <romangg> ilya_b: sure
[12:28:37] <romangg> we are waiting on sebas currently anyway
[12:28:40] <ilya_b> As I asaid, I really, really hope to propose some PlaMo
projects to GSoC students
[12:28:52] <ilya_b> And KCMs seem to be a good idea
[12:28:57] <ilya_b> But we. need. mentorw
[12:29:02] <ilya_b> But we. need. mentors*
[12:29:14] <notmart> romangg: yes (it's cloned from an ubuntu repo tough as i
wanted to have everything else built with the exact same config as i'm a n00b
:p)
[12:29:15] <sebas> sorry ... I got distracted
[12:29:20] <romangg> ilya_b: you know about the KCM redesign effort like one
moth ago, right?
[12:29:43] <notmart> romangg: do you have hardware which the patch could work
on?
[12:29:45] <ilya_b> romangg: I know there is effort around, but don't know
where we are atm
[12:29:54] <sebas> * Testing kscreen osd branch by dvratil D9414 (X works
nicely with a few patches on top, but
[12:29:54] <sebas> * wayland hotplugging is broken on my system, so looking
into that
[12:29:54] <sebas> * More kscreen bug triaging done and to do
[12:29:54] <sebas> * semi-related: Working on privacy goal stuff, collecting
info, rallying people, PR stuff around it
[12:29:54] <sebas> * want to get back onto the Plasma Mobile state of the art
[12:29:54] <sKreamer> Diff 9414 "Implement OSD to select action when unknown
monitors is connected" [Needs Review] https://phabricator.kde.org/D9414
[12:29:57] <sebas> there we go
[12:30:43] <romangg> sebas: regarding hotplugging: you use atomic mode setting ?
[12:30:52] <ilya_b> So I think USB/MTP, Locale & Language, Notifications
mobile-friendly KCMs are still not there?
[12:31:01] <romangg> maybe try it with and without.
[12:31:01] <sebas> romangg: 4.14 kernel, that should be default with intel
graphics, no?
[12:31:02] --> gpunk (~gpunk@80.215.210.51) has joined #plasma
[12:31:16] <gpunk> hi
[12:31:41] <gpunk> the official Instant Messaging application for kde ?
[12:32:10] <sebas> for KDE, you mean used by the KDE developers?
[12:32:13] <romangg> sebas: I thought so. but just take a look in the wayland
log if it says somewhere "atomic"
[12:32:18] <sebas> or do you mean used in Plasma, gpunk?
[12:32:23] <gpunk> yes
[12:32:35] <sebas> romangg: "wayland log" ? :D
[12:32:40] <kbroulik> fvogt: also QProcess start (non-static) is inew in Qt
5.10 :D
[12:32:44] <kbroulik> but good tip nonetheless
[12:32:48] <sebas> gpunk: "yes" is not an answer to an either/or question :D
[12:33:10] <gpunk> "do you mean used in Plasma, gpunk?" yes
[12:33:28] <sebas> There's no official IM app to be used in Plasma
[12:33:45] <gpunk> ok
[12:33:48] <sebas> some devs use telegram, some people use kopete, many people
use IRC
[12:34:04] <gpunk> ok thnx :)
[12:34:06] <sebas> IME it mostly depends on who you want to talk to
[12:34:33] <sebas> The Plasma meeting, however, is closed, thanks all, will
sent notes and log to the list now
