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.

Sign up for GitHub

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

Jump to bottom

html/input: Lowercase attributes #1024

Merged
merged 3 commits into from
Jul 9, 2018
Merged

html/input: Lowercase attributes #1024

merged 3 commits into from
Jul 9, 2018

Commits on Jul 9, 2018

  1. html/input: Lowercase "minlength" attribute 

    Attributes are case-insensitive, and they are conventionally written
in lowercase to reduce confusion.

It seems this particular example accidentally used the naming of
the IDL property (which is different from the DOM attribute).

In most cases the only difference is the casing, but in some cases
(such as node.className vs "class"), the name is different entirely.

In general to avoid confusion, attributes should consistently be
written in lowercase.
    @Krinkle
    Krinkle authored Jul 9, 2018
    Configuration menu
    Copy the full SHA
    13e15e6 View commit details
    Browse the repository at this point in the history

  2. html/input: Update password.html

    @Krinkle
    Krinkle authored Jul 9, 2018
    Configuration menu
    Copy the full SHA
    6037d65 View commit details
    Browse the repository at this point in the history

  3. html/input: Lowercase "maxlength" attribute

    @Krinkle
    Krinkle authored Jul 9, 2018
    Configuration menu
    Copy the full SHA
    aa08e2d View commit details
    Browse the repository at this point in the history