==== Eike ====
---- DONE ----
* [Wayland] Virtual desktops progress:
** "Switch Desktop" containment action plugin ported to libtaskmanager,
bugs fixed
** "Switch Window" containment action updated to new libtaskmanager API,
bugs fixed
---- PLAN ----
* working on making the prototype virtual desktops KCM better (e.g. not
instant apply)
* trying to get the wayland virtual desktop stuff complete enough for an
early-cycle 5.15 merge
** what this will eventually need:
** completing the above KCM work, and subsequently VDG input
** more code reviews, e.g. my libtm patch hasn't been reviewed at all so far
** i think there's outstanding review and maybe coding on marco's code
too (I recall experiencing a recurring crash)
* figuring out what to do with the windows krunner plugin, which needs
porting as well,
** but has the problem of being not in the plasmahsell process and
needing the exclusive wayland protocols
** we should really try to merge this early in 5.15, because it is very
likely going to need a lot of testing and bugfixing,
** and since we originally wanted this to be in 5.14 it'd suck a lot if
it misses out another cycle
* we need to poke Riddell to set up a 5.15 kickoff meeting, but this
was important to discuss now already
---- Q & A ----
<kbroulik> Sho_: maybe dbusrunner can help, plasmashell exposing that
stuff to krunner, but we must think about (not neccessarily address
right now but have that in mind) security implications
<Sho_> kbroulik: I think a more pragmatic solution would be adopting the
KCM's "only in=x11" plugin metadata thing to disable the windows runner
on Wayland for now
<Sho_> kbroulik: And then asking KWin to figure out the auth thing
before long
--
<romangg> In regards to not instant apply in the VD KCM: Is this
alligned with what the VDG wants? There were several discussions in the
past about (not) instant apply
<kbroulik> even if we were to change that (which I'm not saying we
should) we must be consistent and not have a single new KCM randomly
beahve differently
<Sho_> romangg: I think the VDG wants instant-apply, but it's not been
decided yet and we can't just make individual KCMs step out of line randomly
<notmart_> it should use correctly the api needs apply, load, save
<Sho_> I agree with kbroulik
<notmart_> once it uses it really correctly, making the shell doing
apply to save or instant apply is a switch
<Sho_> but the KCM was a prototype so far meant to help Marco mainly,
and the code gets a lot more complex with delayed-apply, so i delayed
the delayed-apply :D
<notmart_> unless the kcm does things by itself, then is a problem
==== Jonathan ====
!! Plasma 5.12.7 LTS is due tomorrow !!
Plasma 5.15 Kickoff meeting shortly after 5.14 release
==== Roman ====
---- DONE ----
* Rework of input redirection: D15595
* Rebased my Wayland touch drag code onto this: D15466
* My Xwayland DND patch series is now complete with addition of D15627
D15628 D15629.
---- PLAN ----
* XDC 2018
* Respond to reviews
* Look into some issue in Xwayland's Present code discussed on xorg-devel
---- Q & A ----
<Sho_> romangg: So with the KRunner plugin thing we ran into the issue
that it needs to use PlasmaWindowManagement, but that can only be used
once right now by plasmashell
<Sho_> martin floeser had a plan there for some sort of auth system etc.
<Sho_> that's probably something to take up again in the mid-term future
<romangg> Do we really have this restriction in place already that it
can only be used once?
<notmart_> the globals? not sure
<romangg> The question is if we really want to have auth via Wayland
protocol at all. Nothing materialized in this direction anyways with our
competitors/partners.
<romangg> So DBus only it might be in the end.
<notmart_> but exposing window tasks via dbus, kinda misses the point
==== Marco ====
---- DONE ----
* fix ripple effect for SwipeListItem on Material style
* enhancements in Card:
** possible to control the sourceSize and all Image proeprties for the
header image
** possible to control wrapMode and size for the title
* fix icons:
** fix a regressions of icons that became smaller on mobile
** improve the heuristics on what icons should be monochrome and what
shouldn't
==== Kai Uwe ====
---- DONE ----
* Made "Open..." action for finished copy/downloads also work in history
* Added Open Document thumbnailer (odt, ods, odp, etc)
* Fixed device actions not udpating live in Device Notifier
* Fixed Discover not showing update notification (also in 5.12)
* Made a patch for Krita to set decoration colorscheme: D15640
* Raise configuration window when reusing it: D15705
* Fixed (worked around) Ark opening Open Document files (that are
*technically* archives) as archives
==== Fabian ====
---- Q & A ----
<fvogt> Someone asked this weekend why plasma-pa had only the
contents.rcc file anymore, apparently it's indeed annoying for some
"code editing users"
<fvogt> I had a look at that file and noticed that it's not compressed
at all - so why are bundled packages still a thing?
<kbroulik> we had experimented with using qrc for packaging plasmoids
but ended up not using it, perhaps was forgotten to remove again?=
<notmart_> yeah, can be removed
<notmart_> i still think it should be used, but ooooh, well
<kbroulik> "had only the contents"
<kbroulik> so something's wrong in cmake?
<fvogt> Would it be possible to just alias the _bundled variant in
kpackage to the non-bundled one?
<notmart_> simply in plasma-pa is using the cmake macro to do bundled
packages, should be changed that
<kbroulik> meeeeeeeeeting? notmart_ Sho_ romangg
<Sho_> pong
<Riddell> hi
<romangg> here
<kbroulik> ok, Sho_ starts
<Sho_> oh noes, we are missing a notmart
<Sho_> d_ed is still on vacation i think
<kbroulik> yup
<Sho_>
<Sho_> * [Wayland] Virtual desktops progress:
<Sho_> ** "Switch Desktop" containment action plugin ported to libtaskmanager, bugs fixed
<Sho_> ** "Switch Window" containment action updated to new libtaskmanager API, bugs fixed
<Sho_> Since then working on making the prototype virtual desktops KCM better (e.g. not instant apply)
<Sho_> basically i'm trying to get the wayland virtual desktop stuff complete enough for an early-cycle 5.15 merge
<Sho_> what this will eventually need:
<Sho_> * completing the above KCM work, and subsequently VDG input
<Sho_> * more code reviews, e.g. my libtm patch hasn't been reviewed at all so far
<Sho_> * i think there's outstanding review and maybe coding on marco's code too (I recall experiencing a recurring crash)
<notmart_> aw, crap
<notmart_> yeah, i'm here :)
<Sho_> * figuring out what to do with the windows krunner plugin, which needs porting as well, but has the problem of being not in the plasmahsell process and needing the exclusive wayland protocols
<kbroulik> fvogt: meeting time, unless munching :P
<Sho_> we should really try to merge this early in 5.15, because it is very likely going to need a lot of testing and bugfixing, and since we originally wanted this to be in 5.14 it'd suck a lot if it misses out another cycle
<kbroulik> Sho_: maybe dbusrunner can help, plasmashell exposing that stuff to krunner, but we must think about (not neccessarily address right now but have that in mind) security implications
<fvogt> kbroulik: Not munching yet - I'm at my parents as the DB outage didn't let me get home...
<Sho_> kbroulik: I think a more pragmatic solution would be adopting the KCM's "only in=x11" plugin metadata thing to disable the windows runner on Wayland for now
<Sho_> kbroulik: And then asking KWin to figure out the auth thing before long
<Sho_> also, we need to poke Riddell to set up a 5.15 kickoff meeting, but this was important t odiscuss now already
<notmart_> Sho_, kbroulik.. yeah
<romangg> In regards to not instant apply in the VD KCM: Is this alligned with what the VDG wants? There were several discussions in the past about (not) instant apply
<kbroulik> even if we were to change that (which I'm not sainyg we should) we must be consistent and not have a single new KCM randomly beahve differently
<Sho_> romangg: I think the VDG wants instant-apply, but it's not been decided yet and we can't just make individual KCMs step out of line randomly
<notmart_> it should use correctly the api needs apply, load, save
<Sho_> I agree with kbroulik
<notmart_> once it uses it really correctly, making the shell doing apply to save or instant apply is a switch
<Sho_> but the KCM was a prototype so far meant to help Marco mainly, and the code gets a lot more complex with delayed-apply, so i delayed the delayed-apply :D
<notmart_> unless the kcm does things by itself, then is a problem
<Sho_> anyway, all fro mme
<kbroulik> alright, Riddell
<Sho_> so please help out with those reviews, etc.
<Riddell> Plasma 5.12.7 LTS is due tomorrow
<Riddell> I know everyone will be very excited
<notmart_> i hope i'll have some plasma time this week... we'll see
<Riddell> of course if there's no changes we can ignore it but I'll look at the changelog and release if there's useful bits
<Riddell> and Kickoff meeting, shall we have one? I'm in BCN next week so my time may be limited for chairing
<kbroulik> I did a couple of 5.12 fixes over passt weeks/months :)
<Sho_> I think I backported one or two, too
<Riddell> actually wait, we shouldn't talk about Kickoff meeting until 5.14 is out
<Sho_> yeah, usually we do it shortly after the release, right?
<Riddell> so that's fine two weeks to go
<Sho_> (speaking of which, please let me know if you know of any 5.14 blockers that need handling)
<Riddell> next()
<kbroulik> 5.14.0 is due 9 October?
<kbroulik> just make sure it's before 26 October, so we should do it somewhere inbetween
<Sho_> and don't make it oct 16/17, i have some stuff there :D
<kbroulik> we can have a doodle :P but just keep in mind the 26 :D
<kbroulik> and the weeks thereafter are basically occupied
<Sho_> pls, cool kids have a framadata instead of a doodle these days
<Sho_> meeting overlord, who's next?
<kbroulik> alright, romangg
<romangg> What I did since last meeting:
<romangg> * Rework of input redirection: D15595
<sKreamer> Diff 15595 "Rework InputDeviceHandler focus tracking" [Needs Review] https://phabricator.kde.org/D15595
<romangg> * Rebased my Wayland touch drag code onto this: D15466
<sKreamer> Diff 15466 "Add Wayland touch drag and drop support" [Needs Review] https://phabricator.kde.org/D15466
<romangg> * My Xwayland DND patch series is now complete with addition of D15627 D15628 D15629.
<sKreamer> Diff 15627 "[xwl] Drag and drop between Xwayland and Wayland native clients" [Needs Review] https://phabricator.kde.org/D15627
<sKreamer> Diff 15628 "[xwl] Support stack optimizing X drag source clients" [Needs Review] https://phabricator.kde.org/D15628
<sKreamer> Diff 15629 "[xwl] text/x-uri converter for selected X url list format targets" [Needs Review] https://phabricator.kde.org/D15629
<romangg> My plan for this week:
<romangg> * XDC 2018.
<romangg> * Respond to reviews.
<romangg> * Look into some issue in Xwayland's Present code discussed on xorg-devel.
<romangg> Questions and requests:
<romangg> * Nothing at the moment. I need my patch series to be reviewed, but I assume for that I have to wait on d_ed to return and when he finds time.
<romangg> Any questions?
<romangg> If you want to test the patch series: this would help of course too.
<Sho_> romangg: So with the KRunner plugin thing we ran into the issue that it needs to use PlasmaWindowManagement, but that can only be used once right now by plasmashell
<Sho_> martin floeser had a plan there for some sort of auth system etc.
<Sho_> that's probably something to take up again in the mid-term future
<romangg> Do we really have this restriction in place already that it can only be used once?
<notmart_> the globals? not sure
<romangg> The question is if we really want to have auth via Wayland protocol at all. Nothing materialized in this direction anyways with our competitors/partners.
<romangg> So DBus only it might be in the end.
<notmart_> but exposing window tasks via dbus, kinda misses the point
<Sho_> yeah
<Sho_> then we have two implementations for the same thing, that's just meh
<Sho_> maybe you can mail martin about what his plans were, and come to your own judgement of them?
<kbroulik> breakout :)
<kbroulik> notmart_:
<notmart_> ok
<notmart_> had to work mostly on $otherstuff, tough i did some work just on kirigami, not really plasma stuff, which is:
<notmart_> * fix ripple effect for SwipeListItem on Material style
<notmart_> * enhancements in Card:
<notmart_> ** possible to control the sourceSize and all Image proeprties for the header image
<notmart_> ** possible to control wrapMode and size for the title
<notmart_> * fix icons:
<notmart_> ** fix a regressions of icons that became smaller on mobile
<notmart_> ** improve the heuristics on what icons should be monochrome and what shouldn't
<kbroulik> "ripple effect"?
<notmart_> questions/next
<kbroulik> ah Material, nevermind, I know what you mean
<notmart_> yeah, the stupid thing on android when you press controls
<kbroulik> :D
<kbroulik> alright, me then
<kbroulik> Work done:
<kbroulik> - Made "Open..." action for finished copy/downloads also work in history
<kbroulik> - Added Open Document thumbnailer (odt, ods, odp, etc)
<kbroulik> - Fixed device actions not udpating live in Device Notifier
<kbroulik> - Fixed Discover not showing update notification (also in 5.12)
<kbroulik> - Made a patch for Krita to set decoration colorscheme: D15640
<kbroulik> - Raise configuration window when reusing it: D15705
<kbroulik> - Fixed (worked around) Ark opening Open Document files (that are *technically* archives) as archives
<sKreamer> Diff 15640 "Set color scheme hint for compositor" [Accepted] https://phabricator.kde.org/D15640
<sKreamer> Diff 15705 "Also raise configuration window when reusing it" [Needs Review] https://phabricator.kde.org/D15705
<kbroulik> Work todo:
<kbroulik> - Mostly company stuff
<kbroulik> if there are no further questions I'll call this a wrap
<kbroulik> thanks everyone for attending :)
-*- fvogt is still here
<kbroulik> ok, go on fvogt
<fvogt> Someone asked this weekend why plasma-pa had only the contents.rcc file anymore, apparently it's indeed annoying for some "code editing users"
<fvogt> I had a look at that file and noticed that it's not compressed at all - so why are bundled packages still a thing?
<kbroulik> we had experimented with using qrc for packaging plasmoids but ended up not using it, perhaps was forgotten to remove again?=
<notmart_> yeah, can be removed
<notmart_> i still think it should be used, but ooooh, well
<kbroulik> "had only the contents"
<fvogt> Would it be possible to just alias the _bundled variant in kpackage to the non-bundled one?
<kbroulik> so something's wrong in cmake?
<notmart_> simply in plasma-pa is using the cmake macro to do bundled packages, should be changed that
<notmart_> one commit to revert
<kbroulik> we can figure it out :) close meeting?
