suggestion after updated Pahro and Seaside

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

suggestion after updated Pahro and Seaside

rshiple

I would like to suggest that


WATagBrush
>>attributes
   ^ attributes ifNil: [ attributes := WAHtmlAttributes new ]


Should be


WATagBrush
>>attributes
    ^ attributes ifNil: [ attributes := WATagAttributes new ]


And then I can handle in my WACurlTagBrush with WACurlOptions and

WAHtmlTagBrush can use WAHtmlAttributes


Curl options are very much like HTML attributes but not entirely


_______________________________________________
seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
Reply | Threaded
Open this post in threaded view
|

Re: suggestion after updated Pahro and Seaside

Lukas Renggli
> WATagBrush>>attributes
>     ^ attributes ifNil: [ attributes := WATagAttributes new ]

True, please create a bug report.

Lukas

--
Lukas Renggli
http://www.lukas-renggli.ch
_______________________________________________
seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside
Reply | Threaded
Open this post in threaded view
|

Re: suggestion after updated Pahro and Seaside

Philippe Marschall
2009/9/18, Lukas Renggli <[hidden email]>:
>> WATagBrush>>attributes
>>     ^ attributes ifNil: [ attributes := WATagAttributes new ]
>
> True, please create a bug report.

http://code.google.com/p/seaside/issues/detail?id=472

What should WATagAttributes look like? Direct subclass of
WASmallDictionary with #at:append: and #at:append:separator:?

Cheers
Philippe
_______________________________________________
seaside mailing list
[hidden email]
http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside