Unfortunately this fix doesn't seem to work at all. :-( The code was
already stopping the event and even when I return false from the
function it still scrolls to the top. We're back to square one.

:-(

Cheers,
Jon

On 10/26/05, Jon Tirsen <[EMAIL PROTECTED]> wrote:
> On 10/26/05, Rahul Bhargava <[EMAIL PROTECTED]> wrote:
> >
> > 1) have an argument so that the event can be passed into the method (this is
> > what bindAsEventListener lets you do)
> >
> >   enterEditMode: function(evt) {
>
> You're correct, but sometimes enterEditMode is called without the
> event argument (from the unit tests for example). So we need to
> conditionally stop it if it's not null, like this:
>
> if(evt) Event.stop(evt);
>
> I'll incorporate this in a patch and send it to Thomas asap.
>
> Thanks for the help everyone!
>
> Cheers,
> Jon
>
_______________________________________________
Rails-spinoffs mailing list
Rails-spinoffs@lists.rubyonrails.org
http://lists.rubyonrails.org/mailman/listinfo/rails-spinoffs

Reply via email to