[Seaside] Suggested extension for WASelectTag

[jtuchel at objektfabrik.de]

Philippe,



Am 03.07.17 um 18:23 schrieb Philippe Marschall:
> On Sat, Jul 1, 2017 at 7:24 AM, jtuchel at objektfabrik.de
> <jtuchel at objektfabrik.de> wrote:
>> ...
>>
>> Was this available already and did I overlook it?
> Not that I know of.
glad to hear that. Wouldn't be the first time, however ;-)

>> Does this sound reasonable? Worth adding to Seaside?
> I have some issues with the API:
> - Config is an abbreviation, we don't usually use abbreviations
> - we don't usually use the Block suffix for methods that accept
> blocks, e.g.. #callback: not #callbackBlock:, see also #titles:
> (WAIframeTag >> #rootBlock is a bad example)
Yeah, I can live with that. I just couldn't come up with a good name ;-)

> I don't really have a good proposal, maybe #optionRenderer: but the
> renderer wouldn't render the whole option, only customize it.
Well, the name is good as it describes what the block does. Renderer, 
however, is already a name with a meaning in Seaside and it is not a Block.


> As Paul has noted if you want full control you may be better off
> rendering the the options yourself as done in
> WAInputElementContainer>>#renderSingleSelectionCustomOn [1]
I already mentioned it in an answer to Johan: I was missing 
#dispatchCallback, and so couldn't make this work.
However, if we have #labels: et al in WASelectTag, why not have a 
respective API for manipulation options individually.
Nevertheless, I will most likely remove my extension from my local image 
and try the iteration thing now that I know how to get things going with 
dispatchCallback.
>
>   [1] https://github.com/SeasideSt/Seaside/blob/master/repository/Seaside-Tests-Functional.package/WAInputElementContainer.class/instance/renderSingleSelectionCustomOn..st

Thanks for the link. I cannot run Pharo 5/6 on my Mac since I upgraded 
to Sierra and the method is not available in VAST ;-)

Best,

Joachim





>
> Cheers
> Philippe
> _______________________________________________
> seaside mailing list
> seaside at lists.squeakfoundation.org
> http://lists.squeakfoundation.org/cgi-bin/mailman/listinfo/seaside


-- 
-----------------------------------------------------------------------
Objektfabrik Joachim Tuchel          mailto:jtuchel at objektfabrik.de
Fliederweg 1                         http://www.objektfabrik.de
D-71640 Ludwigsburg                  http://joachimtuchel.wordpress.com
Telefon: +49 7141 56 10 86 0         Fax: +49 7141 56 10 86 1