Bug 5184 [REVISED] Upgrade jQuery to the latest version (OPAC)
Upgrade jQuery in the OPAC.
To test: Try any JavaScript-dependent interaction
you can think of in the OPAC and watch for errors.
Revision updates the one syntax I've found so far in testing
which requires an update. Where we used to do this to
uncheck a checkbox:
$(foo).attr("checked","");
We should now use:
$(foo).removeAttr("checked");
As far as I can tell the old way will not generate an error
but neither will it work.
Signed-off-by: Chris Nighswonger <cnighswonger@foundations.edu>
This commit is contained in:
parent
d1e3773e3b
commit
806798dc66
File diff suppressed because one or more lines are too long
|
|
@ -6,10 +6,10 @@
|
|||
if($(this).attr("checked")){
|
||||
var rowid = $(this).attr("id");
|
||||
newid = Number(rowid.replace("none",""))
|
||||
$("#sms"+newid).attr("checked","");
|
||||
$("#email"+newid).attr("checked","");
|
||||
$("#digest"+newid).attr("checked","");
|
||||
$("#rss"+newid).attr("checked","");
|
||||
$("#sms"+newid).removeAttr("checked");
|
||||
$("#email"+newid).removeAttr("checked");
|
||||
$("#digest"+newid).removeAttr("checked");
|
||||
$("#rss"+newid).removeAttr("checked");
|
||||
}
|
||||
});
|
||||
});
|
||||
|
|
|
|||
Loading…
Reference in New Issue