Re: Review Request 119964: Avoid dlopening plasmaadapter's devel symlink

2014-08-27 Thread Hrvoje Senjan
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119964/ --- (Updated Aug. 28, 2014, 7:39 a.m.) Review request for Plasma. Changes -

Review Request 119964: Avoid dlopening plasmaadapter's devel symlink

2014-08-27 Thread Hrvoje Senjan
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119964/ --- Review request for Plasma. Bugs: 335904 http://bugs.kde.org/show_bug.

Re: VDG suggestions and wishes about the system tray

2014-08-27 Thread Philipp Stefan
I think I should maybe clarify a few things here as I feel that my original post has left behind the idea in some that we sought a solution in search for a problem. So what is the problem we are trying to solve? The problem is that currently the system tray icons behave unpredictable to the user

Re: Request: better override functionality in locale settings

2014-08-27 Thread Thomas Pfeiffer
On Wednesday 27 August 2014 11:34:52 Sebastian Kügler wrote: > > In my opinion, this is a very serious usability regression and would > > stop me from using Plasma 5. Are there any plans to change the > > behavior/appearance of this settings applet? > > I don't know what it is, but anytime someome

Re: Review Request 119927: Port systemloadviewer plasmoid

2014-08-27 Thread Martin Yrjölä
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119927/ --- (Updated Aug. 27, 2014, 5:02 p.m.) Status -- This change has been ma

Re: Review Request 119927: Port systemloadviewer plasmoid

2014-08-27 Thread Martin Yrjölä
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119927/ --- (Updated Aug. 27, 2014, 4:49 p.m.) Review request for Plasma and David Ed

Re: Review Request 119927: Port systemloadviewer plasmoid

2014-08-27 Thread Martin Yrjölä
> On Aug. 27, 2014, 4:33 p.m., Thomas Pfeiffer wrote: > > File Attachment: Screenshot - 2014-08-24-212539_505x860_scrot.png > > > > > > As already said in the forum thread: This violates the alignment HIG. > > Please put the checkboxes

Re: Review Request 119927: Port systemloadviewer plasmoid

2014-08-27 Thread Thomas Pfeiffer
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119927/#review65378 --- File Attachment: Screenshot - 2014-08-24-212539_505x860_scrot

Re: Review Request 119927: Port systemloadviewer plasmoid

2014-08-27 Thread Thomas Pfeiffer
> On Aug. 25, 2014, 8 a.m., Martin Yrjölä wrote: > > File Attachment: Screenshot - 2014-08-24-212539_505x860_scrot.png > > > > > > I actually changed this string after the screenshot to "Use circular > > monitors", but you're right. Rad

Re: VDG suggestions and wishes about the system tray

2014-08-27 Thread Eike Hein
On 27.08.2014 15:32, Eike Hein wrote: 1 = Pages 6 & 7 of http://arstechnica.com/apple/2011/07/mac-os-x-10-7/7/ are a good read. Sorry - I meant 7 & 8. ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-dev

Re: VDG suggestions and wishes about the system tray

2014-08-27 Thread Eike Hein
On 27.08.2014 06:02, Philipp Stefan wrote: Hmm, could you give me some examples for applications with status notifiers that handles this that way? I mean, sure applications like Inkscape and LibreOffice won't start up in the same state like when they were closed, however, applications like them

Re: Review Request 119943: plasma windowed and associated runner back

2014-08-27 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119943/ --- (Updated Aug. 27, 2014, 1:06 p.m.) Status -- This change has been ma

Re: Review Request 119943: plasma windowed and associated runner back

2014-08-27 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119943/#review65364 --- Ship it! Ship It! - David Edmundson On Aug. 27, 2014, 12:4

Re: Review Request 119943: plasma windowed and associated runner back

2014-08-27 Thread Marco Martin
> On Aug. 27, 2014, 9:58 a.m., David Edmundson wrote: > > plasma-windowed/plasmawindowedview.cpp, line 66 > > > > > > if geom is in valid we should setGeometry based on the applets implicit > > size. > > > >

Re: Review Request 119943: plasma windowed and associated runner back

2014-08-27 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119943/ --- (Updated Aug. 27, 2014, 12:45 p.m.) Review request for Plasma. Reposito

Re: VDG suggestions and wishes about the system tray

2014-08-27 Thread Eike Hein
On 27.08.2014 10:16, Philipp Stefan wrote: One question remains though, why do users not want KTorrent to take up 22px, but are okay with an indicator for the music player while having the music control plasmoid active too. Just a brief note: I'm not - I disable the media control plasmoid bec

Re: VDG suggestions and wishes about the system tray

2014-08-27 Thread Marco Martin
On Wednesday 27 August 2014, Martin Gräßlin wrote: > My personal suggestion (which won't surprise anyone on this list) is to > move the application status notifiers into the tasks applet. But that's > not a really easy task and would not interact well with the remaining > tasks. From my experience

Re: VDG suggestions and wishes about the system tray

2014-08-27 Thread Marco Martin
On Wednesday 27 August 2014, k...@privat.broulik.de wrote: > don't have players > sit in the tray approach), "Active state" > - KMail 149 unread mails, "Active state" > - KTP online, "Active state" > - KMix, 50% volume, "Active state" > > None of them have the "NeedsAttentionStatus" yet

Re: Review Request 119943: plasma windowed and associated runner back

2014-08-27 Thread Marco Martin
> On Aug. 27, 2014, 9:58 a.m., David Edmundson wrote: > > shell/main.cpp, line 43 > > > > > > this is already shipped? > > your diffs are a bit messed up. eww, yeah, doing diff from the wrong checkout > On

Re: Review Request 119866: Thumbnails in Klipper

2014-08-27 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119866/ --- (Updated Aug. 27, 2014, 10:18 a.m.) Status -- This change has been m

Re: Review Request 119866: Thumbnails in Klipper

2014-08-27 Thread Sebastian Kügler
> On Aug. 26, 2014, 11:36 a.m., Kai Uwe Broulik wrote: > > applets/clipboard/contents/ui/ClipboardItemDelegate.qml, line 55 > > > > > > Math.round(…) ? We guarantee units.gridUnit to be an even number, so for /2,

Re: Review Request 119943: plasma windowed and associated runner back

2014-08-27 Thread David Edmundson
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119943/#review65346 --- plasma-windowed/main.cpp

Re: Request: better override functionality in locale settings

2014-08-27 Thread Sebastian Kügler
Hi Matthew, Thanks for your feedback regarding locale settings. Find some explanations for the changes below. On Tuesday, August 26, 2014 13:04:10 Matthew Ruffalo wrote: > I've been a long-time KDE user, and I must say that Plasma 5 looks > *great*. Everything looks very polished and I'm looking

Re: Review Request 119943: plasma windowed and associated runner back

2014-08-27 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119943/ --- (Updated Aug. 27, 2014, 9:32 a.m.) Review request for Plasma. Repositor

Re: Review Request 119866: Thumbnails in Klipper

2014-08-27 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119866/#review65344 --- Ship it! Ship It! - Martin Gräßlin On Aug. 26, 2014, 1:20

Re: Review Request 119866: Thumbnails in Klipper

2014-08-27 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119866/#review65343 --- Ship it! Ship It! - Kai Uwe Broulik On Aug. 25, 2014, 11:2

Jenkins build is back to stable : plasma-workspace_master_qt5 #784

2014-08-27 Thread KDE CI System
See ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 119943: plasma windowed and associated runner back

2014-08-27 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119943/ --- (Updated Aug. 27, 2014, 8:39 a.m.) Review request for Plasma. Repositor

Re: VDG suggestions and wishes about the system tray

2014-08-27 Thread kde
Hi, If your application has finished its job like e.g. an update notifier but should continue to run in the background, then use the "passive" flag. If you want to misuse the system tray as second task bar, then do not use the passive flag, but let it remain active. And if your applicatio

Re: VDG suggestions and wishes about the system tray

2014-08-27 Thread Philipp Stefan
On 27.08.2014 08:16, Martin Gräßlin wrote: On Wednesday 27 August 2014 06:02:11 Philipp Stefan wrote: If the status notifiers are used like we intend them to, then the "passive" status really does not provide any benefit for the user any more. For example, if a music player idles, because the p

Re: VDG suggestions and wishes about the system tray

2014-08-27 Thread Ivan Čukić
I'm not going to chime in regarding the applications - I tend to agree with both parties - that not everything needs to be shown, yet that people like some of their applications minimized to tray (myself included - I even hide those applications from the task bar). One thing where I fully suppo

Re: Review Request 118406: Notify the user if the location containing the media is inaccessible.

2014-08-27 Thread R.Harish Navnit
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118406/ --- (Updated Aug. 27, 2014, 7 a.m.) Status -- This change has been marke