Re: Re: VDG suggestions and wishes about the system tray

2014-08-27 Thread Martin Gräßlin
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 playlist has ended, then there's no

Re: Review Request 118914: [klipper] Fix memory leaks

2014-08-27 Thread Martin Gräßlin
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/118914/ --- (Updated Aug. 27, 2014, 8:56 a.m.) Status -- This change has been

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

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

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

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 application

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.

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

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

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,

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 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.

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 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: 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 https://git.reviewboard.kde.org/r/119866/diff/5/?file=307632#file307632line55 Math.round(…) ? We guarantee units.gridUnit to be an even number, so for /2, no

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

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 https://git.reviewboard.kde.org/r/119943/diff/3/?file=307797#file307797line43 this is already shipped? your diffs are a bit messed up. eww, yeah, doing diff from the wrong checkout On Aug. 27,

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 still they're

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 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

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.

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 https://git.reviewboard.kde.org/r/119943/diff/3/?file=307792#file307792line66 if geom is in valid we should setGeometry based on the applets implicit size. We

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,

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

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

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

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 https://git.reviewboard.kde.org/r/119927/#fcomment261 I actually changed this string after the screenshot to Use circular monitors, but you're right. Radio buttons or

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 -

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 https://git.reviewboard.kde.org/r/119927/#fcomment263 As already said in the forum thread: This violates the alignment HIG. Please put the checkboxes on the left.

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

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

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: 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

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