Skip to content

Commit

Permalink
ui: do not uncheck an appropriate blast method when sequence type cha…
Browse files Browse the repository at this point in the history
…nges (#83)

I don't see any reason why the line:

    $("#blastp, #tblastn").uncheck().disable().first().change();

would uncheck all the blast methods instead of just `blastp` and `tblastn`,
because of which SS had the bug on the first place.  Embarrassingly, I don't
even understand why this patch actually fixes the problem :|.  Thanks to Ben,
for pointing out the issue and the solution :).

Signed-off-by: Anurag Priyam <anurag08priyam@gmail.com>
  • Loading branch information
yeban committed Apr 3, 2012
1 parent 0874faa commit 5be0eb5
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion public/js/sequenceserver.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,8 @@
(function( $ ){
//uncheck an element
$.fn.uncheck = function() {
return this.removeAttr('checked');
this.attr('checked') && this.removeAttr('checked');
return this;
};
})( jQuery );

Expand Down

0 comments on commit 5be0eb5

Please sign in to comment.