Skip to content

Update method and formmethod HTML attrs capitalization#113

Merged
aeschli merged 1 commit into
microsoft:mainfrom
domdomegg:domdomegg/update-form-attrs-capitals
Sep 27, 2021
Merged

Update method and formmethod HTML attrs capitalization#113
aeschli merged 1 commit into
microsoft:mainfrom
domdomegg:domdomegg/update-form-attrs-capitals

Conversation

@domdomegg

Copy link
Copy Markdown
Contributor

As per the HTML standard (and the descriptions in vscode-html-languageservice, and common reference sites like w3schools and MDN, and the w3 validator and libraries like html-validate), the value of these should be in lowercase.

As per the [HTML standard](https://html.spec.whatwg.org/#attr-fs-method) (and the descriptions in [vscode-html-languageservice](https://github.com/microsoft/vscode-html-languageservice/blob/main/src/languageFacts/data/webCustomData.ts), and common reference sites like w3schools and MDN, and the w3 validator and libraries like html-validate), the value of these should be in lowercase.
@domdomegg

Copy link
Copy Markdown
Contributor Author

Also see microsoft/vscode-custom-data#32

@domdomegg

Copy link
Copy Markdown
Contributor Author

Requesting review from @aeschli

@aeschli aeschli merged commit 0a85abb into microsoft:main Sep 27, 2021
@aeschli

aeschli commented Sep 27, 2021

Copy link
Copy Markdown
Collaborator

Thanks @domdomegg !

@aeschli aeschli added this to the September 2021 milestone Sep 27, 2021
@aeschli aeschli self-assigned this Sep 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants