Usability & design improvements

15 files changed, 122 insertions, 80 deletions

diff --git a/app/views/issues/_show.html.haml b/app/views/issues/_show.html.haml
index ef9e156020e..a1f48383537 100644
--- a/app/views/issues/_show.html.haml
+++ b/app/views/issues/_show.html.haml
@@ -12,7 +12,7 @@
   = image_tag gravatar_icon(issue.assignee_email), :class => "avatar"
   %span.update-author
     assigned to
-    %strong= link_to_issue_assignee(issue)
+    %strong= issue.assignee_name
     - if issue.critical
       %span.label.important critical
     - if issue.today?
diff --git a/app/views/issues/index.html.haml b/app/views/issues/index.html.haml
index b83c4f472ae..7a8f4873b79 100644
--- a/app/views/issues/index.html.haml
+++ b/app/views/issues/index.html.haml
@@ -9,30 +9,30 @@
       = link_to new_project_issue_path(@project), :class => "right btn small", :title => "New Issue", :remote => true do 
         New Issue
   %hr
-  %div#issues-table-holder
-    .row
-      .span8
-        %ul.pills.left
-          %li{:class => ("active" if (params[:f] == "0" || !params[:f]))}
-            = link_to project_issues_path(@project, :f => 0) do 
-              Open
-          %li{:class => ("active" if params[:f] == "2")}
-            = link_to project_issues_path(@project, :f => 2) do 
-              Closed
-          %li{:class => ("active" if params[:f] == "3")}
-            = link_to project_issues_path(@project, :f => 3) do 
-              To Me
-          %li{:class => ("active" if params[:f] == "1")}
-            = link_to project_issues_path(@project, :f => 1) do 
-              All
+  %div#issues-table-holder.ui-box
+    .title
+      .row
+        .span8
+          %ul.pills.left
+            %li{:class => ("active" if (params[:f] == "0" || !params[:f]))}
+              = link_to project_issues_path(@project, :f => 0) do 
+                Open
+            %li{:class => ("active" if params[:f] == "2")}
+              = link_to project_issues_path(@project, :f => 2) do 
+                Closed
+            %li{:class => ("active" if params[:f] == "3")}
+              = link_to project_issues_path(@project, :f => 3) do 
+                To Me
+            %li{:class => ("active" if params[:f] == "1")}
+              = link_to project_issues_path(@project, :f => 1) do 
+                All
 
-      .span2.right
-        = form_tag search_project_issues_path(@project), :method => :get, :remote => true, :id => "issue_search_form", :class => :right  do
-          = hidden_field_tag :project_id, @project.id, { :id => 'project_id' }
-          = hidden_field_tag :status, params[:f]
-          = search_field_tag :issue_search, nil, { :placeholder => 'Search', :class => 'issue_search' }
+        .span3.right
+          = form_tag search_project_issues_path(@project), :method => :get, :remote => true, :id => "issue_search_form", :class => :right  do
+            = hidden_field_tag :project_id, @project.id, { :id => 'project_id' }
+            = hidden_field_tag :status, params[:f]
+            = search_field_tag :issue_search, nil, { :placeholder => 'Search', :class => 'issue_search' }
 
-    %hr
     %ul#issues-table.unstyled= render "issues"
 
 :javascript
diff --git a/app/views/issues/show.html.haml b/app/views/issues/show.html.haml
index abf5fc11ed3..170d6831fb4 100644
--- a/app/views/issues/show.html.haml
+++ b/app/views/issues/show.html.haml
@@ -14,6 +14,11 @@
       = link_to edit_project_issue_path(@project, @issue), :class => "btn small" do 
         Edit
 
+.back_link
+  = link_to project_issues_path(@project) do 
+    ← To issues list
+
+
 %hr
 - if @issue.closed
   .alert-message.error Closed
diff --git a/app/views/keys/index.html.haml b/app/views/keys/index.html.haml
index e6979f5e212..002089a00ec 100644
--- a/app/views/keys/index.html.haml
+++ b/app/views/keys/index.html.haml
@@ -4,7 +4,7 @@
   
 %hr
 
-%div#keys-table{ :class => "update-data ui-box ui-box-small ui-box-big" }
+%div#keys-table
   %ul.unstyled
     - @keys.each do |key|
       = render(:partial => 'show', :locals => {:key => key})
diff --git a/app/views/layouts/_project_menu.html.haml b/app/views/layouts/_project_menu.html.haml
index 6d12f30fd60..c47a1863922 100644
--- a/app/views/layouts/_project_menu.html.haml
+++ b/app/views/layouts/_project_menu.html.haml
@@ -1,5 +1,5 @@
 %nav.main_menu
-  = link_to project_path(@project), :class => project_tab_class do 
+  = link_to project_path(@project), :class => "#{project_tab_class}", :title => "Project" do 
     Project
 
   - if @project.repo_exists?
diff --git a/app/views/layouts/_projects_side.html.haml b/app/views/layouts/_projects_side.html.haml
index 2edd99f5767..3b567044e34 100644
--- a/app/views/layouts/_projects_side.html.haml
+++ b/app/views/layouts/_projects_side.html.haml
@@ -4,16 +4,16 @@
       You can create at least
       = current_user.projects_limit
       projects. Click on button to add a new one
-      %hr
-      = link_to new_project_path, :class => "btn small" do 
-        New Project » 
+      .link_holder
+        = link_to new_project_path, :class => "" do 
+          New Project » 
 
   - if current_user.is_admin?
     .widget
       You have administrator privilegies. You can configure application following this button:
-      %hr
-      = link_to admin_root_path, :class => "btn small", :title => "Admin"  do 
-        Visit Admin Area »
+      .link_holder
+        = link_to admin_root_path, :class => "", :title => "Admin"  do 
+          Visit Admin Area »
 
   - if current_user.projects.count > 0
     = render "widgets/recent_projects"
diff --git a/app/views/layouts/profile.html.haml b/app/views/layouts/profile.html.haml
index 8db7af75d81..dd7030f4ff1 100644
--- a/app/views/layouts/profile.html.haml
+++ b/app/views/layouts/profile.html.haml
@@ -10,7 +10,7 @@
         = link_to "Password & token", profile_password_path,  :class => current_page?(:controller => "profile", :action => :password) ? "current" : nil
         = link_to "Design", profile_design_path, :class => current_page?(:controller => "profile", :action => :design) ? "current" : nil
         = link_to keys_path,  :class => controller.controller_name == "keys" ? "current" : nil do
-          Keys
+          SSH Keys
       .content
         .row
           .span12= yield
diff --git a/app/views/merge_requests/_merge_request.html.haml b/app/views/merge_requests/_merge_request.html.haml
index 5852c18cfc8..033c8cf76c4 100644
--- a/app/views/merge_requests/_merge_request.html.haml
+++ b/app/views/merge_requests/_merge_request.html.haml
@@ -1,7 +1,7 @@
 %li.wll
   = image_tag gravatar_icon(merge_request.author_email), :class => "avatar"
   %span.update-author
-    %strong= link_to_merge_request_author(merge_request)
+    %strong= merge_request.author_name
     authored
     = time_ago_in_words(merge_request.created_at)
     ago
diff --git a/app/views/merge_requests/index.html.haml b/app/views/merge_requests/index.html.haml
index ea2f2433aa2..3a23d67321f 100644
--- a/app/views/merge_requests/index.html.haml
+++ b/app/views/merge_requests/index.html.haml
@@ -6,14 +6,21 @@
 
 %hr
 
-%ul.pills
-  %li{:class => ("active" if (params[:f] == "0" || !params[:f]))}
-    = link_to project_merge_requests_path(@project, :f => 0) do 
-      Open
-  %li{:class => ("active" if params[:f] == "2")}
-    = link_to project_merge_requests_path(@project, :f => 2) do 
-      Closed
+.ui-box
+  .title
+    %ul.pills
+      %li{:class => ("active" if (params[:f] == "0" || !params[:f]))}
+        = link_to project_merge_requests_path(@project, :f => 0) do 
+          Open
+      %li{:class => ("active" if params[:f] == "2")}
+        = link_to project_merge_requests_path(@project, :f => 2) do 
+          Closed
+      %li{:class => ("active" if params[:f] == "3")}
+        = link_to project_merge_requests_path(@project, :f => 3) do 
+          To Me
+      %li{:class => ("active" if params[:f] == "1")}
+        = link_to project_merge_requests_path(@project, :f => 1) do 
+          All
 
-%hr
-%ul.unstyled= render @merge_requests
+  %ul.unstyled= render @merge_requests
 
diff --git a/app/views/merge_requests/show.html.haml b/app/views/merge_requests/show.html.haml
index 132f2063ddc..b270175d105 100644
--- a/app/views/merge_requests/show.html.haml
+++ b/app/views/merge_requests/show.html.haml
@@ -19,6 +19,9 @@
       = link_to edit_project_merge_request_path(@project, @merge_request), :class => "btn small" do 
         Edit
 
+.back_link
+  = link_to project_merge_requests_path(@project) do 
+    ← To merge requests
 %hr
 - if @merge_request.closed
   .alert-message.error Closed
diff --git a/app/views/projects/show.html.haml b/app/views/projects/show.html.haml
index 7886daec1da..91828006df5 100644
--- a/app/views/projects/show.html.haml
+++ b/app/views/projects/show.html.haml
@@ -1,6 +1,11 @@
 = render "project_head"
 %h3
   = @project.name
+  - if can? current_user, :download_code, @project
+    = link_to "Download", archive_project_repository_path(@project), :class => "btn small right"
+.back_link
+  = link_to projects_path do 
+    ← To projects list
 %hr
 .alert-message.block-message.warning
   .input
@@ -10,7 +15,8 @@
 
 = simple_format @project.description
 
-%h5.cgray Recent Activity
-.content_list= render "feed"
+.ui-box
+  %h5.cgray Recent Activity
+  .content_list= render "feed"
 
 
diff --git a/app/views/repositories/show.html.haml b/app/views/repositories/show.html.haml
index 6f71ae7c86f..e73db6f053f 100644
--- a/app/views/repositories/show.html.haml
+++ b/app/views/repositories/show.html.haml
@@ -6,6 +6,16 @@
 
 
 %hr
+.entry
+  %p
+    Last commit was 
+    %small
+      %code= @activities.first.commit.id.to_s[0..10]
+
+    = time_ago_in_words(@activities.first.commit.committed_date)
+    ago to 
+    = link_to project_commits_path(@project, :ref => @activities.first.head.name), :class => "visible_link" do 
+      %span.label= @activities.first.head.name
 .alert-message.block-message.warning
   .input
     .input-prepend
@@ -13,20 +23,12 @@
       = text_field_tag :project_clone, @project.url_to_repo, :class => "xlarge one_click_select git_clone_url"
 
 
-%p
-  Last commit was 
-  %small
-    %code= @activities.first.commit.id.to_s[0..10]
 
-  = time_ago_in_words(@activities.first.commit.committed_date)
-  ago to 
-  = link_to project_commits_path(@project, :ref => @activities.first.head.name), :class => "visible_link" do 
-    %span.label= @activities.first.head.name
+.ui-box
+  %h5.cgray
+    Recent Branches
 
-%h4.cgray
-  Recent Branches
-
-%ul.unstyled
-  - @activities.each do |update|
-    = render "repositories/feed", :update => update, :project => @project
+  %ul.unstyled
+    - @activities.each do |update|
+      = render "repositories/feed", :update => update, :project => @project
 
diff --git a/app/views/team_members/show.html.haml b/app/views/team_members/show.html.haml
index 57f00ea10e0..fcf2db9ba84 100644
--- a/app/views/team_members/show.html.haml
+++ b/app/views/team_members/show.html.haml
@@ -1,14 +1,18 @@
 - allow_admin = can? current_user, :admin_project, @project
 - user = @team_member.user
-%div
-  = link_to team_project_path(@project), :class => "btn right" do 
-    Team »
-
-  .media-grid
-    = link_to "#" do
-      = image_tag gravatar_icon(user.email, 90), :class => "thumbnail"
-    %h3.media_h= user.name
-
+.media-grid
+  = link_to "#" do
+    = image_tag gravatar_icon(user.email, 60), :class => "thumbnail", :width => 60
+  %h3.media_h
+    = user.name
+    %br
+    %small= user.email
+
+.back_link
+  = link_to team_project_path(@project), :class => "" do 
+    ← To team list
+
+%hr
 %table.no-borders
   %tr
     %td Name
@@ -23,7 +27,10 @@
     %td= @team_member.created_at.stamp("Aug 21, 2011")
 
   %tr
-    %td Project Access
+    %td
+      Project Access
+      (#{link_to "read more", help_permissions_path, :class => "vlink"})
+
     %td
       = form_for(@team_member, :as => :team_member, :url => project_team_member_path(@project, @team_member)) do |f|
         = f.select :project_access, options_for_select(Project.access_options, @team_member.project_access), {}, :class => "project-access-select", :disabled => !allow_admin
diff --git a/app/views/widgets/_project_member.html.haml b/app/views/widgets/_project_member.html.haml
index ac217b379a0..911af9517ef 100644
--- a/app/views/widgets/_project_member.html.haml
+++ b/app/views/widgets/_project_member.html.haml
@@ -3,20 +3,31 @@
   .media-grid
     %li
       = link_to project_team_member_path(@project, member), :title => current_user.name do 
-        = image_tag gravatar_icon(current_user.email, 90), :class => "thumbnail"
-  .row
-    .span4
-      %h4
-        = truncate(current_user.name, :lenght => 24)
-      - if @project.owner == current_user
-        %span.label Project Owner
+        = image_tag gravatar_icon(current_user.email, 60), :class => "thumbnail", :width => 60
+  %h4
+    Hi, 
+    = truncate current_user.first_name, :lenght => 24
+    !
+  %p 
+    - if @project.issues_enabled
+      Assigned issues:
+      = current_user.assigned_issues.count
+      %br
+    - if @project.merge_requests_enabled
+      Assigned merge request:
+      = current_user.assigned_merge_requests.count
+      %br
+      Your merge requests:
+      = current_user.assigned_merge_requests.count
+  .link_holder
+    = link_to project_team_member_path(@project, member), :title => current_user.name do 
+      = "Access: #{member.project_access_human} »"
 
-    .span3
-      %span.label= member.project_access_human
 
-  - if can? current_user, :write_project, @project
+
+-#- if can? current_user, :write_project, @project
+  .widget
     - if @project.issues_enabled && @project.merge_requests_enabled
-      %hr 
       .span3
         %p You have access to create new issue or merge request.
         %div
diff --git a/app/views/widgets/_recent_projects.html.haml b/app/views/widgets/_recent_projects.html.haml
index 394aa1ad8ca..7f1028bc683 100644
--- a/app/views/widgets/_recent_projects.html.haml
+++ b/app/views/widgets/_recent_projects.html.haml
@@ -7,4 +7,5 @@
         %li
           = link_to project_path(project) do
             = project.name
-    = link_to "More  » ", projects_path
+    .link_holder
+      = link_to "Projects » ", projects_path