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:57:37 +0000 |
| commit | dabc6cd578b20ce4950d1e4052c0a814b8697ca9 (patch) | |
| tree | c11a309717653e2ea2fc3065b102afaaa65c7380 /install/ui/aci.js | |
| parent | f99ab781ea33f0d7ca6df26090cd99f9315454ac (diff) | |
| download | freeipa-dabc6cd578b20ce4950d1e4052c0a814b8697ca9.tar.gz freeipa-dabc6cd578b20ce4950d1e4052c0a814b8697ca9.tar.xz freeipa-dabc6cd578b20ce4950d1e4052c0a814b8697ca9.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 80d5798b..caed44ba 100644 --- a/install/ui/aci.js +++ b/install/ui/aci.js @@ -314,6 +314,8 @@ IPA.attributes_widget = function(spec) { if (that.object_type) { that.populate(that.object_type); } + + that.create_error_link(container); }; that.load = function(record) { @@ -409,6 +411,10 @@ IPA.attributes_widget = function(spec) { } }; + that.show_undo = function() { + $(that.undo_span).css('display', 'inline-block'); + }; + return that; }; |