From cfd001702ff8b76b54f468ff3cce92abbb105f15 Mon Sep 17 00:00:00 2001 From: Anton Lundin Date: Wed, 17 Jun 2015 21:46:38 +0200 Subject: Make the expand/collapse buttons usefull on trips There where some issues with trying to access non-existing dom elements when clicking them when showing trips. Instead of fixing that issue, this actually makes them do something useful, expanding and collapsing the trips. Signed-off-by: Anton Lundin Signed-off-by: Dirk Hohndel --- theme/list_lib.js | 34 ++++++++++++++++++++++++---------- 1 file changed, 24 insertions(+), 10 deletions(-) diff --git a/theme/list_lib.js b/theme/list_lib.js index 17f52061d..0459aad81 100644 --- a/theme/list_lib.js +++ b/theme/list_lib.js @@ -134,11 +134,18 @@ function view_pagging(start, end) */ function expandAll() { - for (var i = start; i < start + sizeofpage; i++) { - if (i >= itemsToShow.length) - break; - unexpand(document.getElementById(itemsToShow[i])); - items[itemsToShow[i]].expanded = false; + if (tripsShown) { + for (var i = 0 ; i < trips.length ; i++) { + if (trips[i].expanded === false) + expand_trip(i); + } + } else { + for (var i = start; i < start + sizeofpage; i++) { + if (i >= itemsToShow.length) + break; + unexpand(document.getElementById(itemsToShow[i])); + items[itemsToShow[i]].expanded = false; + } } } @@ -147,11 +154,18 @@ function expandAll() */ function collapseAll() { - for (var i = start; i < start + sizeofpage; i++) { - if (i >= itemsToShow.length) - break; - expand(document.getElementById(itemsToShow[i])); - items[itemsToShow[i]].expanded = true; + if (tripsShown) { + for (var i = 0 ; i < trips.length ; i++) { + if (trips[i].expanded === true) + unexpand_trip(i); + } + } else { + for (var i = start; i < start + sizeofpage; i++) { + if (i >= itemsToShow.length) + break; + expand(document.getElementById(itemsToShow[i])); + items[itemsToShow[i]].expanded = true; + } } } -- cgit v1.2.3-70-g09d2