Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Develop 378 test this in delegated events #66

Closed
wants to merge 2 commits into from
Closed

Develop 378 test this in delegated events #66

wants to merge 2 commits into from

Conversation

fat
Copy link
Contributor

@fat fat commented Sep 22, 2010

Lighthouse-378. "this" inside a function triggered by event delegation malfunction

https://mootools.lighthouseapp.com/projects/24057-mootoolsmore/tickets/378-this-inside-a-function-triggered-by-event-delegation-malfunction#ticket-378-3

- modifies the test to be a bit more clear with this sort of failure
  (also as a note, i think this particular issue is for -core, not -more)

@cpojer
Copy link
Member

cpojer commented Sep 22, 2010

Exactly.

@anutron
Copy link
Member

anutron commented Sep 22, 2010

does this need to be pulled into 1.2x? or just 1.3? The only code change I see looks like one specific to 1.3 stuff.

@anutron
Copy link
Member

anutron commented Sep 22, 2010

Ah. I see it needs to go on develop. will pull.

@fat
Copy link
Contributor Author

fat commented Sep 22, 2010

Yeah it's for a bug with -core 1.2.5 :)

@anutron
Copy link
Member

anutron commented Sep 22, 2010

this is pulled. Note that I squashed the commits.

http://github.com/mootools/mootools-more/commit/82e651af943c24271ed83f031009257bc867d997

cpojer, arian, tim? one of you want to pick this into 1.3?

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants