From 13ce0399b010241ecc136eb35196cffdd503b73f Mon Sep 17 00:00:00 2001
From: Christian Hofmaier
Date: Thu, 11 Oct 2018 15:25:20 +0200
Subject: [locationinfo] disable add location when 4 rooms in list in
 default-panel-config - don't close dropdown selection when room selected - if
 4 rooms selected disable the add button - if 4 rooms selected, close the
 dropdown selection - re enable the button when a room gets removed

---
 .../templates/page-config-panel-default.html       | 25 +++++++++++++++++-----
 1 file changed, 20 insertions(+), 5 deletions(-)

diff --git a/modules-available/locationinfo/templates/page-config-panel-default.html b/modules-available/locationinfo/templates/page-config-panel-default.html
index ba493579..f7009764 100644
--- a/modules-available/locationinfo/templates/page-config-panel-default.html
+++ b/modules-available/locationinfo/templates/page-config-panel-default.html
@@ -293,7 +293,7 @@
 
 						</ul>
 						<div class="dropdown pull-right">
-							<button type="button" class="btn btn-success dropdown-toggle" data-toggle="dropdown">
+							<button id="addLocDdBtn" type="button" class="btn btn-success dropdown-toggle" data-toggle="dropdown">
 								<span class="glyphicon glyphicon-plus"></span>
 							</button>
 							<ul class="dropdown-menu" id="location-list">
@@ -309,8 +309,10 @@
 
 		</div>
 	</div>
-	<a href="?do=locationinfo&amp;show=panels" class="btn btn-default">{{lang_cancel}}</a>
-	<button type="submit" class="btn btn-primary">{{lang_save}}</button>
+	<div class="text-right">
+		<a href="?do=locationinfo&amp;show=panels" class="btn btn-default">{{lang_cancel}}</a>
+		<button type="submit" class="btn btn-primary">{{lang_save}}</button>
+	</div>
 </form>
 
 <div class="modal fade" id="no-locations-message" tabindex="-1" role="dialog">
@@ -336,6 +338,7 @@ document.addEventListener("DOMContentLoaded", function () {
 	var $selLocs = $('#selected-locations');
 	var $locList = $('#location-list');
 	var $locInput = $('#locationids');
+	var $addLocDbBtn = $('#addLocDdBtn');
 
 	// Initialize fancy tooltips
 	$('p.helptext').tooltip();
@@ -372,6 +375,7 @@ document.addEventListener("DOMContentLoaded", function () {
 	// Adding/removing locations
 	$locList.find('a').click(function(ev) {
 		ev.preventDefault();
+		ev.stopPropagation();
 		var $this = $(this);
 		var name = $this.find('.name').text();
 		var id = $this.data('lid');
@@ -388,13 +392,24 @@ document.addEventListener("DOMContentLoaded", function () {
 
 	function addLocation(id, name) {
 		$selLocs.find('li[data-lid="' + id + '"]').remove();
-		var delButton = $('<button class="btn btn-danger btn-xs" type="button">').append($('<span class="glyphicon glyphicon-remove">')).click(delParent);
-		$selLocs.append($('<li>').attr('data-lid', id).text(name).prepend(delButton));
+		var delButton = $('<button style="margin-right: 5px;" class="btn btn-danger btn-xs" type="button">').append($('<span class="glyphicon glyphicon-remove">')).click(delParent);
+		$selLocs.append($('<li style="margin-top: 5px;">').attr('data-lid', id).text(name).prepend(delButton));
+
+		if ($selLocs.find('li').length >= 4) {
+			$addLocDbBtn.prop("disabled",true);
+			var $addLocBtnParent = $addLocDbBtn.parent();
+			if ($addLocBtnParent.hasClass('open')) {
+				$addLocBtnParent.removeClass('open');
+			}
+		}
 	}
 
 	function delParent() {
 		$(this).parent().remove();
 		serializeLocs();
+		if ($selLocs.find('li').length < 4) {
+			$addLocDbBtn.prop("disabled",false);
+		}
 	}
 
 	function serializeLocs() {
-- 
cgit v1.2.3-55-g7522