Permalink
Browse files

Making the checkboxes work fine on IE. onchange doesnt work very well…

… with checkboxes there. The target is not set too.

Original author: Fabio M. Costa <fabiomcosta@gmail.com>
  • Loading branch information...
1 parent 8ecfd26 commit 377703745f166617301006f233afb6ed996c8265 @xian xian committed Oct 5, 2010
Showing with 4 additions and 4 deletions.
  1. +4 −4 src/html/TrivialReporter.js
@@ -70,16 +70,16 @@ jasmine.TrivialReporter.prototype.reportRunnerStarting = function(runner) {
this.startedAt = new Date();
var self = this;
- showPassed.onchange = function(evt) {
- if (evt.target.checked) {
+ showPassed.onclick = function(evt) {
+ if (showPassed.checked) {
self.outerDiv.className += ' show-passed';
} else {
self.outerDiv.className = self.outerDiv.className.replace(/ show-passed/, '');
}
};
- showSkipped.onchange = function(evt) {
- if (evt.target.checked) {
+ showSkipped.onclick = function(evt) {
+ if (showSkipped.checked) {
self.outerDiv.className += ' show-skipped';
} else {
self.outerDiv.className = self.outerDiv.className.replace(/ show-skipped/, '');

0 comments on commit 3777037

Please sign in to comment.