diff options
| author | Petr Vobornik <pvoborni@redhat.com> | 2012-07-26 11:17:28 +0200 |
|---|---|---|
| committer | Petr Vobornik <pvoborni@redhat.com> | 2012-08-02 10:27:17 +0200 |
| commit | c2783b9b54ecae8299da0dc593c3813c2bd26fe2 (patch) | |
| tree | 8d43ff1b23545ea58a20bbc911f82f23a455acd2 /install/ui/aci.js | |
| parent | 1a94109f4ad5bcb304c4c23af868c3cf07ce92ec (diff) | |
| download | freeipa.git-c2783b9b54ecae8299da0dc593c3813c2bd26fe2.tar.gz freeipa.git-c2783b9b54ecae8299da0dc593c3813c2bd26fe2.tar.xz freeipa.git-c2783b9b54ecae8299da0dc593c3813c2bd26fe2.zip | |
Replace use of attr with prop for booleans
Recommened way of setting boolean HTML attributes is by $.prop(boolean) method not $.attr(boolean) because it sets DOM object property not an attribute. Latter works because of jquery's backward compatibility. This patch makes things clearer.
Some info about prop and attr: http://stackoverflow.com/a/5876747
https://fedorahosted.org/freeipa/ticket/2817
Diffstat (limited to 'install/ui/aci.js')
| -rw-r--r-- | install/ui/aci.js | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/install/ui/aci.js b/install/ui/aci.js index 63181efa..039e6332 100644 --- a/install/ui/aci.js +++ b/install/ui/aci.js @@ -492,7 +492,7 @@ IPA.attributes_widget = function(spec) { type: "checkbox", click: function() { $('.aci-attribute', that.table). - attr('checked', $(this).attr('checked')); + prop('checked', $(this).prop('checked')); that.value_changed.notify([], that); } }) |