http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10310
M. de Rooy <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Signed Off |Passed QA --- Comment #10 from M. de Rooy <[email protected]> --- My former QA comment still holds. Good improvement. Just curious about this line: $(e.target).attr('type') != "submit" Is there a difference with using e.target.type? Does jQuery not guarantee already that object e contains a 'which' and 'target'? Note that you assume that in the lines before this line when you directly refer to e.which and e.target. This tiny inconsistency does not prevent me from passing QA on this patch (second try). Just wait for Owen :) This patch makes it work too on IE9 which did not work. I could not test on earlier versions of IE. Not encouraging IE for staff at all btw.. Passed QA -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
