Minutes Monday Plasma Meeting

Mon Feb 27 11:29:56 UTC 2017

Voila!
-- 
sebas

http://www.kde.org | http://vizZzion.org
-------------- next part --------------
Plasma Meeting minutes 27-02-2017

Present: romangg, notmart, Sho_, bshah, kbroulik, sebas

romangg:
* more KDE SoK work -> problems with reaching out to Martin because of tearing problems most probably coming from KWayland directly, but most should be done in D4539.
Diff 4539 "[WIP][Drm Backend] Direct Buffer Scanout and Atomic Modesetting improvements" [Needs Review] https://phabricator.kde.org/D4539
* worked on a little fork of SimpleMenu, where I can test out additional functionality, which is not (yet ;) ) in SimpleMenu
* while on this realized the possibilities of Kicker.RootModel could be somewhat expanded: D4792
Diff 4792 "[Kicker] Property for hiding app categories in RootModel" [Needs Review] https://phabricator.kde.org/D4792
* worked on ToolButton's design in QML: we settled on replicating the design of ToolButton in QWidgets. now trying to do what kbroulik said in the last comment.
D4797
Diff 4797 "[ToolButtonStyle] Use pure colors and no frame as background in flat mode" [Needs Review] https://phabricator.kde.org/D4797

notmart:
* don't show autohide panels when they resize https://phabricator.kde.org/D4718
Diff 4718 "support for auto-hidden windows to resize" [Needs Review] https://phabricator.kde.org/D4718
* finish framesvg refactor, simplification https://phabricator.kde.org/D4707
Diff 4707 "move setImagePath logic into updateFrameData()" [Needs Review] https://phabricator.kde.org/D4707
* try harder to not flicker when the thumbnail view starts (wallpaper dialog and kwin kcm, pushed)
* plasma themed QQC2: https://phabricator.kde.org/D4508
Diff 4508 "Plasma controls based on QtQuickControls2" [Needs Review] https://phabricator.kde.org/D4508
* make sure the cancel action is the last in the kio drop menu, for prettier folderview drop menus https://phabricator.kde.org/D4739
Diff 4739 "make sure the cancel action is last" [Needs Review] https://phabricator.kde.org/D4739
** Kirigami2:
** some interaction fixes in the fancy new headers
** new page in the gallery with a configurable header
note: i'm quie blocked right now by the reviews, would like to have those things closed and move on :)

Sho:
* [TM] Reworked StartupWMClass handling to more exactly match the fd.o spec, makes Chrome Apps and LibreOffice work (launcher and group associativity) without special mapping rules + speeds up app matching
* [TM] Fixed a second bug in the new per-activity launcher code breaking launcher DND rearanging
* [TM] Fixed duplicated word 'Desktop' in "Move to Desktop" submenu
* [TM] Iterating a patch for better vertical TM sizing behavior + options with a new contributor on Phab
* [Launchers] Minor menu building performance improvements in Kicker
* [Launchers] Helped Maui downstream with a menu structure problem
* [FV] Reworked DND behavior to feel nicer - now using icon center instead of delegate center when calculating drop pos, so the drop location matches the key visual (the icon) more closely, leading to less surprising results
* [Pager] Fixed drag state not being reset properly on drop outside panel window
TMFKASM
* [FV] Started work on ignoring drag move and drop events on items that don't allow drops, needs more work, not high priority
* [Other] Moved development system to Qt 5.9 branch, so far no issues
* [Other] Busy with some theming-related stuff for downstream employer

kbroulik:
Found an article about Plasma http://netrunner-mag.com/2017/02/17/the-state-of-plasma/
Has some valid points and I also had a look at them, notably D4762 and D4817
Diff 4762 "[SliderStyle] Allow providing a "hint-handle-size" element" [Needs Review] https://phabricator.kde.org/D4762
Diff 4817 "[Kickoff] Increase opacity for item description" [Needs Review] https://phabricator.kde.org/D4817
Complains about Media Controller lacking features, such as volume control for player, but I intend to keep it simple. I have played around with adding shuffle/repeat support (in the context menu) but I don't think that's used frequently enough to warrant adding it there. For volume we have that in Plasma PA anyway and I wouldn't know how to add a slider without overloading the UI or having people confuse it with the seek slider...
I want to look into removable media/devices shown in Dolphin/Device notifier, e.g. to exclude Snaps (seen tons of "Loop Device"), and also perhaps not offer unmounting / or /home (Bug 376498) but in Solid there's StorageVolume and StorageAccess and both of them need to announce "ignore" for them to not show up but StorageVolume only looks for "Filesystem" type whereas only StorageAccess knows the mount point (e.g. /snap/* or / or /home), so dunno how to
proceed here without changing the predicate used in tons of places... also, UDisks doesn't have any helpful properties (HintSystem is true for most of my partitions whereas HintIgnore is not), no idea what Unity and Gnome do to exclude such devices.
KDE bug 376498 in plasmashell (Device Notifier) ""Safe remove" should not be available for external drives containing OS install and/or /home folder" [normal,] https://bugs.kde.org/show_bug.cgi?id=376498

bshah:
- Fancy things first : https://ptpb.pw/md50.log
    - This is systemd working inside android container perfectly fine
    - Will look into packaging/running plasma mobile there
- Looking into issue of Nexus 5X overheating with higher priority currently
    - some stuff in android keeps restarting at full speed and I've no idea why (yet)
- personal
    - spending some time with conf.kde.in 2017 organization
whooooot!
so first, thing about the systemd, is we never got it working inside android lxc
but, with some "tweaks" I got it working
this will allow us to use other distros than xenial base we are using
about overheating problem.. there are two android daemons.. which keep restarting and using CPU

sebas:
* Some reviewing, esp kcheckpass improvements by mgraesslin
* Mostly not directly plasma-related stuff on my plate right now
* Dot story for sprint to finish

IRC Log:
[11:59:27] <sebas> meeting? :)
[11:59:38] --> HoloIRCUser1 (~holoirc at 87.116.230.210) has joined #plasma
[12:00:44] <sebas> Anybody? :)
[12:00:57] <bshah> I am here.. but.. schedule me slightly late please
[12:01:02] <bshah> need to fix notes
[12:01:07] <notmart> me is here too
[12:01:09] <Sho_> I'm here
[12:01:12] <sebas> notmart, kbroulik, bshah, d_ed, romangg, Sho_?
[12:01:16] <sebas> ah, good morning!
[12:01:25] <Sho_> for some people I'm sure
[12:01:27] <romangg> here. can I go first again? have to leave early.
[12:01:31] <sebas> notmart to start, then Sho, then bshah, romangg?
[12:01:37] <sebas> romangg: sure, go ahead
[12:01:55] -*- notmart lets romangg
[12:02:02] <romangg> * more KDE SoK work -> problems with reaching out to Martin because of tearing problems most probably coming from KWayland directly, but most should be done in D4539.
[12:02:02] <sKreamer> Diff 4539 "[WIP][Drm Backend] Direct Buffer Scanout and Atomic Modesetting improvements" [Needs Review] https://phabricator.kde.org/D4539
[12:02:23] <romangg> * worked on a little fork of SimpleMenu, where I can test out additional functionality, which is not (yet ;) ) in SimpleMenu
[12:02:31] <romangg> * while on this realized the possibilities of Kicker.RootModel could be somewhat expanded: D4792
[12:02:32] <sKreamer> Diff 4792 "[Kicker] Property for hiding app categories in RootModel" [Needs Review] https://phabricator.kde.org/D4792
[12:02:39] <romangg> * worked on ToolButton's design in QML: we settled on replicating the design of ToolButton in QWidgets. now trying to do what kbroulik said in the last comment.
[12:02:53] <romangg> D4797
[12:02:54] <sKreamer> Diff 4797 "[ToolButtonStyle] Use pure colors and no frame as background in flat mode" [Needs Review] https://phabricator.kde.org/D4797
[12:03:05] <romangg> done. questions?
[12:03:33] <sebas> all clear to me :)
[12:03:36] <sebas> notmart!
[12:04:13] <starbuck1> romangg: what is missing from simplemenu? :)
[12:04:26] <romangg> I want a view for recently used documents/apps
[12:04:35] <-> HoloIRCUser1 is now known as llucas
[12:04:36] <starbuck1> ah okay
[12:04:40] <-- jalalsfs (~jalal at unaffiliated/jalalsfs) has quit (Ping timeout: 240 seconds)
[12:04:51] <notmart> * don't show autohide panels when they resize https://phabricator.kde.org/D4718
[12:04:51] <sKreamer> Diff 4718 "support for auto-hidden windows to resize" [Needs Review] https://phabricator.kde.org/D4718
[12:04:52] <notmart> * finish framesvg refactor, simplification https://phabricator.kde.org/D4707
[12:04:53] <sKreamer> Diff 4707 "move setImagePath logic into updateFrameData()" [Needs Review] https://phabricator.kde.org/D4707
[12:04:54] <notmart> * try harder to not flicker when the thumbnail view starts (wallpaper dialog and kwin kcm, pushed)
[12:04:56] <notmart> * plasma themed QQC2: https://phabricator.kde.org/D4508
[12:04:56] <sKreamer> Diff 4508 "Plasma controls based on QtQuickControls2" [Needs Review] https://phabricator.kde.org/D4508
[12:04:57] <notmart> * make sure the cancel action is the last in the kio drop menu, for prettier folderview drop menus https://phabricator.kde.org/D4739
[12:04:57] <sKreamer> Diff 4739 "make sure the cancel action is last" [Needs Review] https://phabricator.kde.org/D4739
[12:04:59] <notmart> ** Kirigami2:
[12:05:00] <notmart> ** some interaction fixes in the fancy new headers
[12:05:02] <notmart> ** new page in the gallery with a configurable header
[12:05:31] <notmart> note: i'm quie blocked right now by the reviews, would like to have those things closed and move on :)
[12:05:46] <notmart> questoins/next?
[12:06:19] <kbroulik> re
[12:06:27] <sebas> notmart: I'll try to get at least some of them reviewed
[12:06:29] <romangg> starbuck1: http://imgur.com/a/c72EM
[12:06:33] --> nowrep (~david at kde/developer/drosca) has joined #plasma
[12:06:37] <notmart> sebas: thx
[12:07:13] <sebas> Sho_!
[12:07:17] <Sho_> romangg: not bad although I'm not sure it's simple anymore :)
[12:07:22] <Sho_> * [TM] Reworked StartupWMClass handling to more exactly match the fd.o spec, makes Chrome Apps and LibreOffice work (launcher and group associativity) without special mapping rules + speeds up app matching
[12:07:24] <Sho_> * [TM] Fixed a second bug in the new per-activity launcher code breaking launcher DND rearanging
[12:07:25] <Sho_> * [TM] Fixed duplicated word 'Desktop' in "Move to Desktop" submenu
[12:07:27] <Sho_> * [TM] Iterating a patch for better vertical TM sizing behavior + options with a new contributor on Phab
[12:07:28] <Sho_> * [Launchers] Minor menu building performance improvements in Kicker
[12:07:30] <Sho_> * [Launchers] Helped Maui downstream with a menu structure problem
[12:07:31] <Sho_> * [FV] Reworked DND behavior to feel nicer - now using icon center instead of delegate center when calculating drop pos, so the drop location matches the key visual (the icon) more closely, leading to less surprising results
[12:07:33] <Sho_> * [Pager] Fixed drag state not being reset properly on drop outside panel window
[12:07:33] <sebas> TMFKASM
[12:07:34] <Sho_> * [FV] Started work on ignoring drag move and drop events on items that don't allow drops, needs more work, not high priority
[12:07:36] <Sho_> * [Other] Moved development system to Qt 5.9 branch, so far no issues
[12:07:37] <Sho_> * [Other] Busy with some theming-related stuff for downstream employer
[12:07:54] --> brdcrbs (~innermous at 213.110.97.144) has joined #plasma
[12:08:04] <-- moofang (~lim at 101.127.0.50) has quit (Ping timeout: 260 seconds)
[12:08:17] <d_ed> notmart: RE: your patches, I think I have comments remaining on most of them.
[12:08:26] <d_ed> notmart: are we wanting QQC2 for this frameworks, or the start of the next
[12:08:30] <Sho_> questions || next
[12:08:41] <notmart> d_ed: qqc2, i think starting from next
[12:08:45] <starbuck1> romangg: interesting, but maybe what Sho_ said ;)
[12:08:47] <brdcrbs> What happened with last plasma update? Literally nothing in my "Desktop effects" doesn't work anymore: wobbly windows, fade animation, desktop grid - nuffin
[12:08:51] <sebas> d_ed: would mean bumping the Qt dep, right?
[12:08:54] <notmart> the framesvg stuff, i would like it for this one instead
[12:09:19] <notmart> also the kio dropmenu stuff
[12:10:19] <kbroulik> brdcrbs: check whether desktop compositing is enabled (Alt+Shift+F12, and/or in system settings → display → compositor → check the checkbox, might have a warning there), we're in a meeting right now, will take half an hour, then we can help you in more detail :)
[12:10:34] <d_ed> notmart: ok, I'll make sure they get priority
[12:10:44] <notmart> thx
[12:10:49] <sebas> alright, moving on ... bshah's turn!
[12:11:10] <bshah> moment
[12:11:16] <brdcrbs> kbroulik: hm did have Ogl warning...
[12:11:20] <sebas> kbroulik: first then?
[12:11:20] <brdcrbs> kbroulik: thanks
[12:11:28] <kbroulik> Found an article about Plasma http://netrunner-mag.com/2017/02/17/the-state-of-plasma/
[12:11:29] <kbroulik> Has some valid points and I also had a look at them, notably D4762 and D4817
[12:11:29] <sKreamer> Diff 4762 "[SliderStyle] Allow providing a "hint-handle-size" element" [Needs Review] https://phabricator.kde.org/D4762
[12:11:29] <sKreamer> Diff 4817 "[Kickoff] Increase opacity for item description" [Needs Review] https://phabricator.kde.org/D4817
[12:11:39] <kbroulik> Complains about Media Controller lacking features, such as volume control for player, but I intend to keep it simple. I have played around with adding shuffle/repeat support (in the context menu) but I don't think that's used frequently enough to warrant adding it there. For volume we have that in Plasma PA anyway and I wouldn't know how to add a slider without overloading the UI or having people confuse it with the seek slider...
[12:11:55] <kbroulik> I want to look into removable media/devices shown in Dolphin/Device notifier, e.g. to exclude Snaps (seen tons of "Loop Device"), and also perhaps not offer unmounting / or /home (Bug 376498) but in Solid there's StorageVolume and StorageAccess and both of them need to announce "ignore" for them to not show up but StorageVolume only looks for "Filesystem" type whereas only StorageAccess knows the mount point (e.g. /snap/* or / or /home), so dunno how to
[12:11:56] <kbroulik> proceed here without changing the predicate used in tons of places... also, UDisks doesn't have any helpful properties (HintSystem is true for most of my partitions whereas HintIgnore is not), no idea what Unity and Gnome do to exclude such devices.
[12:11:56] <sKreamer> KDE bug 376498 in plasmashell (Device Notifier) ""Safe remove" should not be available for external drives containing OS install and/or /home folder" [normal,] https://bugs.kde.org/show_bug.cgi?id=376498
[12:12:22] <kbroulik> (didnt we say we wanted to avoid wall-of-commits and instead also focus on what we want to do next? :D)
[12:12:59] <sebas> I like it :D
[12:13:03] <sebas> this is really useful
[12:13:52] <kbroulik> questions || next
[12:13:56] <-- brdcrbs (~innermous at 213.110.97.144) has quit (Read error: Connection reset by peer)
[12:14:04] <d_ed> RE: your opacity commits, we should try and standardise that a bit - it's the one place where we're using magic numbers everywhere
[12:14:21] <d_ed> previously I've seen enabled: false on labels
[12:14:34] <d_ed> which then makes it theme dependent
[12:14:38] <sebas> opacity: 0.6 is what we use consistently
[12:14:49] <sebas> at least in lots of other places
[12:15:29] <kbroulik> oh, enabled: false does opacity 0.6 on label
[12:15:34] <notmart> kbroulik: +1 for keeping it simple
[12:15:41] <kbroulik> yeah, 0.6 is the magic number we use everywhere consistently xD
[12:15:44] <notmart> there is always the store for alternative stuff
[12:15:48] <sebas> enabled: false is semantically incorrect though
[12:15:48] <kbroulik> exactly
[12:15:54] <kbroulik> there's even a media widget on the store which has all of this
[12:15:55] --> stikonas (~gentoo at wesnoth/translator/stikonas) has joined #plasma
[12:16:01] <kbroulik> it even places all buttons in the panel (e.g. not just the play icon but also next/prev/stop)
[12:16:18] <notmart> we could have a Theme.disabledOpacity if we think is important enough to remove the magic
[12:16:25] <kbroulik> also, the article sys "yeah the gnome thing is much better ... it's an extension though", whereas we provide something simple good ootb... :D
[12:16:28] --> jalalsfs (~jalal at unaffiliated/jalalsfs) has joined #plasma
[12:16:44] <d_ed> or a new Label subclass where all the headings are
[12:17:03] <sebas> PlasmaExtras is already underused IMO
[12:17:23] <d_ed> if you want semantic, that's the way to go
[12:17:29] <sebas> agree
[12:17:35] <sebas> I came up with it :D
[12:17:44] <llucas> kbroulik: Yes widget that cannot be installed
[12:18:01] <sebas> for headings, I think it works quite well though, would have liked to see the animation classes be extended and more actively used
[12:18:10] <Sho_> the last time I suggested factoring out the magic numbers into lib code it was shot down in meeting btw
[12:18:31] <kbroulik> yeah, the pulse anim thing in systray should be in p-f extras animations
[12:18:52] <sebas> kbroulik: deja vu to two weeks back? :D
[12:18:59] <kbroulik> indeed :)
[12:19:17] <notmart> sebas: for me the animation classes in plasmaextras weren't that good api-wise, they had to be used in a kindof imperative way iirc, while what we use most (as we should, i think), is mostly behavior animations
[12:19:40] <kbroulik> yeah
[12:19:41] <sebas> notmart: ah, right... could be that indeed
[12:19:56] <kbroulik> having some standard behavior anims would also help using consistent easing and stuff, which Sho_ always complains about
[12:20:12] <notmart> systray anim... i would love so much to get rid of that pulse that makes it pixelated and use the requesting attention icon again\
[12:20:22] <kbroulik> :D
[12:20:47] <d_ed> anyway, I think we've got some ideas, I didn't mean to sidetrack things too much
[12:21:12] <sebas> I like the pulse for being rather subtle, and not tracking *too much* attention
[12:21:24] <sebas> I mean, I can still read while it's "flashing"
[12:21:33] <sebas> bshah's turn
[12:21:45] <bshah> okay
[12:21:52] <bshah> first some copy pasta:
[12:21:54] <bshah> - Fancy things first : https://ptpb.pw/md50.log
[12:21:56] <bshah>     - This is systemd working inside android container perfectly fine
[12:21:58] <bshah>     - Will look into packaging/running plasma mobile there
[12:22:01] <bshah> - Looking into issue of Nexus 5X overheating with higher priority currently
[12:22:02] <bshah>     - some stuff in android keeps restarting at full speed and I've no idea why (yet)
[12:22:04] <bshah> - personal
[12:22:06] <bshah>     - spending some time with conf.kde.in 2017 organization
[12:22:16] <sebas> whooooot!
[12:22:41] <bshah> so first, thing about the systemd, is we never got it working inside android lxc
[12:22:49] <bshah> but, with some "tweaks" I got it working
[12:23:10] <bshah> this will allow us to use other distros than xenial base we are using
[12:23:26] <bshah> about overheating problem.. there are two android daemons.. which keep restarting and using CPU
[12:23:33] <bshah> so thats it
[12:23:53] <bshah> questions?
[12:24:04] <sebas> just praise
[12:24:26] <sebas> also, my turn, not that i've much to add
[12:24:35] <sebas> * Some reviewing, esp kcheckpass improvements by mgraesslin
[12:24:35] <sebas> * Mostly not directly plasma-related stuff on my plate right now
[12:24:35] <sebas> * Dot story for sprint to finish
[12:25:44] <-- Delta-One (~zero at 2001:2002:51e2:7ba1:6391:acc7:b55f:c617) has quit (Ping timeout: 240 seconds)
[12:25:47] <kbroulik> speaking of dot story, I wanted to blog a bit but I can't find the motivation to do so... :( all I collected is some screenshots, and a couple of bullet points :D
[12:26:07] <sebas> kbroulik: send 'em over to me, I'll work it into the dot story?
[12:26:23] <sebas> same here, motivation to write isn't very high, but I'm disciplined :)
[12:27:28] <Sho_> do anyone remember if QML had a constant for max integer?
[12:27:48] <sebas> I don't. Meeting officially over, thanks for attending everybody!
[12:27:53] <sebas> notes go to the list shortly