webui: remove unused collapsible feature from section

Reviewed-By: Endi Sukma Dewata <edewata@redhat.com>

7 files changed, 19 insertions, 55 deletions

diff --git a/install/ui/src/freeipa/automember.js b/install/ui/src/freeipa/automember.js
index ae7304d95..9182e1352 100644
--- a/install/ui/src/freeipa/automember.js
+++ b/install/ui/src/freeipa/automember.js
@@ -268,7 +268,7 @@ IPA.automember.rule_details_facet = function(spec) {
             ]
         },
         {
-            $factory: IPA.collapsible_section,
+            $factory: IPA.section,
             name: 'inclusive',
             label: '@i18n:objects.automember.inclusive',
             widgets: [
@@ -298,7 +298,7 @@ IPA.automember.rule_details_facet = function(spec) {
             ]
         },
         {
-            $factory: IPA.collapsible_section,
+            $factory: IPA.section,
             name: 'exclusive',
             label: '@i18n:objects.automember.exclusive',
             widgets: [
diff --git a/install/ui/src/freeipa/details.js b/install/ui/src/freeipa/details.js
index 279b76a1a..b2f07e463 100644
--- a/install/ui/src/freeipa/details.js
+++ b/install/ui/src/freeipa/details.js
@@ -44,18 +44,6 @@ define([
 var exp = {};
 
 /**
- * Name of expanded icon
- * @member details
- */
-exp.expanded_icon = IPA.expanded_icon = 'expanded-icon';
-
-/**
- * Name of collapsed icon
- * @member details
- */
-exp.collapsed_icon = IPA.collapsed_icon = 'collapsed-icon';
-
-/**
  * Details builder
  *
  * Processes containers spec and builds sections, widget and fields according
diff --git a/install/ui/src/freeipa/hbac.js b/install/ui/src/freeipa/hbac.js
index 2e9b96a8e..e8809f037 100644
--- a/install/ui/src/freeipa/hbac.js
+++ b/install/ui/src/freeipa/hbac.js
@@ -290,7 +290,7 @@ var add_hbacrule_details_facet_widgets = function (spec) {
 
     spec.widgets.push(
         {
-            $factory: IPA.collapsible_section,
+            $factory: IPA.section,
             name: 'user',
             label: '@i18n:objects.hbacrule.user',
             widgets: [
@@ -359,7 +359,7 @@ var add_hbacrule_details_facet_widgets = function (spec) {
 
     spec.widgets.push(
         {
-            $factory: IPA.collapsible_section,
+            $factory: IPA.section,
             name: 'host',
             label: '@i18n:objects.hbacrule.host',
             widgets: [
@@ -432,7 +432,7 @@ var add_hbacrule_details_facet_widgets = function (spec) {
 
     spec.widgets.push(
     {
-            $factory: IPA.collapsible_section,
+            $factory: IPA.section,
             name: 'service',
             label: '@i18n:objects.hbacrule.service',
             widgets: [
diff --git a/install/ui/src/freeipa/netgroup.js b/install/ui/src/freeipa/netgroup.js
index 0dd29f1db..d84aca29d 100644
--- a/install/ui/src/freeipa/netgroup.js
+++ b/install/ui/src/freeipa/netgroup.js
@@ -149,7 +149,7 @@ var add_netgroup_details_facet_widgets = function (spec) {
 
     spec.widgets.push(
         {
-            $factory: IPA.collapsible_section,
+            $factory: IPA.section,
             name: 'user',
             label: '@i18n:objects.netgroup.user',
             widgets: [
@@ -233,7 +233,7 @@ var add_netgroup_details_facet_widgets = function (spec) {
 
     spec.widgets.push(
         {
-            $factory: IPA.collapsible_section,
+            $factory: IPA.section,
             name: 'host',
             label: '@i18n:objects.netgroup.host',
             widgets: [
diff --git a/install/ui/src/freeipa/selinux.js b/install/ui/src/freeipa/selinux.js
index af21356f5..62b081fad 100644
--- a/install/ui/src/freeipa/selinux.js
+++ b/install/ui/src/freeipa/selinux.js
@@ -192,7 +192,7 @@ var add_selinux_details_facet_widgets = function (spec) {
 
     spec.widgets.push(
         {
-            $factory: IPA.collapsible_section,
+            $factory: IPA.section,
             name: 'user',
             label: '@i18n:objects.selinuxusermap.user',
             widgets: [
@@ -261,7 +261,7 @@ var add_selinux_details_facet_widgets = function (spec) {
 
     spec.widgets.push(
         {
-            $factory: IPA.collapsible_section,
+            $factory: IPA.section,
             name: 'host',
             label: '@i18n:objects.selinuxusermap.host',
             widgets: [
diff --git a/install/ui/src/freeipa/sudo.js b/install/ui/src/freeipa/sudo.js
index d0b0539ef..d1ab20a93 100644
--- a/install/ui/src/freeipa/sudo.js
+++ b/install/ui/src/freeipa/sudo.js
@@ -312,7 +312,7 @@ var add_sudorule_details_facet_widgets = function (spec) {
 
     spec.widgets.push(
         {
-            $factory: IPA.collapsible_section,
+            $factory: IPA.section,
             name: 'user',
             label: '@i18n:objects.sudorule.user',
             widgets: [
@@ -387,7 +387,7 @@ var add_sudorule_details_facet_widgets = function (spec) {
 
     spec.widgets.push(
     {
-            $factory: IPA.collapsible_section,
+            $factory: IPA.section,
             name: 'host',
             label: '@i18n:objects.sudorule.host',
             widgets: [
@@ -473,7 +473,7 @@ var add_sudorule_details_facet_widgets = function (spec) {
 
     spec.widgets.push(
         {
-            $factory: IPA.collapsible_section,
+            $factory: IPA.section,
             name: 'command',
             label: '@i18n:objects.sudorule.command',
             widgets: [
@@ -589,7 +589,7 @@ var add_sudorule_details_facet_widgets = function (spec) {
 
     spec.widgets.push(
         {
-            $factory: IPA.collapsible_section,
+            $factory: IPA.section,
             name: 'runas',
             label: '@i18n:objects.sudorule.runas',
             widgets: [
@@ -682,7 +682,7 @@ IPA.sudo.options_section = function(spec) {
 
     spec = spec || {};
 
-    var that = IPA.collapsible_section(spec);
+    var that = IPA.section(spec);
 
     function setup_table(){
         that.table = IPA.table_widget({
diff --git a/install/ui/src/freeipa/widget.js b/install/ui/src/freeipa/widget.js
index bb668c371..34a9660c1 100644
--- a/install/ui/src/freeipa/widget.js
+++ b/install/ui/src/freeipa/widget.js
@@ -4104,13 +4104,12 @@ IPA.composite_widget = function(spec) {
  * @class
  * @extends IPA.composite_widget
  */
-IPA.collapsible_section = function(spec) {
+IPA.section = function(spec) {
 
     spec = spec || {};
 
     var that = IPA.composite_widget(spec);
 
-    that.collabsible = !!spec.collabsible;
     that.show_header = spec.show_header === undefined ? true : spec.show_header;
 
     that.create = function(container) {
@@ -4135,18 +4134,6 @@ IPA.collapsible_section = function(spec) {
             name: that.name
         }).appendTo(container);
 
-        if (that.collabsible) {
-            that.icon = $('<span/>', {
-                name: 'icon',
-                'class': 'icon section-expand '+IPA.expanded_icon
-            }).appendTo(that.header);
-
-            that.header.click(function() {
-                var visible = that.content_container.is(":visible");
-                that.toggle(!visible);
-            });
-        }
-
         that.header.append(' ');
 
         that.header.append(that.label);
@@ -4156,16 +4143,6 @@ IPA.collapsible_section = function(spec) {
         that.composite_widget_create(that.content_container);
     };
 
-    that.toggle = function(visible) {
-
-        that.icon.toggleClass(IPA.expanded_icon, visible);
-        that.icon.toggleClass(IPA.collapsed_icon, !visible);
-
-        if (visible != that.content_container.is(":visible")) {
-            that.content_container.slideToggle('slow');
-        }
-    };
-
     return that;
 };
 
@@ -4416,15 +4393,15 @@ exp.fluid_layout = IPA.fluid_layout = function(spec) {
 };
 
 /**
- * Collapsible section with table layout
+ * Section with fluid form layout
  * @class
- * @extends IPA.collapsible_section
+ * @extends IPA.section
  */
 IPA.details_section = function(spec) {
 
     spec = spec || {};
 
-    var that = IPA.collapsible_section(spec);
+    var that = IPA.section(spec);
     that.layout = IPA.build(spec.layout || IPA.fluid_layout);
     that.action_panel = that.build_child(spec.action_panel, {},
                                          { $factory: IPA.action_panel });
@@ -4460,13 +4437,12 @@ IPA.details_section = function(spec) {
 };
 
 /**
- * Collapsible section with table layout
+ * Section with table layout
  * @class
  * @extends IPA.details_section
  */
 IPA.details_table_section = function(spec) {
 
-    spec.collabsible = spec.collabsible === undefined ? true : spec.collabsible;
     spec.layout = spec.layout || IPA.table_layout;
 
     var that = IPA.details_section(spec);