Skip to content

fixed case that CHANGE was not triggered if the field loses focus with u...#91

Closed
viniciusoyama wants to merge 1 commit intoplentz:masterfrom
viniciusoyama:master
Closed

fixed case that CHANGE was not triggered if the field loses focus with u...#91
viniciusoyama wants to merge 1 commit intoplentz:masterfrom
viniciusoyama:master

Conversation

@viniciusoyama
Copy link

Fixed a little bug:

If I change the field value and leave the field by clicking outside the field (not using TAB or ENTER) the CHANGE event was not triggered.

@plentz
Copy link
Owner

plentz commented Aug 17, 2013

Thanks @viniciusoyama. This is a know bug and as you find out, related to that "dirty" check. I think that we need to fix the whole problem, and your change fix just a few parts of it. Anyway, I will leave it open to use as a reference.

related #44 #66

@viniciusoyama
Copy link
Author

Hi. I didn't know that there was more problems related to this bug. I will see what I can do about this.

Thanks for your help.

@ghost ghost assigned plentz Dec 13, 2013
@plentz
Copy link
Owner

plentz commented Dec 16, 2013

I'm closing this one to centralize everything related to the change event problem in #44

@plentz plentz closed this Dec 16, 2013
plentz pushed a commit that referenced this pull request Dec 31, 2013
…44 #47 #66 #86 #91

also, closing #13, as we have now a good(not perfect) test suite. fixes #13

to celebrate, I'm launching maskMoney 3.0.0 :-)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments