Hi
I have a really nast "too much recursion" bug..
I have a really large table in my page, and, when I scroll to the
bottom of the table - whenever I hover over a <tr> element, Firefox
throws an error at:
too much recursion
[Break on this error] if (observer.call(this, event) === false) {
which is in this function in lowpro.js: (line 165)
Ok - I think I have found the issue.
In lowpro.js, when it's ovverrideing the prototype extend method, and
adding observers to elements, it runs the load method - which is
defined as thus:
the call to observer = Event.addBehavior._wrapObserver(observer);
means that observer is wrapped with itself, and then in the next
iteration, you are wrapping the already wrapped observer..Therefore
observer.call() will run observer.call() (etc etc)
surely the fix is to run
observer = Event.addBehavior._wrapObserver(rules[selector])
instead?
It worked for me..let me know what you think
On 5 May, 12:44, Ro <rohittriv...@gmail.com> wrote:
There is some work going on for the load method in the alternate forks
of Low Pro.
I will try to incorporate your changes remove this problem when I work
on the load method again and then try to get the other forks to accept
the changes, unless someone beats me to it ;-)
On 5 May, 17:32, MRoderick <roderick.mor...@gmail.com> wrote:
> I think you've cracked it!
> There is some work going on for the load method in the alternate forks
> of Low Pro.
> I will try to incorporate your changes remove this problem when I work
> on the load method again and then try to get the other forks to accept
> the changes, unless someone beats me to it ;-)
> On 5 May, 17:32, MRoderick <roderick.mor...@gmail.com> wrote:
> > I think you've cracked it!
> > There is some work going on for the load method in the alternate forks
> > of Low Pro.
> > I will try to incorporate your changes remove this problem when I work
> > on the load method again and then try to get the other forks to accept
> > the changes, unless someone beats me to it ;-)