Review Request: Add a tooltip to battery monitor applet

2009-10-29 Thread Kåre Särs
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2010/ --- Review request for Plasma. Summary --- This path adds a tool-tip for the ba

Re: Review Request: Add a tooltip to battery monitor applet

2009-10-30 Thread Kåre Särs
" would have been a good enough compromise. If I want a battery applet that shows the _estimated_ remaining time without needing to edit a configuration file, that is by the way very hard to find and edit, should I make another one in another place? - Kåre

Re: Review Request: Add a tooltip to battery monitor applet

2009-11-01 Thread Kåre Särs
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2010/ --- (Updated 2009-11-01 08:04:09.930792) Review request for Plasma. Changes --

Re: Review Request: Add a tooltip to battery monitor applet

2009-11-01 Thread Kåre Särs
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2010/ --- (Updated 2009-11-01 09:34:16.376436) Review request for Plasma. Changes --

Re: Review Request: Add a tooltip to battery monitor applet

2009-11-25 Thread Kåre Särs
http://reviewboard.kde.org/r/2010/#review3267 --- On 2009-11-01 09:34:16, Kåre Särs wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://reviewboard.kde.org/r/2010/ >

Re: Review Request: Add a tooltip to battery monitor applet

2009-11-25 Thread Kåre Särs
nges. - Kåre --- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2010/#review3276 --- On 2009-11-01 09:34:16, Kåre Särs wrote: > > --

Re: Review Request: Add a tooltip to battery monitor applet

2009-11-26 Thread Kåre Särs
> On 2009-11-25 17:27:53, Cyberbeat wrote: > > I really like the estimated time. And it really helps. And for not being > > precise, it is "estimated" :) Would really like to see this in kde-4.4, and > > perhaps also backported to kde-4.3? > > Kåre Särs wr

Re: Review Request: Add a tooltip to battery monitor applet

2009-11-26 Thread Kåre Särs
On Thursday 26 November 2009, Chani wrote: > > > > I disagree about the usefulness of time-remaining, but since it is not my > > code I'll let it be. > > > > That said, would I get objections if I would fork "Battery Monitor" into > > a "Battery Time" and then later push it to kdeplasma-addons? >

Re: plasma memory/pixmap leak ?

2010-06-23 Thread Kåre Särs
own to the Systemloadviewer. The only problem is that I can not see a any commits in the history log of the applet that could result in this memory leak compared to KDE SC 4.4 -- Kåre Särs -- Kåre ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Behavior on systray icon clicking

2012-09-25 Thread Kåre Särs
On Tuesday 25 September 2012 12:01:17 Aaron J. Seigo wrote: > On Tuesday, September 25, 2012 11:42:40 Ivan Čukić wrote: > > This replicates the behaviour of tasks applet - think of > > click-to-minimise. > > .. and why appliation tray icons really ought to be merged with the tasks > applet. I h

Re: Behavior on systray icon clicking

2012-09-25 Thread Kåre Särs
On Tuesday 25 September 2012 12:56:40 Aaron J. Seigo wrote: > On Tuesday, September 25, 2012 13:33:11 Kåre Särs wrote: > > I have spacial memory > > this is a basic part of human cognitive abilities. even toddlers who can > only crawl exhibit it :) What I mean is that people a

Review Request 125858: Fix SystemLoadViewer freezing plasmashell

2015-10-29 Thread Kåre Särs
. The limitation of the value to max 1, prevented freezes. Thanks, Kåre Särs ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 125858: Fix SystemLoadViewer freezing plasmashell

2015-10-29 Thread Kåre Särs
On Oct. 29, 2015, 7:20 a.m., Kåre Särs wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/125858/ > -

Re: Review Request 125858: Fix SystemLoadViewer freezing plasmashell

2015-10-29 Thread Kåre Särs
marked as submitted. Review request for Plasma, David Edmundson and Martin Yrjölä. Changes --- Submitted with commit 351c2a03827040d148a1b73dbeab12eb3d3e69de by Kåre Särs to branch master. Repository: kdeplasma-addons Description --- The SystemLoadViewer freezes plasmashell when

Re: Review Request 125858: Fix SystemLoadViewer freezing plasmashell

2015-10-29 Thread Kåre Särs
view87660 --- On Oct. 29, 2015, 12:18 p.m., Kåre Särs wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewb

Re: Review Request 125006: Add individual monitor for each cpu option for system load viewer

2015-11-02 Thread Kåre Särs
contents/config/main.xml 0edfd89 applets/systemloadviewer/package/contents/ui/GeneralSettings.qml a577b1a applets/systemloadviewer/package/contents/ui/SystemLoadViewer.qml 3a805b5 Diff: https://git.reviewboard.kde.org/r/125006/diff/ Testing --- Running it on my own computer ;) Tha

Re: Review Request 125006: Add individual monitor for each cpu option for system load viewer

2015-11-02 Thread Kåre Särs
marked as submitted. Review request for Plasma, David Edmundson, Dominik Haumann, and Martin Yrjölä. Changes --- Submitted with commit 1c0cf92208b8a7589e1d6103f3fd5640840509c8 by Kåre Särs to branch master. Repository: kdeplasma-addons Description --- When compact-bar monitos are

Re: robots.txt in quickgit.kde.org

2015-12-29 Thread Kåre Särs
On Tuesday, December 29, 2015 10:39:01 PM Ben Cooksley wrote: > On Tue, Dec 29, 2015 at 7:59 AM, Lydia Pintscher wrote: > > On Sun, Dec 27, 2015 at 12:35 PM, Ben Cooksley wrote: > >>> Is there some place where search engines can easily index our source > >>> code or are we shooting ourselves in t

Re: Review Request 127325: Replace all keywords by Qt macros

2016-03-10 Thread Kåre Särs
tps://git.reviewboard.kde.org/r/127325/#comment63669> Is this a mistakenly included addition? - Kåre Särs On March 10, 2016, 8:42 a.m., Jan Grulich wrote: > > --- > This is an automatically generated e-mail. To reply,

Re: Review Request: Tasks applet: Make order independent of whether the row count is forced.

2011-01-10 Thread Kåre Särs
--- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/5776/#review9601 --- There is a problem with the new behavior for forced row count. Pr

Review Request 123738: Use column major in the taskbar when "Force row settings" is set

2015-05-12 Thread Kåre Särs
ns and tried the vertical layout Thanks, Kåre Särs ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 123738: Use column major in the taskbar when "Force row settings" is set

2015-05-13 Thread Kåre Särs
eviewboard.kde.org/r/123738/diff/ Testing --- I'm using this patch on all my installations and tried the vertical layout Thanks, Kåre Särs ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 123738: Use column major in the taskbar when "Force row settings" is set

2015-05-17 Thread Kåre Särs
onfigGeneral.qml 36dd134 applets/taskmanager/package/contents/ui/main.qml 98ba7c3 Diff: https://git.reviewboard.kde.org/r/123738/diff/ Testing (updated) --- I have applied this patch to the system installed plasma 5.3 installation. Thanks, Kåre Särs __

Re: Review Request 123738: Use column major in the taskbar when "Force row settings" is set

2015-05-18 Thread Kåre Särs
General.qml 36dd134 applets/taskmanager/package/contents/ui/main.qml 98ba7c3 Diff: https://git.reviewboard.kde.org/r/123738/diff/ Testing --- I have applied this patch to the system installed plasma 5.3 installation. Thanks, Kåre Särs ___ Pl

Re: Review Request 123738: Use column major in the taskbar when "Force row settings" is set

2015-05-20 Thread Kåre Särs
marked as submitted. Review request for Plasma and Eike Hein. Changes --- Submitted with commit c685b9ce1f0d567b2c2c2e27a546506d29b41114 by Kåre Särs to branch master. Repository: plasma-desktop Description --- When we have "Force row settings" and more than one row of

Review Request 124956: Prevent the lockscreen password field from loosing focus

2015-08-27 Thread Kåre Särs
s not gone anywhere this time. Thanks, Kåre Särs ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 124956: Prevent the lockscreen password field from loosing focus

2015-08-28 Thread Kåre Särs
l. To reply, visit: https://git.reviewboard.kde.org/r/124956/#review84515 --- On Aug. 27, 2015, 6:53 p.m., Kåre Särs wrote: > > --- > This is an automatic

Re: Review Request 124956: Prevent the lockscreen password field from loosing focus

2015-08-28 Thread Kåre Särs
play alarm, lock the screen and wait for the minute to change and notice that the focus has not gone anywhere this time. Thanks, Kåre Särs ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Review Request 125006: Add individual monitor for each cpu option for system load viewer

2015-08-31 Thread Kåre Särs
number of cpus are over 4 (6?). Thanks, Kåre Särs ___ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel

Re: Review Request 125006: Add individual monitor for each cpu option for system load viewer

2015-09-17 Thread Kåre Särs
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125006/#review85603 --- Ping? - Kåre Särs On Aug. 31, 2015, 8:13 p.m., Kåre Särs

Re: Review Request 125006: Add individual monitor for each cpu option for system load viewer

2015-10-09 Thread Kåre Särs
/systemloadviewer/package/contents/ui/SystemLoadViewer.qml 8023b44 Diff: https://git.reviewboard.kde.org/r/125006/diff/ Testing (updated) --- Running it on my own computer ;) Thanks, Kåre Särs ___ Plasma-devel mailing list Plasma-devel@kde.org htt

Re: Review Request 125006: Add individual monitor for each cpu option for system load viewer

2015-10-09 Thread Kåre Särs
, visit: https://git.reviewboard.kde.org/r/125006/#review85629 --- On Oct. 9, 2015, 1:07 p.m., Kåre Särs wrote: > > --- > This is an automatically generated e-mail. T

Re: Review Request 125006: Add individual monitor for each cpu option for system load viewer

2015-10-09 Thread Kåre Särs
ralSettings.qml a577b1a applets/systemloadviewer/package/contents/ui/SystemLoadViewer.qml 8023b44 Diff: https://git.reviewboard.kde.org/r/125006/diff/ Testing --- Running it on my own computer ;) Thanks, Kåre Särs ___ Plasma-devel mail

Re: New idea for Plasma's logo

2016-05-24 Thread Kåre Särs
Hi, Did you notice that the Qt logo has changed? The rounded corners are not round anymore: http://brand.qt.io/guidelines/ /Kåre On måndag 23 maj 2016 kl. 22:52:47 EEST Alessandro Longo wrote: > (resent because apparently mailman rejected it) > Hi, > after a lot of iterations I found a simple i

Re: Review Request 129838: fix no-display of CPU bars per core (and fix some warnings)

2017-01-16 Thread Kåre Särs
What happens if you use connectSource(...) in onNewData in stead of setting all of connectedSources again? - Kåre Särs On Jan. 15, 2017, 2:36 p.m., Martin Koller wrote: > > --- > This is an automatically generate

Re: Review Request 129838: fix no-display of CPU bars per core (and fix some warnings)

2017-06-11 Thread Kåre Särs
6 p.m., Martin Koller wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/129838/ > --- > > (Updated Jan. 15, 2017, 2:36 p.m.)

Re: Review Request 129838: fix no-display of CPU bars per core (and fix some warnings)

2017-06-11 Thread Kåre Särs
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/129838/#review103314 --- Ship it! Ship It! - Kåre Särs On Jan. 15, 2017, 2:36