Drop expandable.js

doc/80-Upgrading.md

@@ -8,6 +8,8 @@ v2.6 to v2.8 requires to follow the instructions for v2.7 too.
 **Framework changes affecting third-party code**
 
 * Asset support for modules (#3961) introduced with v2.8 has now been removed.
+* `expandable-toggle`-support has been removed. Use `class="collapsible" data-visible-height=0`
+  to achieve the same effect. (Available since v2.7.0)
 
 ## Upgrading to Icinga Web 2 2.9.1
 

library/Icinga/Web/JavaScript.php

@@ -38,7 +38,6 @@ class JavaScript
         'js/icinga/behavior/form.js',
         'js/icinga/behavior/actiontable.js',
         'js/icinga/behavior/flyover.js',
-        'js/icinga/behavior/expandable.js',
         'js/icinga/behavior/filtereditor.js',
         'js/icinga/behavior/selectable.js',
         'js/icinga/behavior/modal.js',

public/css/icinga/widgets.less

@@ -654,51 +654,6 @@ ul.tree li a.error:hover {
   }
 }
 
-input[type="checkbox"].expandable-toggle {
-  display: none;
-
-  & + label {
-    float: right;
-    margin-right: 1em;
-
-    cursor: pointer;
-    user-select: none;
-    -webkit-user-select: none;
-    -moz-user-select: none;
-    -ms-user-select: none;
-  }
-
-  & ~ .expandable-content {
-    clear: right;  // Because the label is floating right
-  }
-
-  &:checked ~ .expandable-content {
-    -webkit-transition: opacity 0.2s linear;
-    -moz-transition: opacity 0.2s linear;
-    -o-transition: opacity 0.2s linear;
-    transition: opacity 0.2s linear;
-  }
-
-  &:not(:checked) ~ .expandable-content {
-    height: 0;
-    opacity: 0;
-    visibility: hidden;
-
-    -webkit-transition: opacity 0.2s linear, visibility 0.2s;
-    -moz-transition: opacity 0.2s linear, visibility 0.2s;
-    -o-transition: opacity 0.2s linear, visibility 0.2s;
-    transition: opacity 0.2s linear, visibility 0.2s;
-  }
-
-  &:checked ~ label .expandable-expand-label {
-    display: none;
-  }
-
-  &:not(:checked) ~ label .expandable-collapse-label {
-    display: none;
-  }
-}
-
 html.no-js .progress-label {
   display: none;
 }

public/js/icinga/behavior/expandable.js

@@ -1,47 +0,0 @@
-/* Icinga Web 2 | (c) 2017 Icinga Development Team | GPLv2+ */
-
-/**
- * Icinga.Behavior.Expandable
- *
- * Initially collapsed, but expandable content
- */
-(function(Icinga, $) {
-
-    'use strict';
-
-    var expandedExpandables = {};
-
-    function Expandable(icinga) {
-        Icinga.EventListener.call(this, icinga);
-
-        this.on('rendered', this.onRendered, this);
-        this.on('click', this.onClick, this);
-    }
-
-    Expandable.prototype = new Icinga.EventListener();
-
-    Expandable.prototype.onRendered = function(event) {
-        $(event.target).find('.expandable-toggle').each(function() {
-            var $this = $(this);
-
-            if (typeof expandedExpandables['#' + $this.attr('id')] !== 'undefined') {
-                $this.prop('checked', true);
-            }
-        });
-    };
-
-    Expandable.prototype.onClick = function(event) {
-        var $expandableToggle = $(event.target);
-
-        if ($expandableToggle.prop('checked')) {
-            expandedExpandables['#' + $expandableToggle.attr('id')] = null;
-        } else {
-            delete expandedExpandables['#' + $expandableToggle.attr('id')];
-        }
-    };
-
-    Icinga.Behaviors = Icinga.Behaviors || {};
-
-    Icinga.Behaviors.Expandable = Expandable;
-
-}) (Icinga, jQuery);