From 2ec06c1a38bc28ea32ed791f1fb85df697b3474d Mon Sep 17 00:00:00 2001
From: fabrixxm <fabrix.xm@gmail.com>
Date: Sat, 11 Jun 2016 09:58:26 +0200
Subject: [PATCH] use js/modal.js to show Login page in modal

---
 view/theme/frio/js/modal.js            | 10 ++++++++--
 view/theme/frio/templates/nav.tpl      | 22 +++++-----------------
 view/theme/frio/templates/nav_head.tpl |  8 +-------
 3 files changed, 14 insertions(+), 26 deletions(-)

diff --git a/view/theme/frio/js/modal.js b/view/theme/frio/js/modal.js
index 973847509..0a881f10f 100644
--- a/view/theme/frio/js/modal.js
+++ b/view/theme/frio/js/modal.js
@@ -50,7 +50,11 @@ $(document).ready(function(){
 		}
 	});
 
-
+	// Navbar login
+	$("body").on("click", "#nav-login", function(e){
+		e.preventDefault();
+		Dialog.show(this.href, this.dataset.originalTitle || this.title);
+	});
 
 	// Jot nav menu.
 	$("body").on("click", "#jot-modal .jot-nav li a", function(e){
@@ -81,8 +85,9 @@ $(document).ready(function(){
 });
 
 // overwrite Dialog.show from main js to load the filebrowser into a bs modal
-Dialog.show = function(url) {
+Dialog.show = function(url, title="") {
 	var modal = $('#modal').modal();
+	modal.find("#modal-header h4").html(title);
 	modal
 		.find('#modal-body')
 		.load(url, function (responseText, textStatus) {
@@ -129,6 +134,7 @@ Dialog._load = function(url) {
 
 	// try to fetch the hash form the url
 	var match = url.match(/fbrowser\/[a-z]+\/\?mode=none(.*)/);
+	if (match===null) return; //not fbrowser
 	var hash = match[1];
 
 	// initialize the filebrowser
diff --git a/view/theme/frio/templates/nav.tpl b/view/theme/frio/templates/nav.tpl
index a3f479db9..a0ee08884 100644
--- a/view/theme/frio/templates/nav.tpl
+++ b/view/theme/frio/templates/nav.tpl
@@ -240,29 +240,17 @@
 		<div class="hidden-sm hidden-xs">
 			<ul class="nav navbar-nav navbar-right">
 				<li><a href="register" data-toggle="tooltip" title="{{$register.title}}"><i class="fa fa-street-view fa-fw"></i> {{$register.desc}}</a></li>
-				<li data-toggle="tooltip" title="{{$nav.login.3}}">
-					<a href="login?mode=none" 
-						data-remote="false" data-toggle="modal" data-target="#loginModal"
-						><i class="fa fa-sign-in fa-fw"></i></a>
+				<li>
+					<a href="login?mode=none" id="nav-login"
+						data-toggle="tooltip" title="{{$nav.login.3}}">
+							<i class="fa fa-sign-in fa-fw"></i>
+					</a>
 				</li>
 			</ul>
 		</div>
 	</div>
 </nav>
 
-<div class="modal fade" id="loginModal" tabindex="-1" role="dialog" aria-labelledby="loginModalLabel" aria-hidden="true">
-  <div class="modal-dialog">
-	<div class="modal-content">
-	  <div class="modal-header">
-		<button type="button" class="close" data-dismiss="modal" aria-label="Close"><span aria-hidden="true">&times;</span></button>
-		<h4 class="modal-title" id="myModalLabel">{{$nav.login.3}}</h4>
-	  </div>
-	  <div class="modal-body">
-		...
-	  </div>
-	</div>
-  </div>
-</div>
 {{/if}}
 
 {{* The second navbar which contains nav points of the actual page - (nav points are actual handled by this theme throug js *}}
diff --git a/view/theme/frio/templates/nav_head.tpl b/view/theme/frio/templates/nav_head.tpl
index 13842bbe8..d7b50a58d 100644
--- a/view/theme/frio/templates/nav_head.tpl
+++ b/view/theme/frio/templates/nav_head.tpl
@@ -1,12 +1,6 @@
 
 <script>
 $(document).ready(function() {
-	$("#nav-search-input-field").search_autocomplete(baseurl + '/acl');
-	
-	$("#loginModal").on("show.bs.modal", function(e) {
-	    var link = $(e.relatedTarget);
-		$(this).find(".modal-body").load(link.attr("href"));
-	});
-	
+	$("#nav-search-input-field").search_autocomplete(baseurl + '/acl');	
 });
 </script>