jquery.makeCollapsible: Use .prop() for value attribute of list item
authorTimo Tijhof <krinklemail@gmail.com>
Sat, 24 May 2014 01:27:25 +0000 (03:27 +0200)
committerKrinkle <krinklemail@gmail.com>
Sat, 24 May 2014 21:19:45 +0000 (21:19 +0000)
commit4ba7d3c24462c57e2e085c2028512425eda4dc36
treefcc0df98c2c7665655645674bc88766c65849b7c
parent23563a3c10d394eebed7a4b946e029917161e565
jquery.makeCollapsible: Use .prop() for value attribute of list item

The issue with "value" attributes is popularised through form
fields (e.g. change <input> value with prop, because it doesn't work
with attribute, attributes only change the DOM, not the live rendering).
However the attribute/property issue isn't limited to form fields and the
typical value/selected/disabled properties . It applies to all properties
and for any element that affects active state (if it changes the display,
one should use a property, not an attribute).

As far as I know, unlike for form fields, browsers do actually synchronise
attributes in the many of those cases (including list items), so there is
no immediate danger. But since it's caught by jQuery Migrate and a good
practice regardless, change it to use prop() as well.

To test:
 https://test.wikipedia.org/?oldid=199994#head-collapsible-ol

Change-Id: I246c26bd2958209feed73fc2ef31cf678b12fa3b
resources/src/jquery/jquery.makeCollapsible.js