[rfc][icedtea-web][itweb-settings] Improve Icedtea-Web cache disk space

[Lukasz Dracz]

Hello,

> Here I go again:
> (and again, its just my humble opinion).
> 
> With the current labels:
>     If limit disabled->no limit
>     If limit enabled with value of 0 -> no cache, therefor
> compression+location+view (ie: all components) must be disabled.
> Default could be limit disabled.

I like this suggestion, but I did not disable view files so the user can still check the files that will be deleted without having to change the value to check. 

> Aesthetics: "WARNING" and warning-message could be on the same line to
> prevent ugly scrolling.
> Maybe warning message would be clearer saying: "WARNING: Using more than
> xxxx MB availabe"

Okay, I had to move the warning label under the other label instead of the spinner since the label is so long it would extend the panel to the right causing the user to have to scroll to the right on the default size.

> Can view be right after change? (in the same line)

Good idea, done.

> I loved the old-school powers of 2 :P
> Holidays, as usual, where short ;)

I put it back :) , I like it too.

> Thanks for the (IMHO) improvements.

Thank you for the reviews !

Regards,
Lukasz Dracz
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cacheSizeSpinner-12.patch
Type: text/x-patch
Size: 20499 bytes
Desc: not available
URL: <http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20140827/47920a96/cacheSizeSpinner-12-0001.patch>