Set open attr, not just prop

[mathiasbynens]

Set open attr, not just prop

[MadLittleMods]

👍

It seems to work just fine if you swap prop for attr.

toggleOpen = function($details, $detailsSummary, $detailsNotSummary, toggle) {
    var isOpen = $details.attr('open'),
        close = isOpen && toggle || !isOpen && !toggle;
    if (close) {
        $details.removeClass('open').attr('open', false).triggerHandler('close.details');
        $detailsSummary.attr('aria-expanded', false);
        $detailsNotSummary.hide();
    } else {
        $details.addClass('open').attr('open', true).triggerHandler('open.details');
        $detailsSummary.attr('aria-expanded', true);
        $detailsNotSummary.show();
    }
};

[WebDragon]

http://api.jquery.com/prop/ presumably there was a reason prop was used instead of attr

[pthorson]

http://api.jquery.com/prop/ presumably there was a reason prop was used instead of attr

Specifically support for newer jQuery version: 3cc7fc6#diff-b76a01b9ccf8cf65939d876b7f8d079f