Voila!
--
sebas
http://www.kde.org | http://vizZzion.org
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@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@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@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@213.110.97.144) has joined #plasma
[12:08:04] <-- moofang (~lim@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@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@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@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@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
