add correct page titles

app/controllers/application_controller.rb

@@ -24,13 +24,13 @@ class ApplicationController < ActionController::Base
   def mobile_except_ipad
     if is_mobile_device?
       if request.env["HTTP_USER_AGENT"].include? "iPad"
-        session[:mobile_view] = false 
+        session[:mobile_view] = false
       else
         session[:mobile_view] = true
       end
     end
   end
-  
+
   def count_requests
     @request_count = current_user.requests_for_me.count if current_user
   end

app/helpers/application_helper.rb

@@ -9,6 +9,16 @@ module ApplicationHelper
     false
   end
 
+  def page_title text=nil
+    title = ""
+    if text.blank?
+      title = "#{current_user.name} | " if current_user
+    else
+      title = "#{text} | "
+    end
+    title += "DIASPORA*"
+  end
+
   def aspects_with_post aspects, post
     aspects.select do |a|
       post.aspect_ids.include?(a.id)

app/views/aspects/manage.html.haml

@@ -5,6 +5,9 @@
 - content_for :head do
   = include_javascripts :aspects
 
+- content_for :page_title do
+  = "Manage aspects"
+
 #section_header
   %h2=t('.manage_aspects')
   .right{:style=>"top:0;"}

app/views/aspects/show.html.haml

@@ -2,6 +2,9 @@
 -#   licensed under the Affero General Public License version 3 or later.  See
 -#   the COPYRIGHT file.
 
+- content_for :page_title do
+  = @aspect.name
+
 .span-24.last
   %h2{:style=>"position:relative;margin-bottom:0;"}
     = @aspect

app/views/layouts/application.html.haml

@@ -8,10 +8,7 @@
     %meta{:charset => 'utf-8'}
 
     %title
-      - if current_user
+      = page_title yield(:page_title)
-        = "#{current_user.name} | DIASPORA*"
-      - else
-        DIASPORA*
 
     %meta{'http-equiv' => 'X-UA-Compatible', :content => 'IE=edge,chrome=1'}
 

app/views/people/_edit.html.haml

@@ -2,6 +2,8 @@
 -#   licensed under the Affero General Public License version 3 or later.  See
 -#   the COPYRIGHT file.
 
+- content_for :page_title do
+  = "Edit profile"
 
 = form_tag person_path(@person), :method => :put, :multipart => true, :id => 'update_profile_form' do
   %h3

app/views/people/index.html.haml

@@ -2,6 +2,9 @@
 -#   licensed under the Affero General Public License version 3 or later.  See
 -#   the COPYRIGHT file.
 
+- content_for :page_title do
+  = "Search"
+
 .span-15.prepend-5.last
   %h2
     -if params[:q].blank?

app/views/people/show.html.haml

@@ -2,6 +2,8 @@
 -#   licensed under the Affero General Public License version 3 or later.  See
 -#   the COPYRIGHT file.
 
+- content_for :page_title do
+  = @person.name
 
 .span-24.last
   #author_info

app/views/services/index.html.haml

@@ -2,6 +2,9 @@
 -#   licensed under the Affero General Public License version 3 or later.  See
 -#   the COPYRIGHT file.
 
+- content_for :page_title do
+  = "Edit services"
+
 #section_header
   %h2
     = t('settings')

app/views/users/edit.html.haml

@@ -2,6 +2,9 @@
 -#   licensed under the Affero General Public License version 3 or later.  See
 -#   the COPYRIGHT file.
 
+- content_for :page_title do
+  = "Edit account"
+
 :javascript
   $(document).ready(function(){
     $("#settings_nav li > a").live("click", function() {