Hi, boy ;) !
About typos and some other english errors: my english is poor enough
to shame (thanks to Cambridge Dictionaries Online) ;)
1st question: sorry Christophe but my code said this.Listeners issue
works ok
2on question: I revise it. For me is weird too because is the first
time it happens since version 1.7.something. I have Scriptaculous
1.8.1
On 20 ene, 22:32, Christophe Porteneuve <[EMAIL PROTECTED]> wrote:
> Garito a écrit :
>
> > For that I have some questions to ask. As they are minor questions (at
> > least it seems to me) I make them here in only one threat (sorry if
> > you think will be better in a separated threats)
>
> I love your typo ("threat" instead of "thread"); it feels like we're in
> the comments on some Vi-vs-Emacs flamewar.
>
> > Perhaps they will be for Christophe again (sorry for make you work
> > again ;) )
>
> Hey, if I minded, I wouldn't read it. Plus, I need to stretch my legs
> on this list again and maintain my reply score ;-)
>
> > This is my question: have you any problem if Christophe change this
> > line to something like:
>
> > $H(this.Listeners).each(function(pair) {
>
> > ?
> > With this change you could subclass Ajax.InPlaceEditor to register
> > other events instead of the original ones
>
> Except this won't work. You're mistaking instance fields and class
> fields. A.IPE.Listeners is a class-level member. However, you could
> look into how A.IPCE augments default options for a way to do such a
> thing (which admittedly is a bit tortuous).
>
> I could imagine a _extraListeners field in there somewhere, but to make
> all this really cleaner I'd need to use Prototype's inheritance mechanism.
>
> In the meantime you can just override registerListeners and
> unregisterListeners in your subclass and start by calling the inherited
> version, something like:
>
> Ajax.MyInPlaceEditor = Class.create(Ajax.InPlaceEditor, {
> // ...
> registerListeners: function($super) {
> $super();
> // your registrations here
> },
> unregisterListeners: function($super) {
> // your "unregistrations" here
> $super();
> }
>
> });
> > Or, if you prefer, how can I change the default Listeners?
>
> Just overwrite Ajax.InPlaceEditor.Listeners, but that would be dangerous
> (in the sense that it could break behavior elsewhere on your page).
>
> What you could do is tweak A.IPE directly by adding new methods in it
> and then adding the matching defs in A.IPE.Listeners.
>
> That could work, but will affect *all* IPE's in your page, when you
> might want to create a subclass for specific needs that do not pertain
> to all IPEs on your page.
>
> > I notice that if I try to cancel an edition with the escape key it
> > doesn't work (nothing happens) and if I cancel with the link button
> > the url location changes to the parent location plus #
>
> That's just plain wrong. None of these behaviors should occur, and they
> sure DON'T occur on my implementation. I suggest you check that you
> have the latest version and run the unit test page for IPE to verify
> everything passes. If it doesn't, I and Thomas are interested in
> learning more about it.
>
> --
> Christophe Porteneuve aka TDD
> [EMAIL PROTECTED]
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups "Ruby
on Rails: Spinoffs" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/rubyonrails-spinoffs?hl=en
-~----------~----~----~----~------~----~------~--~---
