Question

I got this code:

$(document).ready(function () {
    $('tr.DataDark input[type=checkbox], tr.DataLight input[type=checkbox]').click(function () {
        $(this).closest('tr').toggleClass('Selected');
    });
    $('tr.DataDark input[type=checkbox]:checked, tr.DataLight input[type=checkbox]:checked').each(function () {
        $(this).closest('tr').addClass('Selected');
    });
});

Can it be shorter written?

Thanks for suggestions.

Answer 1

Maybe this? Not exactly shorter, but possibly simpler.

$(document).ready(function() {
  var checkBoxes = $('tr.DataDark input[type=checkbox], tr.DataLight input[type=checkbox]');

  checkBoxes.on("click", function() {
    $(this).closest('tr').toggleClass('Selected');
  });

  checkBoxes.filter(':checked').closest('tr').addClass('Selected');
});

To make it shorter you could forego the checkBoxes variable and chain the filter().closest().addClass() calls directly to on("click").