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

Lukasz Dracz ldracz at redhat.com
Thu Aug 28 15:47:14 UTC 2014


Hello,

> Roll, Roll, Roll the Ball...
> 
> 
> Im on the opinion that disabled/unusable components should be hidden. Is
> this against any usability/accessibility guide? Let me show you some
> examples:
> 
> -----
> limit disabled                      (ie: full-cache)
>                                          (Notice spinner/warning is hidden)
> jar-compresion-level
> location | change | view
> -----
> 
> -----
> limit=0
> "No files will be cached. Cached files will be deleted."      (files on
> cache should be automatically deleted when changes applied? I Think so)
>                                          (Notice I have changed the text to
> a -IMHO- more friendly one)
>                                          (Notice
> compression/location/change/files are hidden)
> 
> -----
> 
> -----
> limit=x
> Warning ONLY if needed
> jar-compresion-level
> location | change | view
> -----

Okay :), yeah it does look cleaner.
 
> I would remove titledborders (disk/location) and put everything under main
> panel. Trying to KISS.

Alright.

> Thank you for the changes, hope you dont hate me (yet) for being so
> critical.
> I'm sure you have much more important fixes to do, so don't hesitate to
> stop/resolve when you think so ;)

Not at all, it is great to be getting all this feedback :)
And I think we have made great improvements to the Cache Panel thanks to your suggestions and jacobs.
I have attached the updated patch.

Thanks,
Lukasz Dracz 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: cacheSizeSpinner-14.patch
Type: text/x-patch
Size: 22357 bytes
Desc: not available
URL: <http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20140828/34da34ac/cacheSizeSpinner-14-0001.patch>


More information about the distro-pkg-dev mailing list