[icedtea-web] RFC: extend "itweb-settings reset" to accept "all"

[Omair Majid]

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]:
>>>> Hi,
>>>>
>>>> The attached patch makes "itweb-settings reset all" reset the entire
>>>> configuration.
>>>>
>>>> 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
>>> switch.
>>>
>>
>> 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?

Thanks,
Omair
-------------- next part --------------
A non-text attachment was scrubbed...
Name: reset-all-settings-gui-01.patch
Type: text/x-patch
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