HTML5 Validation error in Screen Options > Pagination

[Arena94]

Error: Attribute “maxlength” is only allowed when the input type is “email”, “password”, “search”, “tel”, “text”, or “url”.

[Arena94]

patch

[afercia]

@Arena94 thanks for your patch.

Looking back for some history, this input type was changed from text to number in [20168], see #17863. Seems the maxlength attribute was kept for some backwards compatibility concerns. However, ​the next 4.8 release is going to drop support for old IEs so maybe worth considering to remove it.

Worth noting seems browsers have all sorts of different behaviours, most of them allow to enter more than 3 characters ignoring the max="999" and maxlength="3" attributes when manually entering numbers, regardless of the presence of the attribute (it only works when the input type is text).

As far as I see, removing maxlength="3" makes some difference just in IE11 and Edge. With maxlength="3" it's not possible to manually enter more than 3 numbers. Without maxlength="3" it becomes possible. It is always possible when the entered value starts with an alpha character.

Chrome Mac (doesn't honor max and maxlength when manually entering numbers)

Firefox Mac (doesn't prevent alpha characters input, doesn't honor max and maxlength when manually entering numbers)

IE11 (doesn't prevent alpha characters input, doesn't honor max and maxlength when entering a mix of alpha characters and numbers)

Edge (same as IE11 plus doesn't honor max when incrementing the field value using the up arrow)

Moving to 4.8 consideration.

[afercia]

Related: #37004.

[ocean90]

#47530 was marked as a duplicate.

[joedolson]

This is pretty trivial; let's resolve it.

[joedolson]

Refresh patch

[joedolson]

IE 11 is no longer officially supported, and removing maxlength no longer causes issues in Edge, which now behaves the same as expected with maxlength removed. LGTM.

[joedolson]

In 57272:

Administration: Remove invalid attribute maxlength on number input.

Remove the maxlength attribute on screen options number of items per page input. Previously kept due to input inconsistencies in IE 11 and Edge, this invalid usage is no longer needed. IE 11 is no longer supported, and Edge now behaves according to specifications.

Props Arena94, afercia, joedolson.
Fixes #40610.