Excuse me for my bad English.
I am using the forms Event and Views for Drupal 5.1.
They work very well, but they doesn't pass the validation of the W3C (http://validator.w3.org /).
The error is this:
... "event_term_select" onChange = "this.form.submit () " class = "for...
Error: there is no attribute onChange for this element (in this HTML version)
Is it possible to correct the error?
I am making the site of my school and the new Italian law wants the sites of the public administration to be correct.
Thanks for the help.

CommentFileSizeAuthor
#3 event_onchange.diff2.32 KBdturover

Comments

killes@www.drop.org’s picture

Component: Event Views » User interface

moving

killes@www.drop.org’s picture

Version: 5.x-1.0 » 6.x-2.x-dev
Assigned: luisasasi » Unassigned
Priority: Critical » Normal

Actually, this I'll try to fix this in the .2 versions only.

dturover’s picture

StatusFileSize
new2.32 KB

onChange is an easy fix. Just convert it to the lower-case onchange. The bug is that w3c outlawed what was standard coding practice at the time =)

It looks like the problem is only in three places in event.module. I am going to try to upload a patch. I just did a mass search-and-replace, so if you don't want it lowercased in the comments too, then leave those parts out.

gerhard killesreiter’s picture

Status: Active » Reviewed & tested by the community

Thanks!

Could I bug you to convert this to jQuery?

dturover’s picture

Sorry, I would not know how to begin doing that. Isn't jQuery a javascript library or is there something else with that name?

killes@www.drop.org’s picture

Status: Reviewed & tested by the community » Fixed

yes, jQuery is the Drupal JS library. Thanks for the patch anyway!

Anonymous’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.