>> b. If I should contribute some sort of "scroll" effect to >> Tapestry.ElementEffect. > > I'd pick this route, then you can just set the "show" attribute to > "scroll" in whatever zones you want to have this behavior and not > duplicate the code. You might want to use > http://www.prototypejs.org/api/element/scrollto instead of the > location.hash method. > >> c. If Zone should just get a new parameter: scroll, if set to true it >> will emit the above code in addition to the transition. > > Personally I don't think this calls for a new parameter.
The logic for having a new parameter would be that you would want to combine them, that you would want: scrollTo(); to happen BEFORE the effect. Otherwise then you need a pre-scrolling variant of each effect, which seems like a lot. Using prototype scrollTo() seems to lock us more into ProtoType when my designer want's to use jQuery... Pierce --------------------------------------------------------------------- To unsubscribe, e-mail: users-unsubscr...@tapestry.apache.org For additional commands, e-mail: users-h...@tapestry.apache.org
