onclick/onClick attribute should all be lowercase

[TobiasBg]

dd32 ​mentioned that various code editors replace "onclick" attributes with "onClick".
However, this breaks XHTML validation as all attributes need to be lowercase.
Recent commits ([13242], [13503]) commited such changes to core.

Attached patch replaces all found (grep) occurances of "onClick" in core with "onclick".

[TobiasBg]

Patch replaces "onClick" with "onclick" throughout core

[nacin]

(In [13510]) s/onClick/onclick/. props TobiasBg fixes #12429

[Denis-de-Bernardy]

attached patch catches a few more

[nacin]

Will handle, though your patch would revert [13503]. Would this otherwise apply clean? Looks like .dev.js sheets are missing.

[TobiasBg]

Oh, sht. Didn't check in JavaScript files... Thanks Denis!

Although colorpicker.dev.js will probably need more cleanup, as having all uppercase attributes around a lowercase "onclick" doesn't look to good somehow (and is probably also invalid XHTML):

<A HREF="#" onclick="cp.select(document.forms[0].color,'pick');return false;" NAME="pick" ID="pick">Pick</A>

[dd32]

Sorry for those again.. I cant for the life of me work out why my code edits are insisting changing those even with that functionality disabled! (And tortoise seems to miss case changes in the preview)

[TobiasBg]

dd32 was so kind to change a few more "onClick"s in [13600], so that I'm again able to contribute a little :-)

Attached patch changes them back to "onclick".

The only remaining "onClick"s throughout core are in the .js files as Denis mentioned.

[TobiasBg]

Patch to replace "onClick" with "onclick", introduced in [13600]

[nacin]

(In [13612]) s/onClick/onclick/. props TobiasBg, see #12429

[rmccue]

All of these appear to be gone now.