diff options
| author | Petr Vobornik <pvoborni@redhat.com> | 2011-09-14 13:01:25 +0200 |
|---|---|---|
| committer | Endi S. Dewata <edewata@redhat.com> | 2011-09-29 16:58:53 +0000 |
| commit | 1e2c0562834d0ee3c3a7a815f5449befa4797a5a (patch) | |
| tree | e395edfa7099efc1fd9cb1fed84db35b723de734 /install/ui/aci.js | |
| parent | a60ee8b7810ed2dbe0affe91112c4d7c95394018 (diff) | |
| download | freeipa-1e2c0562834d0ee3c3a7a815f5449befa4797a5a.tar.gz freeipa-1e2c0562834d0ee3c3a7a815f5449befa4797a5a.tar.xz freeipa-1e2c0562834d0ee3c3a7a815f5449befa4797a5a.zip | |
Fixed: Some widgets do not have space for validation error message
https://fedorahosted.org/freeipa/ticket/1454
The following widgets should call create_error_link() to create a space to show validation error messages:
IPA.checkbox_widget
IPA.checkboxes_widget
IPA.radio_widget
IPA.select_widget
IPA.table_widget
IPA.attributes_widget
IPA.rights_widget
IPA.target_section (it's a widget)
Solution:
* added call to checkbox, checkboxes, radio, select, table, attributes widget
* rights_widget inherits it from checkboxes_widget.
* target_section IS NOT a widget as it doesn't inherit from widget. It's still a section, which shows different widgets based on its state.
* table_widget displays error_link between pagination and summary.
Additional:
* added padding and unified font-weight for error message
Diffstat (limited to 'install/ui/aci.js')
| -rw-r--r-- | install/ui/aci.js | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/install/ui/aci.js b/install/ui/aci.js index 6fd4a525a..e5b54912d 100644 --- a/install/ui/aci.js +++ b/install/ui/aci.js @@ -276,6 +276,8 @@ IPA.attributes_widget = function(spec) { if (that.object_type) { that.populate(that.object_type); } + + that.create_error_link(container); }; that.load = function(record) { @@ -371,6 +373,10 @@ IPA.attributes_widget = function(spec) { } }; + that.show_undo = function() { + $(that.undo_span).css('display', 'inline-block'); + }; + return that; }; |