From 23a5edbe721348f7e570ead73e73bb7b312bfcb0 Mon Sep 17 00:00:00 2001
From: Ramon Wijnands <rayman747@hotmail.com>
Date: Fri, 7 Nov 2014 21:15:01 +0100
Subject: [PATCH] move the profile link out of the login buttons

---
 client/components/header/header.html | 53 ++++++++++++++--------------
 client/components/header/header.js   | 13 +------
 2 files changed, 28 insertions(+), 38 deletions(-)

diff --git a/client/components/header/header.html b/client/components/header/header.html
index 82f9f02..9005829 100644
--- a/client/components/header/header.html
+++ b/client/components/header/header.html
@@ -1,24 +1,33 @@
 <template name="header">
   <div class="navbar navbar-default" role="navigation">
-      <div class="navbar-header">
-        <button type="button" class="navbar-toggle" data-toggle="collapse" data-target=".navbar-collapse">
-          <span class="sr-only">Toggle navigation</span>
-          <span class="icon-bar"></span>
-          <span class="icon-bar"></span>
-          <span class="icon-bar"></span>
-        </button>
-        <a class="navbar-brand" href="{{pathFor 'home'}}">Fusic <small>Realtime Social Music Collaboration</small></a>
-      </div>
 
-		<div class="navbar-right container-fluid">
-			<ul id="login" class="nav navbar-nav">
-			  {{> loginButtons }}
-			</ul>
-		</div>
+    <div class="navbar-header">
+      <button type="button" class="navbar-toggle" data-toggle="collapse" data-target=".navbar-collapse">
+        <span class="sr-only">Toggle navigation</span>
+        <span class="icon-bar"></span>
+        <span class="icon-bar"></span>
+        <span class="icon-bar"></span>
+      </button>
+      <a class="navbar-brand" href="{{pathFor 'home'}}">Fusic <small>Realtime Social Music Collaboration</small></a>
+    </div>
 
-	  {{#if currentUser}}
-      <div class="collapse navbar-collapse">
+    <div class="navbar-right container-fluid">
+      {{#with user}}
+        <ul class="nav navbar-nav">
+          <li class="{{ isActivePath regex='profile' }}">
+            <a href="{{pathFor 'userProfile'}}">
+              <span class="glyphicon glyphicon-user"></span> Profile
+            </a>
+          </li>
+        </ul>
+      {{/with}}
+      <ul id="login" class="nav navbar-nav">
+        {{> loginButtons }}
+      </ul>
+    </div>
 
+    {{#if currentUser}}
+      <div class="collapse navbar-collapse">
         <ul class="nav navbar-nav">
           <li class="{{ isActivePath regex='playlists' }}"><a href="{{pathFor 'playlists'}}">Playlists</a></li>
           <li class="{{ isActivePath regex='loved'     }}"><a href="{{pathFor 'loved'}}">Loved songs</a></li>
@@ -28,16 +37,8 @@
             </a>
           </li>
         </ul>
-		</div><!--/.nav-collapse -->
-		{{/if}}
+    </div><!--/.nav-collapse -->
+    {{/if}}
 
   </div>
 </template>
-
-<template name="_loginButtonsAdditionalLoggedInDropdownActions">
-  {{#with user}}
-    <a href="{{pathFor 'userProfile'}}" class="btn btn-default btn-block" role="button">
-      <span class="glyphicon glyphicon-pencil"></span> Edit profile
-    </a>
-  {{/with}}
-</template>
diff --git a/client/components/header/header.js b/client/components/header/header.js
index 3b530aa..d0639c8 100644
--- a/client/components/header/header.js
+++ b/client/components/header/header.js
@@ -1,14 +1,3 @@
-Template.header.rendered = function() {
-	//apparantly still the easiest way to make slight adjustments to accounts-ui package
-	setTimeout(function(){
-		var html = '<span class="glyphicon glyphicon-user"></span>';
-    var current = $("#login a.dropdown-toggle").html();
-    html = html + current;
-		$("#login a.dropdown-toggle").html(html);
-	}, 3000);
-
-}
-
-Template._loginButtonsAdditionalLoggedInDropdownActions.user = function(){
+Template.header.user = function (){
   return Meteor.user();
 }