Use Fluid layout be default

https://fedorahosted.org/freeipa/ticket/3904

11 files changed, 21 insertions, 62 deletions

diff --git a/install/ui/src/freeipa/aci.js b/install/ui/src/freeipa/aci.js
index b848073b0..ce4eb0efa 100644
--- a/install/ui/src/freeipa/aci.js
+++ b/install/ui/src/freeipa/aci.js
@@ -116,7 +116,7 @@ return {
             ],
             widgets: [
                 {
-                    $type: 'details_table_section',
+                    $type: 'details_section',
                     name: 'identity',
                     label: '@i18n:objects.permission.identity',
                     widgets: [
@@ -124,7 +124,7 @@ return {
                     ]
                 },
                 {
-                    $type: 'details_table_section',
+                    $type: 'details_section',
                     name: 'rights',
                     label: '@i18n:objects.permission.rights',
                     widgets: [
@@ -136,7 +136,7 @@ return {
                 },
                 {
                     $type: 'permission_target',
-                    container_factory: IPA.details_table_section,
+                    container_factory: IPA.details_section,
                     label: '@i18n:objects.permission.target',
                     name: 'target',
                     show_target: false
@@ -216,7 +216,7 @@ return {
         ],
         widgets: [
             {
-                $type: 'details_table_section_nc',
+                $type: 'details_section',
                 name: 'general',
                 widgets: [
                     'cn',
@@ -646,13 +646,13 @@ IPA.rights_widget = function(spec) {
 
 /**
  * @class IPA.permission_target_widget
- * @extends IPA.details_table_section_nc
+ * @extends IPA.details_section
  */
 IPA.permission_target_widget = function(spec) {
 
     spec = spec || {};
 
-    var factory = spec.container_factory || IPA.details_table_section_nc;
+    var factory = spec.container_factory || IPA.details_section;
 
     var that = factory(spec);
 
diff --git a/install/ui/src/freeipa/automember.js b/install/ui/src/freeipa/automember.js
index c18f619e5..703b4a173 100644
--- a/install/ui/src/freeipa/automember.js
+++ b/install/ui/src/freeipa/automember.js
@@ -250,7 +250,7 @@ IPA.automember.rule_details_facet = function(spec) {
 
     spec.widgets = [
             {
-            $type: 'details_table_section',
+            $type: 'details_section',
             name: 'general',
             label: '@i18n:details.general',
             widgets: [
diff --git a/install/ui/src/freeipa/details.js b/install/ui/src/freeipa/details.js
index 687926c8c..596dc1ac0 100644
--- a/install/ui/src/freeipa/details.js
+++ b/install/ui/src/freeipa/details.js
@@ -208,7 +208,7 @@ exp.section_builder = IPA.section_builder = function(spec) {
      * TODO: should be modified so it can be a class too
      * @property {IPA.composite_widget}
      */
-    that.section_factory = spec.section_factory || IPA.details_table_section;
+    that.section_factory = spec.section_factory || IPA.details_section;
 
     /**
      * Field builder
@@ -624,47 +624,6 @@ exp.details_facet = IPA.details_facet = function(spec, no_init) {
         that.facet_create_header(container);
 
         that.create_controls();
-
-        that.expand_button = IPA.action_button({
-            name: 'expand_all',
-            href: 'expand_all',
-            label: '@i18n:details.expand_all',
-            'class': 'right-aligned-facet-controls',
-            style: 'display: none;',
-            click: function() {
-                that.expand_button.css('display', 'none');
-                that.collapse_button.css('display', 'inline-block');
-
-                var widgets = that.widgets.get_widgets();
-                for (var i=0; i<widgets.length; i++) {
-                    var widget = widgets[i];
-                    if(widget.toggle) {
-                        widget.toggle(true);
-                    }
-                }
-                return false;
-            }
-        }).appendTo(that.controls);
-
-        that.collapse_button = IPA.action_button({
-            name: 'collapse_all',
-            href: 'collapse_all',
-            label: '@i18n:details.collapse_all',
-            'class': 'right-aligned-facet-controls',
-            click: function() {
-                that.expand_button.css('display', 'inline-block');
-                that.collapse_button.css('display', 'none');
-
-                var widgets = that.widgets.get_widgets();
-                for (var i=0; i<widgets.length; i++) {
-                    var widget = widgets[i];
-                    if(widget.toggle) {
-                        widget.toggle(false);
-                    }
-                }
-                return false;
-            }
-        }).appendTo(that.controls);
     };
 
     that.widgets.create_widget_delimiter = function(container) {
diff --git a/install/ui/src/freeipa/dialog.js b/install/ui/src/freeipa/dialog.js
index a31c9f453..a289e52f7 100644
--- a/install/ui/src/freeipa/dialog.js
+++ b/install/ui/src/freeipa/dialog.js
@@ -423,7 +423,7 @@ IPA.dialog = function(spec) {
         });
         var section_builder = IPA.section_builder({
             container: that,
-            section_factory: IPA.details_table_section_nc,
+            section_factory: IPA.details_section,
             widget_builder: widget_builder,
             field_builder: field_builder
         });
diff --git a/install/ui/src/freeipa/dns.js b/install/ui/src/freeipa/dns.js
index 71a3f5b82..347461584 100644
--- a/install/ui/src/freeipa/dns.js
+++ b/install/ui/src/freeipa/dns.js
@@ -1207,7 +1207,7 @@ return {
                 {
                     name: 'identity',
                     label: '@i18n:details.identity',
-                    $type: 'details_table_section',
+                    $type: 'details_section',
                     widgets: [
                         {
                             $type: 'dnsrecord_host_link',
@@ -1237,7 +1237,7 @@ return {
         widgets: [
             {
                 name: 'general',
-                $type: 'details_table_section_nc',
+                $type: 'details_section',
                 widgets: [
                     'idnsname',
                     {
@@ -1375,7 +1375,7 @@ IPA.dns.record_prepare_editor_for_type = function(type, fields, widgets, update)
 
     var section = {
         name: type.name,
-        $type: 'details_table_section_nc',
+        $type: 'details_section',
         widgets: []
     };
     widgets.push(section);
@@ -1430,14 +1430,14 @@ IPA.dns.record_prepare_details_spec = function(spec, type_prepare_method) {
 
     var standard_record_section = {
         name: 'standard_types',
-        $type: 'details_table_section',
+        $type: 'details_section',
         label: '@i18n:objects.dnsrecord.standard',
         widgets: []
     };
 
     var other_record_section = {
         name: 'other_types',
-        $type: 'details_table_section',
+        $type: 'details_section',
         label: '@i18n:objects.dnsrecord.other',
         widgets: []
     };
diff --git a/install/ui/src/freeipa/hbac.js b/install/ui/src/freeipa/hbac.js
index 0fb40373e..f912fbd53 100644
--- a/install/ui/src/freeipa/hbac.js
+++ b/install/ui/src/freeipa/hbac.js
@@ -248,7 +248,7 @@ var add_hbacrule_details_facet_widgets = function (spec) {
 
     spec.widgets = [
         {
-            $type: 'details_table_section',
+            $type: 'details_section',
             name: 'general',
             label: '@i18n:details.general',
             widgets: [
diff --git a/install/ui/src/freeipa/netgroup.js b/install/ui/src/freeipa/netgroup.js
index 2922d5b94..0dd29f1db 100644
--- a/install/ui/src/freeipa/netgroup.js
+++ b/install/ui/src/freeipa/netgroup.js
@@ -104,7 +104,7 @@ var add_netgroup_details_facet_widgets = function (spec) {
 
     spec.widgets = [
         {
-            $type: 'details_table_section',
+            $type: 'details_section',
             name: 'identity',
             label: '@i18n:details.general',
             widgets: [
diff --git a/install/ui/src/freeipa/selinux.js b/install/ui/src/freeipa/selinux.js
index e71487667..ff0ad848c 100644
--- a/install/ui/src/freeipa/selinux.js
+++ b/install/ui/src/freeipa/selinux.js
@@ -141,7 +141,7 @@ var add_selinux_details_facet_widgets = function (spec) {
 
     spec.widgets = [
         {
-            $type: 'details_table_section',
+            $type: 'details_section',
             name: 'general',
             label: '@i18n:details.general',
             widgets: [
diff --git a/install/ui/src/freeipa/sudo.js b/install/ui/src/freeipa/sudo.js
index b01621ddd..aa2597be1 100644
--- a/install/ui/src/freeipa/sudo.js
+++ b/install/ui/src/freeipa/sudo.js
@@ -250,7 +250,7 @@ var add_sudorule_details_facet_widgets = function (spec) {
 
     spec.widgets = [
         {
-            $type: 'details_table_section',
+            $type: 'details_section',
             name: 'general',
             label: '@i18n:details.general',
             widgets: [
diff --git a/install/ui/src/freeipa/trust.js b/install/ui/src/freeipa/trust.js
index 2653d3e67..b2fe567f8 100644
--- a/install/ui/src/freeipa/trust.js
+++ b/install/ui/src/freeipa/trust.js
@@ -163,7 +163,7 @@ return {
         ],
         widgets: [
             {
-                $type: 'details_table_section_nc',
+                $type: 'details_section',
                 name: 'realm',
                 widgets: [
                     'realm_server'
@@ -207,7 +207,7 @@ return {
                 ]
             },
             {
-                $type: 'details_table_section_nc',
+                $type: 'details_section',
                 name: 'range',
                 widgets: [
                     {
diff --git a/install/ui/src/freeipa/widget.js b/install/ui/src/freeipa/widget.js
index 778468965..f49498843 100644
--- a/install/ui/src/freeipa/widget.js
+++ b/install/ui/src/freeipa/widget.js
@@ -4088,7 +4088,7 @@ IPA.multiple_choice_section = function(spec) {
 
     var that = IPA.composite_widget(spec);
     that.choices = $.ordered_map().put_array(spec.choices, 'name');
-    that.layout = IPA.build(spec.layout || IPA.table_layout);
+    that.layout = IPA.build(spec.layout || IPA.fluid_layout);
 
     that.create = function(container) {