Re: [Help-smalltalk] Iliad - updatePage: aPage

[Nicolas Petton]

Le lundi 13 juillet 2009 à 21:52 +0200, Stefan Schmiedl a écrit :
> Hi Nico,
> 
> what exactly is updatePage: aPage for?
> 
> We all seem to be using it to update head information and guessing
> from the position in respondOn: it is not intended to modify the
> body element.
> 
>       | page |
>       page := self defaultPageClass new.
>       self buildOn: page body.
>       self updatePage: page.
>       page respondOn: aResponse
> 
> So why not call it "updateHead: aHead" and use it like
> 
>       | page |
>       page := self defaultPageClass new.
>       self buildOn: page body.
>       self updateHead: page head.
>       page respondOn: aResponse
> 

Indeed, I thought about that when refactoring Application building
process. I thought that maybe being allowed to update any part of the
page could still be usefull, to add a script at the end of the body, or
add a css class to the body element.


Now, if everyone think it's completely useless, then I agree,
#updateHead: would be more convenient.

Cheers!

Nico

signature.asc
Description: Ceci est une partie de message numériquement signée