[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-panel-devel
Subject:    Re: Review Request: The KSystemActivityDialog uses MB as default unit.
From:       Sebastian =?iso-8859-1?q?K=FCgler?= <sebas () kde ! org>
Date:       2010-11-30 17:05:04
Message-ID: 201011301805.04289.sebas () kde ! org
[Download RAW message or body]

On Tuesday, November 30, 2010 17:08:52 Matthias Fuchs wrote:
> Am Montag 29 November 2010, 23:02:50 schrieb Sebastian Kügler:
> > On Monday, November 29, 2010 22:25:29 Matthias Fuchs wrote:
> > > What do you think on the idea I mentioned above, i.e. not using one
> > > "size" (KB, MB), but those that fit best, like in Dolphin? Actually
> > > only if it is possible from an implementation point of view (is BC/SC
> > > needed here?)
> > 
> > How do you mean "fit best"?
> > 
> > In short, I think consistency trumps everything else.
> 
> Yes that is what I think too, that is why we should maybe make it like
> Dolphin.
> 
> e.g. memory usage of 3 programs (I assume a base of 10 here, as it makes
> the example easier)
> 1) 56 K
> 2) 3,744 K
> 3) 5,019,112 K
> 
> how it is displayed to the user:
> 1) 56 K
> 2) 3.7 M
> 3) 5 G
> 
> What would be used as "K", "M" depends on the global settings, so either
> "MiB" or "MB" would be displayed.
> 
> 
> 
> Alternatively adding seperators to make the numbers more readable could be
> an option:
> firefox  matthias  3% 650,292 K
> vs as it is now
> firefox  matthias  3% 650292 K
> 
> 
> I hope my thoughs are better understandable now. :)

Not really. I don't get why you bother with these units _at_all_. You simply 
shouldn't make up how things are displayed, that's the task of KLocale. Please 
have a look at 
http://api.kde.org/4.x-api/kdelibs-
apidocs/kdecore/html/classKLocale.html#a566a3d868abfca050e1191b16e85ed73
-- 
sebas

http://www.kde.org | http://vizZzion.org | GPG Key ID: 9119 0EF9
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic