unmask() does not remove the "maxlength" attribute

[kefir500]

Version 1.14.2 adds the next feature:

apply auto maxlength in case the mask doesn't have recursive pattern

However, the unmask() function does not remove the maxlength attribute added by this behavior, and the unmasked input length is still limited by the previously set mask pattern.

Example: https://jsfiddle.net/t2cakzgb/3

The affected versions are 1.14.2 to the current (1.14.8).
The temporary solution is to manually call the removeAttr('maxlength').

[nunesbeto]

+1

[chirogamer]

what is this

[pedrofurtado]

@igorescobar Any news about this issue?

Because I am having the same issue. Here is my example in JSFiddle: https://jsfiddle.net/13cqy6jy/2/

Even calling unmask() function, the attribute "maxlength" is not removed from input tag.

Is it possible to make something like this in source code of library?

$.fn.unmask = function() {
        clearInterval($.maskWatchers[this.selector]);
        delete $.maskWatchers[this.selector];
        return this.each(function() {
            var dataMask = $(this).data('mask');
            if (dataMask) {
                dataMask.remove().removeData('mask');
            }

            $(this).removeAttr('maxlength'); // <- Here is the new code!
        });
    };

[igorescobar]

I will fix it asap, no problem.

[nusendra]

is this already fixed?
Any news 'bout this issue?

[natezander]

I still see this bug in version 1.14.12

[nusendra]

is this bug still exists?

[dougjohnston]

Still seeing this bug in 1.14.13.

[nusendra]

Now i'm playing with vue, i dont use this anymore

[vonsky104]

Guys, what @pedrofurtado said is working, just find fn.unmask function in jquery.mask.js and paste
$(this).removeAttr('maxlength'); just after
if (dataMask) { dataMask.remove().removeData('mask'); }
Thanks for that I can now mask, then unmask and mask again (I had a select with different Phone numbers from different countries and input masks needed to be different).

[pedrofurtado]

@igorescobar Any news about this issue? I've created a pull request with this change in behavior and with its tests.

[marciojg]

Any news?