Minutes Monday Plasma Meeting

Mon Mar 6 11:25:27 UTC 2017

Attached, the minutes and log of today's meeting...
-- 
sebas

http://www.kde.org | http://vizZzion.org
-------------- next part --------------
Plasma Weekly Meeting, 6-2-2017

Sho:
* [FV] Tweaks to item delegate margins - results in less wide gutters on the desktop, fitting more rows and cols, in response to frequent user feedback
* [FV] Tweaks to hover hitbox - now wraps icon and label more tightly, giving rectangle select wider gutters to start in. Un-hover and drag hover hotboxes unchanged
* [FV] A few right-to-left locale fixes (file rename box positioning, list view mode item frame positioning)
* [FV] Fixed actions overlay position not adjusting properly when switching between icon and listview modes at runtime
* [FV] Blog about FV default and recent work -> https://blogs.kde.org/2017/03/01/plasma-510-folder-view-default-desktop-mode
* [TM] Fixed pinning Krita properly - it was incorrectly mapping to one of the tons of extra .desktop files it installs for MIME associations because we weren't ignoring NoDisplay=true services, meaning e.g. recent files in context menu didn't work as things don't match up with the KAStats db
* [TM] Merged new contributor's patch for better vertical sizing behavior after iterating with him
* [Appdash] Fixed switching categories via the filter list view on touchscreens
* [General] Upstreamed a few UI tweaks Maui/Netrunner make for release as options

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
* fallback chain as prefix of framesvgitem: prettier code for falling back in svg items while using new elements https://phabricator.kde.org/D4827
Diff 4827 "possibility to use a fallback chain as prefix" [Closed] https://phabricator.kde.org/D4827
** as a sideeffect: toolbuttons in breeze plasma theme look the same as breeze widgets
* now a pinned systray popup gets correctly closed
* prettyfication of lock screen on screen keyboard layout and transitions https://phabricator.kde.org/D4893
Diff 4893 "Layout and animations on the on screen keyboard" [Needs Review] https://phabricator.kde.org/D4893
* fix of size hints propagation when applets like folderview go from full representation to collapsed
* fix of old theme cache deletion: https://phabricator.kde.org/D4925
Diff 4925 "actually delete old theme caches" [Needs Review] https://phabricator.kde.org/D4925
* kirigami2:
Sho_: later
** ported lookandfeelexplorer to kirigami2
waiting for this frameworks release before pushing the qqc2 stuff, still 3 open reviews to go

kbroulik:
Did a ton of work on Folder View, especially wrt renaming files. Also improved Trash handling (Empty Trash option), improved keyboard shortcuts (Ctrl+Z support etc)
Slider can now provide a hint for the actual size of the handle, excluding any shadows, allowing sliders to actually touch the end of the rail underneath (looking so much nicer now)
And a ton of open to-be-reviewed changes: D4956, D4924, D4923, D4899, D4861, D4761, D4614
Diff 4956 "[Folder View] Lower toolbox when an item is hovered" [Needs Review] https://phabricator.kde.org/D4956
Diff 4924 "[Containment Interface] Trigger context menus on pressing "Menu" key" [Needs Review] https://phabricator.kde.org/D4924
Diff 4923 "[desktop:/ KIO] Strip superfluous slashes and fixup local root url" [Needs Review] https://phabricator.kde.org/D4923
Diff 4899 "[Folder View] Workaround label eliding bug" [Needs Review] https://phabricator.kde.org/D4899
Diff 4861 "[ConfigCategory] Deduce name and icon from pluginName" [Needs Review] https://phabricator.kde.org/D4861
ejay (~edwin at 94.158.225.38) has quit (Quit: Konversation terminated!)
Diff 4761 "[Task Manager] Don't delay appearance of audio stream in popup" [Needs Review] https://phabricator.kde.org/D4761
Better action icons D4917 needs VDG input but haven't gotten any so far...
Diff 4614 "[Baloo Widgets] Add KPropertiesDialog Plugin with file metadata" [Needs Review] https://phabricator.kde.org/D4614
Diff 4917 "[Breeze Plasma Theme] Improve action-overlay icons" [Needs Review] https://phabricator.kde.org/D4917
Had an idea for Jump List Action plugins. Currently they are just static actions in .desktop files which the app cannot dynamically change. My idea was to add a JumpListActionPlugin that apps can ship to overwrite the jump list actions and add their own stuff. For instance, Kate could show its Sessions there, KDevelop could replace recent docs by "recent projects". The app could also be told a context (where the menu was opened, e.g. TM or kickoff, if it is
 running or not), etc. I also don't think it would conflict with window metadata as jump list actions are mostly meant for launching an app and a task within, rather than controling running instances. Needs to be discussed with Sho_ obviously but it was an idea I had at the weekend.

romangg:
Done:
- D4867 and D4868
Diff 4867 "[Menu] Improved available space correction for openRelative" [Closed] https://phabricator.kde.org/D4867
Diff 4868 "[TaskManager] For contextMenuComponent placement use accurate enum on right edge" [Closed] https://phabricator.kde.org/D4868
- D4904
Diff 4904 "[Applet] Open contextMenu relatively to visual parent contextMenuButton" [Closed] https://phabricator.kde.org/D4904
In review:
- D4921 (succesor to D-4867 above)
Diff 4921 "[Menu] Harmonize openRelative placement" [Needs Review] https://phabricator.kde.org/D4921
Want to do next this week:
- After D-4921 got merged simplify the correctional code for openRelative
- Look into tearing issues of scanout patch
- Related to D4792 look into how to do it like Sho_ wanted it (example is Menu component)
Diff 4792 "[Kicker] Property for hiding app categories in RootModel" [Needs Review] https://phabricator.kde.org/D4792

sebas:
* Not much time for Plasma, other than that the Dot story for the sprint is still on my plate

Log:

[12:00:46] <Sho_> meeting? who hosts?
[12:01:08] -*- sebas /me /me /me!
[12:01:14] <sebas> Who's there>
[12:01:15] <sebas> ?
[12:01:19] <Sho_> meememe
[12:01:28] <Sho_> but at anniv dinner with gf
[12:01:30] <kbroulik> brb but will be in
[12:01:35] <kbroulik> einar77_work: <3
[12:01:35] <Sho_> so i need to dump sorry
[12:01:46] <sebas> Sho_: skip this meeting maybe?
[12:02:05] <sebas> Anniversary dinner sounds like something more important than our weekly...
[12:02:09] <Sho_> It's ok she alao has a work tether
[12:02:13] <Sho_> Mondays :)
[12:02:18] <sebas> So romantic ;)
[12:02:21] <Sho_> ;)
[12:02:28] <sebas> anyway, let's get started
[12:02:33] <Sho_> if you don't mind I will just dump now ahead of others sorry
[12:02:38] <sebas> yes please
[12:02:40] <Sho_> I've continued to do lots of FV polish to make our new 5.10 default containment a success, addressing some often-voiced user feedback * [FV] Tweaks to item delegate margins - results in less wide gutters on the desktop, fitting more rows and cols, in response to frequent user feedback * [FV] Tweaks to hover hitbox - now wraps icon and label more tightly, giving rectangle select wider gutters to start in. Un-hover and drag hover hotboxes unchanged * [FV] A few right-to-
[12:02:45] <Sho_> ugh no lime breaks
[12:02:54] <Sho_> i will mail.you for notes
[12:02:58] <Sho_> sorry!
[12:03:01] <Sho_> running ofd
[12:03:03] <notmart> uh, meeting!
[12:03:14] <notmart> damn, almost forgot, i'm here
[12:03:34] <sebas> hehe, hey notmart
[12:03:56] <sebas> notmart: want to continue?
[12:04:00] <notmart> ok, sure
[12:04:10] <Sho_> mail sent & food time, cheers later :)
[12:04:32] <Sho_> (sorry for chaos)
[12:04:39] <notmart> * don't show autohide panels when they resize https://phabricator.kde.org/D4718
[12:04:39] <sKreamer> Diff 4718 "support for auto-hidden windows to resize" [Needs Review] https://phabricator.kde.org/D4718
[12:04:40] <notmart> * fallback chain as prefix of framesvgitem: prettier code for falling back in svg items while using new elements https://phabricator.kde.org/D4827
[12:04:41] <sKreamer> Diff 4827 "possibility to use a fallback chain as prefix" [Closed] https://phabricator.kde.org/D4827
[12:04:42] <notmart> ** as a sideeffect: toolbuttons in breeze plasma theme look the same as breeze widgets
[12:04:44] <notmart> * now a pinned systray popup gets correctly closed
[12:04:45] <notmart> * prettyfication of lock screen on screen keyboard layout and transitions https://phabricator.kde.org/D4893
[12:04:45] <sKreamer> Diff 4893 "Layout and animations on the on screen keyboard" [Needs Review] https://phabricator.kde.org/D4893
[12:04:47] <notmart> * fix of size hints propagation when applets like folderview go from full representation to collapsed
[12:04:48] <notmart> * fix of old theme cache deletion: https://phabricator.kde.org/D4925
[12:04:48] <sKreamer> Diff 4925 "actually delete old theme caches" [Needs Review] https://phabricator.kde.org/D4925
[12:04:50] <notmart> * kirigami2:
[12:04:51] <sebas> Sho_: later
[12:04:51] <notmart> ** ported lookandfeelexplorer to kirigami2
[12:05:18] <notmart> waiting for this frameworks release before pushing the qqc2 stuff, still 3 open reviews to go
[12:05:19] --> kyan (~kyan at cpe-24-198-97-231.maine.res.rr.com) has joined #plasma
[12:05:35] <sebas> aye, thanks
[12:05:54] <sebas> kirigami port of lnfexplorer, did it go easily, or did you run into anything?
[12:05:59] <notmart> a couple reviews of qt gerrit still open
[12:06:06] <notmart> sebas: renamed imports, done
[12:06:10] <sebas> nice :)
[12:06:16] <sebas> kbroulik: are you ready yet?
[12:06:48] <kbroulik> not yet
[12:07:01] <-- agurenko (agurenko at nat/redhat/x-huqkpckqbmtbjohi) has quit (Ping timeout: 240 seconds)
[12:07:29] <sebas> I have almost nothing to report ...
[12:07:37] <sebas> * Not much time for Plasma, other than that the Dot story for the sprint is still on my plate
[12:07:48] <sebas> Anybody else participating in this meeting?
[12:07:59] <bshah> I am "lurking"..
[12:08:08] <bshah> about my report, nothing really
[12:08:17] <bshah> I am mostly busy with final preps of conf.kde.in
[12:08:23] <sebas> what's your plan this week then?
[12:08:26] <sebas> ah, figures
[12:09:18] <kbroulik> ok, I'll go ahead
[12:09:59] <kbroulik> Did a ton of work on Folder View, especially wrt renaming files. Also improved Trash handling (Empty Trash option), improved keyboard shortcuts (Ctrl+Z support etc)
[12:10:02] <kbroulik> Slider can now provide a hint for the actual size of the handle, excluding any shadows, allowing sliders to actually touch the end of the rail underneath (looking so much nicer now)
[12:10:41] <kbroulik> And a ton of open to-be-reviewed changes: D4956, D4924, D4923, D4899, D4861, D4761, D4614
[12:10:41] <sKreamer> Diff 4956 "[Folder View] Lower toolbox when an item is hovered" [Needs Review] https://phabricator.kde.org/D4956
[12:10:41] <sKreamer> Diff 4924 "[Containment Interface] Trigger context menus on pressing "Menu" key" [Needs Review] https://phabricator.kde.org/D4924
[12:10:41] <sKreamer> Diff 4923 "[desktop:/ KIO] Strip superfluous slashes and fixup local root url" [Needs Review] https://phabricator.kde.org/D4923
[12:10:41] <sKreamer> Diff 4899 "[Folder View] Workaround label eliding bug" [Needs Review] https://phabricator.kde.org/D4899
[12:10:43] <sKreamer> Diff 4861 "[ConfigCategory] Deduce name and icon from pluginName" [Needs Review] https://phabricator.kde.org/D4861
[12:10:44] <-- ejay (~edwin at 94.158.225.38) has quit (Quit: Konversation terminated!)
[12:10:47] <sKreamer> Diff 4761 "[Task Manager] Don't delay appearance of audio stream in popup" [Needs Review] https://phabricator.kde.org/D4761
[12:10:47] <kbroulik> Better action icons D4917 needs VDG input but haven't gotten any so far...
[12:10:49] <sKreamer> Diff 4614 "[Baloo Widgets] Add KPropertiesDialog Plugin with file metadata" [Needs Review] https://phabricator.kde.org/D4614
[12:10:52] <sKreamer> Diff 4917 "[Breeze Plasma Theme] Improve action-overlay icons" [Needs Review] https://phabricator.kde.org/D4917
[12:11:36] <sebas> wow, nice :)
[12:11:49] <sebas> With that, I think we're done for today, that was quick :)
[12:11:54] <romangg> me!
[12:12:20] <sebas> romangg!
[12:12:22] <sebas> go
[12:12:27] <kbroulik> sec
[12:12:32] <kbroulik> have a textblock coming in
[12:12:34] <kbroulik> then ou can go :)
[12:12:55] <romangg> sure :)
[12:13:09] <kbroulik> Had an idea for Jump List Action plugins. Currently they are just static actions in .desktop files which the app cannot dynamically change. My idea was to add a JumpListActionPlugin that apps can ship to overwrite the jump list actions and add their own stuff. For instance, Kate could show its Sessions there, KDevelop could replace recent docs by "recent projects". The app could also be told a context (where the menu was opened, e.g. TM or kickoff, if it is
[12:13:09] <kbroulik>  running or not), etc. I also don't think it would conflict with window metadata as jump list actions are mostly meant for launching an app and a task within, rather than controling running instances. Needs to be discussed with Sho_ obviously but it was an idea I had at the weekend.
[12:13:11] <kbroulik> done
[12:13:47] <romangg> ok, my turn:
[12:13:53] <romangg> Done:
[12:13:54] <sebas> sounds logical, kbroulik
[12:13:58] <romangg> - D4867 and D4868
[12:13:58] <sKreamer> Diff 4867 "[Menu] Improved available space correction for openRelative" [Closed] https://phabricator.kde.org/D4867
[12:13:58] <sKreamer> Diff 4868 "[TaskManager] For contextMenuComponent placement use accurate enum on right edge" [Closed] https://phabricator.kde.org/D4868
[12:14:03] <romangg> - D4904
[12:14:03] <sKreamer> Diff 4904 "[Applet] Open contextMenu relatively to visual parent contextMenuButton" [Closed] https://phabricator.kde.org/D4904
[12:14:08] <romangg> In review:
[12:14:13] <romangg> - D4921 (succesor to D-4867 above)
[12:14:13] <sKreamer> Diff 4921 "[Menu] Harmonize openRelative placement" [Needs Review] https://phabricator.kde.org/D4921
[12:14:18] <notmart> kbroulik: how would they be exported? dbusmenu? atoms? wayland protocol?
[12:14:30] <romangg> Want to do next this week:
[12:14:31] <romangg> - After D-4921 got merged simplify the correctional code for openRelative
[12:14:32] <romangg> - Look into tearing issues of scanout patch
[12:14:34] <romangg> - Related to D4792 look into how to do it like Sho_ wanted it (example is Menu component)
[12:14:34] <sKreamer> Diff 4792 "[Kicker] Property for hiding app categories in RootModel" [Needs Review] https://phabricator.kde.org/D4792
[12:14:35] <romangg> done
[12:15:20] <kbroulik> notmart: they have nothing to do with windows. the app has a plugin and TM then does plugin->jumpListActions(servicePtr, context) and the plugin returns a list of actions
[12:16:09] <notmart> hmm, a library dynamically loaded by the tm?
[12:16:15] <kbroulik> or kickoff could do the same when you right click an entry, or the icon applet, etc. it's purely kservice-based, won't care about a window. (it might tell the app "yo this was invoked on a window, not a launcher" and the app could then hide actions that don't make sense when already running or so)
[12:16:17] <kbroulik> yeah
[12:16:24] <kbroulik> it was just an idea, I'm open to suggestions, obviously :)
[12:16:59] <notmart> bit concerned to yet more native code that can be shipped by anyone loaded dynamically by the shell
[12:17:02] <kbroulik> but in essence, it should aid launching a new instance of an app, not control currently running apps, hence nothing wrt atoms, wayland, etc
[12:17:04] <kbroulik> yeah :/
[12:17:22] <kbroulik> we could also have a lib which then writes things into a config file instead. I don't know. We can figure it out :)
[12:17:29] <notmart> i would go more on a route to load it from dbus?
[12:17:35] <kbroulik> and if the app is not running?
[12:17:54] <kbroulik> btw, unity launchre api actually allows apps to export a dbus menu path which it will add to the menu. we don't support that, (yet?) though
[12:17:58] <notmart> then load the default one
[12:18:16] <notmart> and yeah, a config file that overrides the section in the desktop file may be an idea as well
[12:18:51] <kbroulik> the idea is for more dynamic actions, though. for example launching Kate with a specific session. or kdevelop recent projects instead of recent docs
[12:19:05] <kbroulik> so, dbus thing doesn't make sense in that context imho. I'm focusing on "launching an app", not for running apps
[12:19:25] <notmart> then config file sounds better?
[12:20:34] <kbroulik> oki. so we make a ~/.local/share/plasma_jumplistactions folder where apps can dump their stuff in?
[12:20:34] <kbroulik> e.g. we have a org.kde.kate.jumplistactions file which is a ini/kconfig thing?
[12:20:41] <kbroulik> that way also 3rd party apps could in theory use it without needing to use a lib
[12:20:55] <kbroulik> sebas: meeting over? :D
[12:21:03] <sebas> if you say so :)
[12:21:42] <notmart> kbroulik: yeah, like that :)
[12:22:00] <sebas> alright, coolio
[12:22:03] <sebas> meeting over then!
[12:22:04] <kbroulik> notmart: ok :) I'll come up with something
[12:22:06] <sebas> thanks everybody
[12:22:10] <notmart> :)