-
Notifications
You must be signed in to change notification settings - Fork 1
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
Warn before exit Message #3
Labels
bug
Something isn't working
Comments
Full solution (in JS file): ...
isChanged:function() {
// #3: Use the unformated numer to determine if the value has been changed or not
return (this.node.defaultValue != (AutoNumeric.isManagedByAutoNumeric("#"+this.id) ? AutoNumeric.getNumber("#"+this.id) : this.getValue()));
}, |
Nice one seems more neat and clean, I'll change with current setting : isChanged:function() {
return (this.node.defaultValue != this.getValue());
}, |
@grlicaa I'm actually working on all the changes so give me the day before you apply. I'll do a PR and you can apply the changes you want |
@martindsouza Perfect :) |
martindsouza
added a commit
to martindsouza/AutoNumeric
that referenced
this issue
Jan 30, 2021
Added to version 4.6.0.4 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Example: If changing a number to a percentage (
0.12
to12%
) everything displays properly on the screen for the user. If the user tries to exit page without saving (iecancel
etc) they'll get the warning before saving.Need to resolve this issue so that this is not raised. Sample snippet to resolve:
The text was updated successfully, but these errors were encountered: