There is an existing overload that accepts a string and interprets it as
HTML, so that's not an option. And asking Konstantin to add a new
addItemText(String) method seems too far outside the scope of this patch.

On Wed, Jan 5, 2011 at 10:02 AM, John Tamplin <j...@google.com> wrote:

> On Wed, Jan 5, 2011 at 12:47 PM, Ray Ryan <rj...@google.com> wrote:
>
>> Taking this a bit further, if we're going to be playing with interfaces we
>> might as well go the whole nine yards:
>>
>> interface IsTreeItem {
>>   TreeItem asTreeItem();
>> }
>>
>> interface HasTreeItems {
>>   void addItem(IsTreeItem);
>>   void addItem(Widget);
>>   void addItem(SafeHtml);
>>   /* No addItem(String), it's unsafe */
>>
>
> Taking a string is fine, but it should be interpreted as plain text not
> HTML..
>
> --
> John A. Tamplin
> Software Engineer (GWT), Google
>
> --
> http://groups.google.com/group/Google-Web-Toolkit-Contributors

-- 
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Reply via email to