[rfc][icedtea-web] Show cache size in itweb-settings (PR1604)

[Andrew Azores]

On 12/05/2013 06:10 PM, Jacob Wisor wrote:
> Andrew Azores schrieb:
>> I thought of that too but it seemed like the unit should be pretty 
>> universal on these labels. I'm not opposed to moving it into the 
>> Messages files though.
>
> In fact, "MB" is not an internationally acknowledged unit symbol. The 
> IEC has defined "MiB" for megabyte (2^20 bytes). Btw, "MB" is commonly 
> rather understood as 10^6 bytes.

And for the average end user, which unit are they familiar with? Is the 
average end user going to care so much about the difference between 1 
MiB and 1 MB, or are they going to see "200 MiB" and go around trying to 
figure out what a mebibyte is exactly? If other localizations have their 
own "more colloquial" symbol equivalent to MB, as opposed to MiB, then I 
would also lean toward having the localizations present those colloquial 
symbols as well. Presenting information to the end user is useless at 
best if the information is unintelligible to them. This is the same as 
the discussion on the message to show to users when running signed vs 
unsigned applets - telling users that their signed applets would be run 
with "all Java permissions" is really not as clear or useful to an 
average end user as "unrestricted access to your data," even if the 
former is technically more precise.

> Please read Wikipedia articles on the definition of units of data 
> storage and their proper writing.

I am very well aware of the distinction, thank you.

>
> If this is going to be fixed, I would rather insist to do it right. 
> Either the IEC way with proper ISO number formatting or the localized 
> way with a locale specific unit symbol and number formatting.
> Omair is right, this has to be done correctly, otherwise it is a waste 
> of effort because there are still other locale specific data 
> renderings in IcedTea-Web pending to be fixed.

What number formatting are you talking about? The cache size slider in 
question is limited to the range [-1, 1000]. 1000 is displayed as 
"1,000" if that's along the lines of what you mean. The formatting and 
display of that number is also not handled by the JLabel or the text put 
into it read out of the Messages bundles, so I'd say if the slider's 
presentation is something requiring discussion, it belongs in a new 
discussion thread.

>
>> ----- Original Message -----
>> From: "Omair Majid" <omajid at redhat.com>
>> To: "Andrew Azores" <aazores at redhat.com>
>> Cc: "IcedTea" <distro-pkg-dev at openjdk.java.net>
>> Sent: Thursday, December 5, 2013 4:00:44 PM
>> Subject: Re: [rfc][icedtea-web] Show cache size in itweb-settings 
>> (PR1604)
>>
>> Hi,
>>
>> * Andrew Azores <aazores at redhat.com> [2013-12-05 15:57]:
>>> Very very simple patch here. The Temporary Internet Files Panel
>>> displays a cache size slider, but there's no indication of what
>>> units are in use here. Evidently it's megabytes, so I've added this
>>> unit to the label so that it's actually clear how large the cache
>>> will be allowed to grow.
>>
>>> +        JLabel lCacheSize = new 
>>> JLabel(Translator.R("TIFPCacheSize") + " (MB) :");
>>
>> I think it makes more sense to add the '(MB)' string to the localized
>> text. Not all languages will have (MB) after the cache size string. What
>> do you think?
>>
>> Thanks,
>> Omair
>>
>
>


-- 
Andrew A