nicer and simpler logout

diff --git a/auth/js/logout.js b/auth/js/logout.js
new file mode 100644 (file)
index 0000000..63bc197
--- /dev/null
+++ b/auth/js/logout.js
@@ -0,0 +1,6 @@
+function logout () {
+    var username=$(this).attr('username');
+    var msg="Are you sure you want to logout as " + username + " ?";
+    if (confirm(msg)) window.location="/logout/";
+}
+$(document).ready(function() { $('#logout').click(logout); })

diff --git a/auth/templates/widget-logout.html b/auth/templates/widget-logout.html
deleted file mode 100644 (file)
index 28e91bc..0000000
--- a/auth/templates/widget-logout.html
+++ /dev/null
@@ -1,3 +0,0 @@
-{% insert_str prelude "css/logout.css" %}
-<p> xxx This needs a bit of ironing out </p>
-<a class='logout' href='/logout/confirm/'>Confirm logout</a>

diff --git a/auth/views.py b/auth/views.py
index 6490ef5..c26bc1d 100644 (file)
--- a/auth/views.py
+++ b/auth/views.py
@@ -44,13 +44,6 @@ def login_user(request):
 
 # hard question : where should we redirect requests to logout if user is not logged in ?
 def logout_user (request):
-    # xxx check that we're indeed logged in
-    if not request.user.is_authenticated():
-        return HttpResponseRedirect ('/')
-    return render_to_response('view-logout.html',{'username':the_user(request)},
-                              context_instance=RequestContext(request))
-
-def do_logout_user (request):
     # xxx check that we're indeed logged in
     if not request.user.is_authenticated():
         return HttpResponseRedirect ('/')

diff --git a/myslice/urls.py b/myslice/urls.py
index 669961f..5c18529 100644 (file)
--- a/myslice/urls.py
+++ b/myslice/urls.py
@@ -34,7 +34,6 @@ urlpatterns = patterns(
     (r'^accounts/login/$', 'auth.views.login_user'),
     (r'^login/?$', 'auth.views.login_user'),
     (r'^logout/?$', 'auth.views.logout_user'),
-    (r'^logout/confirm/?$', 'auth.views.do_logout_user'),
     #
     # the manifold proxy
     #

diff --git a/trash/templates/view-tab.html b/trash/templates/view-tab.html
index 94a541a..1e48e5c 100644 (file)
--- a/trash/templates/view-tab.html
+++ b/trash/templates/view-tab.html
@@ -30,6 +30,12 @@ $(function(){console.log('hey!');$('#tooltipid').tooltip({'selector':'','placeme
 
 a first <a id='tooltipid' href="#" data-toggle="tooltip" data-original-title="what the tooltip displays"> attempt with tooltips </a>
 
+Let us now try to attach <button id='mylougout' class='btn btn-danger' data='mydata'>a click function to a button</button>
+<script>
+function foo() {alert ('foo' + $(this).attr('data'));}
+$(function() {$('#mylougout').click(foo);})
+</script>
+
 <div>
  {{ lorem }}   Lorem <span class='bold'>ipsum dolor</span> <code> counter += 1
   </code> sit amet, consectetuer

diff --git a/unfold/js/unfold-helper.js b/unfold/js/unfold-helper.js
index 6d8e01c..71c5111 100644 (file)
--- a/unfold/js/unfold-helper.js
+++ b/unfold/js/unfold-helper.js
@@ -3,6 +3,6 @@ var unfold = {
        return "<button class='unfold-warning btn btn-warning'>"+text+"</button>"; 
     },
     error : function(text){ 
-       return "<button class='unfold-error btn btn-error'>"+text+"</button>"; 
+       return "<button class='unfold-error btn btn-danger'>"+text+"</button>"; 
     },
 } // global unfold

diff --git a/views/css/topmenu.css b/views/css/topmenu.css
index 7916386..3d48182 100644 (file)
--- a/views/css/topmenu.css
+++ b/views/css/topmenu.css
@@ -4,6 +4,8 @@ body {
     padding-bottom: 20px;
 }
 
-div.topmenu {
-    padding-top: 10px;
+div.topmenu { padding-top: 10px; }
+button#logout { 
+    margin-left: 8px; 
+    margin-bottom: 8px; 
 }

diff --git a/views/templates/view-logout.html b/views/templates/view-logout.html
deleted file mode 100644 (file)
index 16b3cf4..0000000
--- a/views/templates/view-logout.html
+++ /dev/null
@@ -1,6 +0,0 @@
-{% extends 'layout-unfold1.html' %}
-
-{% block unfold1_main %}
-{% include 'widget-logout.html' %}
-{% endblock %}
-

diff --git a/views/templates/widget-topmenu.html b/views/templates/widget-topmenu.html
index 97249f0..8b8ff57 100644 (file)
--- a/views/templates/widget-topmenu.html
+++ b/views/templates/widget-topmenu.html
@@ -2,8 +2,9 @@
 {% insert_str prelude "js/bootstrap.js" %}
 {% insert_str prelude "css/bootstrap.css" %}
 {% insert_str prelude "css/topmenu.css" %}
+{% insert_str prelude "js/logout.js" %}
 
-<div id='topmenu' class="navbar navbar-fixed-top">
+<div class="navbar navbar-fixed-top">
   <div class="navbar-inner">
     <div class="container-fluid">
       <a class="brand" href="/"><img src="{{ STATIC_URL }}img/myslice-logo.png" height="30" alt="MySlice logo" /></a>
@@ -17,12 +18,8 @@
          {% endif %}
          {% endfor %}
        </ul>
-       <p class="navbar-text pull-right">
-         {% if username %}
-       Logged in as <a href="/logout/">{{ username }}</a></p>
-         {% else %}
-         Not logged in
-         {% endif %}
+       <span class="navbar-text pull-right"> {% if username %} Logged in as {{ username }}<button id='logout' class='btn btn-danger' username='{{ username }}'>Logout</a>
+{% else %}Not logged in{% endif %}</span>
       </div><!--nav-collapse-->
     </div><!--container-fluid-->
   </div><!--navbar-inner-->