Re: [T5.2.0] two problems when extending Autocomplete

2010-03-07 Thread Joost Schouten (ml)
My apologies, there was a problem in my code that for some reason only started playing up after the latest commits. I mixed in both the Autocomplete and my own ForceAutocomplete which extends Autocomplete. This caused the problem and for some reason worked in the previous versions. Thank you f

Re: [T5.2.0] two problems when extending Autocomplete

2010-03-07 Thread Joost Schouten (ml)
Thanks for fixing this so quickly. I'll try to dive into the problem of generateResponseMarkup(MarkupWriter writer, List matches)) no longer being called and report back on it. Cheers, Joost Howard Lewis Ship wrote: On Fri, Mar 5, 2010 at 7:39 PM, Joost Schouten (ml) wrote: Hi, I have f

Re: [T5.2.0] two problems when extending Autocomplete

2010-03-06 Thread Howard Lewis Ship
On Fri, Mar 5, 2010 at 7:39 PM, Joost Schouten (ml) wrote: > Hi, > > I have found another problem when extending the Autocomplete Mixin which has > been introduced by recent changes on the 5.2.0-SNAPSHOT version. > > - The @Override methods (eg. generateResponseMarkup(MarkupWriter writer, > List m

Re: [T5.2.0] two problems when extending Autocomplete

2010-03-06 Thread Joost Schouten (ml)
I think that is a good idea, but it is not the problem I am faced with here. I have no need to change the js file, it's just that tapestry now tries to find the js file at the classpath of the extending class. On the current snapshot I get a "Unable to locate asset 'classpath:com/joostschouten/

Re: [T5.2.0] two problems when extending Autocomplete

2010-03-06 Thread Robin Komiwes
Hi! Yes, in someway, it should be possible to change the automatically included JS when extending a component. Already encountred this problem too. Had to create and advice/decorator on asset source to fix the thing without forking the component. On Sat, Mar 6, 2010 at 4:39 AM, Joost Schouten (ml

[T5.2.0] two problems when extending Autocomplete

2010-03-05 Thread Joost Schouten (ml)
Hi, I have found another problem when extending the Autocomplete Mixin which has been introduced by recent changes on the 5.2.0-SNAPSHOT version. - The @Override methods (eg. generateResponseMarkup(MarkupWriter writer, List matches)) are no longer called on the extending class On the dev li