[icedtea-web] RFC: extend "itweb-settings reset" to accept "all"
omajid at redhat.com
Tue Mar 1 09:24:38 PST 2011
On 02/28/2011 05:15 PM, Deepak Bhole wrote:
> * Omair Majid<omajid at redhat.com> [2011-02-28 17:11]:
>> On 02/28/2011 04:54 PM, Deepak Bhole wrote:
>>> * Omair Majid<omajid at redhat.com> [2011-02-28 16:32]:
>>>> The attached patch makes "itweb-settings reset all" reset the entire
>>>> Okay to commit?
>>> I think we should have a GUI button for reset all, rather than a
>>> command-line switch. If a GUI button already exists, we don't need a CL
>> This is just extending the existing command line (try
>> "itweb-settings help" to see more info about it). Currently we have
>> "itweb-settings reset<property_name>" which resets just one
>> property; this patch makes reset work with "all" instead of a
>> property name too.
>> I can certainly add something to the GUI, but that would be another patch.
> Please do.
The attached patch adds a GUI method to accomplish this. However, I am
not sure if adding this is needed (the change in command line from reset
foo to reset all is simple to use/understand, but there is no analogue
in the GUI) . I certainly dont think adding it in a new tab is the right
place. Any ideas?
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 6730 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20110301/10fec7f0/reset-all-settings-gui-01.patch
More information about the distro-pkg-dev