[Breeze] [Bug 363602] Symlinked SVG files fails to load under Windows

2016-05-28 Thread Jasem Mutlaq via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=363602 --- Comment #3 from Jasem Mutlaq --- draw-star would be available when KStars is running under a non KDE environment and using a different theme. It's always good to fallback to the icons shipped by KStars. At any rate, this

Review Request 128040: [shellcorona] Containment can be nullptr

2016-05-28 Thread Anthony Fieroni
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128040/ --- Review request for Plasma, Aleix Pol Gonzalez and David Edmundson. Bugs:

Re: Review Request 128038: [libtaskmanager] Stop highlighted window effect when close window in group item

2016-05-28 Thread Anthony Fieroni
> On Май 28, 2016, 10:35 след обяд, Eike Hein wrote: > > The description is insufficient. If it works, why is this necessary? What > > differences does it make? What is it trying to achieve? > > Anthony Fieroni wrote: > Activate works because of kwin, closing not works. It is correct to

[Breeze] [Bug 363602] Symlinked SVG files fails to load under Windows

2016-05-28 Thread andreas via KDE Bugzilla
https://bugs.kde.org/show_bug.cgi?id=363602 andreas changed: What|Removed |Added CC||kain...@gmail.com --- Comment #2

Re: Review Request 128038: [libtaskmanager] Stop highlighted window effect when close window in group item

2016-05-28 Thread Eike Hein
> On May 28, 2016, 7:35 p.m., Eike Hein wrote: > > The description is insufficient. If it works, why is this necessary? What > > differences does it make? What is it trying to achieve? > > Anthony Fieroni wrote: > Activate works because of kwin, closing not works. It is correct to me, >

Re: plasma-desktop on other environments (bis)

2016-05-28 Thread René J . V . Bertin
Martin Klapetek wrote: > It's gone completely, everything happens in KNotification > at runtime. Great. That's 1 daemon less, and also 1 less source of focus loss (and 1 less patch I can now really forget about) :) I'd completely forgotten about the thing, until earlier today when apparently

Re: plasma-desktop on other environments (bis)

2016-05-28 Thread René J . V . Bertin
Martin Klapetek wrote: > That's not what I'm telling anyone at all. Great. It's not always clear how literally to take statements (or not, apparently :)) I think I don't need convincing for the rest but always agreed with that. > But if you're going to split events out from

Re: Review Request 128038: [libtaskmanager] Stop highlighted window effect when close window in group item

2016-05-28 Thread Anthony Fieroni
> On Май 28, 2016, 10:35 след обяд, Eike Hein wrote: > > The description is insufficient. If it works, why is this necessary? What > > differences does it make? What is it trying to achieve? Activate works because of kwin, closing not works. It is correct to me, that code to present. So if

Re: Review Request 128038: [libtaskmanager] Stop highlighted window effect when close window in group item

2016-05-28 Thread Eike Hein
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128038/#review95957 --- The description is insufficient. If it works, why is this

Review Request 128038: [libtaskmanager] Stop highlighted window effect when close window in group item

2016-05-28 Thread Anthony Fieroni
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128038/ --- Review request for Plasma and Eike Hein. Repository: plasma-desktop

Re: Review Request 128030: [libtaskamanger] Dismiss context menu when window is closed

2016-05-28 Thread Anthony Fieroni
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128030/ --- (Updated May 28, 2016, 6:48 p.m.) Status -- This change has been

Re: Review Request 128021: [libtaskmanager] Prevent from dereferencing dangling pointer

2016-05-28 Thread Anthony Fieroni
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128021/ --- (Updated May 28, 2016, 6:36 p.m.) Status -- This change has been

Re: plasma-desktop on other environments (bis)

2016-05-28 Thread Martin Klapetek
On Sat, May 28, 2016 at 2:22 PM, René J. V. wrote: > Quick related question: has knotify4 been replaced by a KF5 equivalent, or > rather made obsolete? > It's gone completely, everything happens in KNotification at runtime. Cheers -- Martin Klapetek | KDE Developer

Re: plasma-desktop on other environments (bis)

2016-05-28 Thread Martin Klapetek
On Sat, May 28, 2016 at 11:14 AM, René J. V. wrote: > > > Please don't make our apps aliens in other environments. > > Please don't tell users of other environments what they're allowed to do > with > their apps either. > That's not what I'm telling anyone at all. I'm

Re: plasma-desktop on other environments (bis)

2016-05-28 Thread René J . V . Bertin
Quick related question: has knotify4 been replaced by a KF5 equivalent, or rather made obsolete? ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: plasma-desktop on other environments (bis)

2016-05-28 Thread René J . V . Bertin
David Edmundson wrote: > Renè isn't doing the individual standalone OS X packages, which should have > the tight host integration. Other people are doing that. MacPorts makes it much more straightforward to provide cross-platform homogeneity because all required resources are installed in

Re: plasma-desktop on other environments (bis)

2016-05-28 Thread René J . V . Bertin
David Edmundson wrote: >> Where and how exactly are the default sounds configured? > .notifyrc files shipped with the application / library. That would mean changing lots of files then... I'm not sure how to do that in an efficient way if there isn't something like a category mechanism in

Re: Review Request 128033: Add kcon to systemsettings

2016-05-28 Thread Burkhard Lück
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128033/ --- (Updated May 28, 2016, 3:09 p.m.) Status -- This change has been

Re: plasma-desktop on other environments (bis)

2016-05-28 Thread Martin Klapetek
On Sat, May 28, 2016 at 10:27 AM, David Edmundson < da...@davidedmundson.co.uk> wrote: > On Sat, May 28, 2016 at 3:11 PM, Martin Klapetek < > martin.klape...@gmail.com> wrote: > >> On Sat, May 28, 2016 at 9:10 AM, Kai Uwe Broulik >> wrote: >> >>> >>> > Even more so than

Re: plasma-desktop on other environments (bis)

2016-05-28 Thread Kai Uwe Broulik
> He's doing macports which is *very* different. Macports even has X11 and > Fluxbox. > If you're running kate on X with a fluxbox WM, having native OS X integration > just because of your kernel doesn't make too much sense. Heh, so indeed we talked past each other, I'm out of this discussion

Re: plasma-desktop on other environments (bis)

2016-05-28 Thread David Edmundson
On Sat, May 28, 2016 at 3:11 PM, Martin Klapetek wrote: > > On Sat, May 28, 2016 at 9:10 AM, Kai Uwe Broulik > wrote: > >> >> > Even more so than with look and feel that will be beneficial for >> cross-platform users. After all alert sound

Re: plasma-desktop on other environments (bis)

2016-05-28 Thread Martin Klapetek
On Sat, May 28, 2016 at 9:10 AM, Kai Uwe Broulik wrote: > > > Even more so than with look and feel that will be beneficial for > cross-platform users. After all alert sound specificity is supposed to aid > in determining what's going on and how to react. > > If I hear the

Re: Proposal: kde-libs.org, a repository of contributed libraries

2016-05-28 Thread kdea
I'm copying here the message I have sent to opendesktop.org ... Hi, I rencently published a new version of my "Take a break" plasmoid at http://kde-apps.org/ . The plasmoid contains a small reusable library that could be useful for other developers, but I didn't know where to upload the library

Re: plasma-desktop on other environments (bis)

2016-05-28 Thread David Edmundson
> > Where and how exactly are the default sounds configured? > .notifyrc files shipped with the application / library. David ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: plasma-desktop on other environments (bis)

2016-05-28 Thread Kai Uwe Broulik
Hi, > I guess there must be a generic Qt beep that one can tap into to replace > beep,  QApplication::beep() > Given that distinction I would find it confusing to use the same sound for > both empty actions. I rather find this distinction awkward and would expect emptying the trash empty

Re: plasma-desktop on other environments (bis)

2016-05-28 Thread René J . V . Bertin
Martin Klapetek wrote: So the apparently generic categories that concern application error, crash etc. do not control the way those notifications are delivered if you aren't running a Plasma session? Mind you, we're not only talking about the sounds here. > That's not entirely correct -

Plasma 5.7 Repo Freeze this Thursday

2016-05-28 Thread David Edmundson
If any repos need to be added/removed split. Let me know ASAP A reminder of the other upcoming freezes and releases Thu 2016-06-02 - Repo freeze Thu 2016-06-16 - Beta tag and release Thu 2016-06-30 - Final release gets tagged David ___

Re: Review Request 128021: [libtaskmanager] Prevent from dereferencing dangling pointer

2016-05-28 Thread Eike Hein
> On May 28, 2016, 12:36 p.m., Eike Hein wrote: > > Ship It! Since it won't /hurt/ and this code is on the way out anyway, why not .. - Eike --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 128021: [libtaskmanager] Prevent from dereferencing dangling pointer

2016-05-28 Thread Eike Hein
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128021/#review95944 --- Ship it! Ship It! - Eike Hein On May 28, 2016, 7:23

Re: Review Request 128021: [libtaskmanager] Prevent from dereferencing dangling pointer

2016-05-28 Thread Anthony Fieroni
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128021/#review95942 --- Eike, did you agree it? - Anthony Fieroni On Май 28,

Re: Review Request 128030: [libtaskamanger] Dismiss context menu when window is closed

2016-05-28 Thread Anthony Fieroni
> On Май 28, 2016, 3:17 след обяд, Kai Uwe Broulik wrote: > > Meh. I thought I tested it thoroughly enough when switching from > > QMetaObject::invokeMethod to QTimer::singleShot(0 ... > > > > Isn't it sufficient to just cache the QQuickWindow pointer in a variable > > and pass that into the

Re: Review Request 128030: [libtaskamanger] Dismiss context menu when window is closed

2016-05-28 Thread Eike Hein
> On May 28, 2016, 12:16 p.m., Eike Hein wrote: > > Hmm ok ... note this will likely never ship though, in the applet ported to > > the new libtaskmanager the context menu is in QML and I think we've done > > the last 5.6.x already ... > > Kai Uwe Broulik wrote: > There can still be a

Re: Review Request 128030: [libtaskamanger] Dismiss context menu when window is closed

2016-05-28 Thread Kai Uwe Broulik
> On Mai 28, 2016, 12:16 nachm., Eike Hein wrote: > > Hmm ok ... note this will likely never ship though, in the applet ported to > > the new libtaskmanager the context menu is in QML and I think we've done > > the last 5.6.x already ... There can still be a 5.6.5 and I'd insist on having one

Re: Review Request 128030: [libtaskamanger] Dismiss context menu when window is closed

2016-05-28 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128030/#review95938 --- Meh. I thought I tested it thoroughly enough when switching

Re: Review Request 128030: [libtaskamanger] Dismiss context menu when window is closed

2016-05-28 Thread Eike Hein
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128030/#review95937 --- Ship it! Hmm ok ... note this will likely never ship

Re: Review Request 128030: [libtaskamanger] Dismiss context menu when window is closed

2016-05-28 Thread Anthony Fieroni
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128030/ --- (Updated Май 28, 2016, 3:07 след обяд) Review request for Plasma, David

Re: Review Request 128033: Add kcon to systemsettings

2016-05-28 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128033/#review95935 --- Ship it! Ship It! - David Edmundson On May 28, 2016,

[Differential] [Closed] D1647: kfontinst: Port to QDialog

2016-05-28 Thread volkov (Alexander Volkov)
This revision was automatically updated to reflect the committed changes. Closed by commit rPLASMADESKTOP20bbf2366ae5: kfontinst: Port to QDialog (authored by volkov). REPOSITORY rPLASMADESKTOP Plasma Desktop CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D1647?vs=3908=4030 REVISION

Jenkins-kde-ci: plasma-workspace master kf5-qt5 » Linux,gcc - Build # 104 - Still Unstable!

2016-05-28 Thread no-reply
GENERAL INFO BUILD UNSTABLE Build URL: https://build.kde.org/job/plasma-workspace%20master%20kf5-qt5/PLATFORM=Linux,compiler=gcc/104/ Project: PLATFORM=Linux,compiler=gcc Date of build: Sat, 28 May 2016 09:56:29 + Build duration: 10 min CHANGE SET Revision

Review Request 128033: Add kcon to systemsettings

2016-05-28 Thread Burkhard Lück
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128033/ --- Review request for Plasma and Christoph Feck. Bugs: 347028

Review Request 128030: [libtaskamanger] Dismiss context menu when window is closed

2016-05-28 Thread Anthony Fieroni
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128030/ --- Review request for Plasma, David Edmundson and Eike Hein. Summary

Re: Review Request 128021: [libtaskmanager] Prevent from dereferencing dangling pointer

2016-05-28 Thread Anthony Fieroni
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128021/ --- (Updated Май 28, 2016, 10:23 преди обяд) Review request for Plasma and