Dashboard tabs restyle, messages if no issue or merge request

Dmitriy Zaporozhets
1 parent ed3f3a0d
Showing 5 changed files with 65 additions and 44 deletions Show diff stats
app/assets/stylesheets/projects.css.scss
app/controllers/dashboard_controller.rb
app/views/dashboard/_issues_feed.html.haml
app/views/dashboard/_menu.html.haml
app/views/dashboard/_merge_requests_feed.html.haml
@@ -496,14 +496,23 @@ h4.dash-tabs {
   font-weight: bold; text-transform: uppercase;
   background: #F7F7F7;
   margin-bottom:20px;
+  height:13px;
+
 }
 .dash-button { 
-  margin-right:5px;
-  @include round-borders-all(4px);
-  border: 1px solid #ddd;
+  border-right: 1px solid #ddd;
   background:none;
+  padding: 10px 15px;
+  float:left;
+  position:relative;
+  top:-10px;
+  left:0px;
+  height:13px;
+  &:first-child { 
+    border-left: 1px solid #ddd;
+  }
   &.active { 
     background: #eaeaea;
   }
@@ -11,9 +11,10 @@ class DashboardController &lt; ApplicationController
     end
   end
+  # Get authored or assigned open merge requests
   def merge_requests
     @projects = current_user.projects.all
-    @merge_requests = current_user.assigned_merge_requests.order("created_at DESC").limit(40)
+    @merge_requests = MergeRequest.where("author_id = :id or assignee_id = :id", :id => current_user.id).opened.order("created_at DESC").limit(40)
     respond_to do |format|
       format.html
@@ -21,6 +22,7 @@ class DashboardController &lt; ApplicationController
     end
   end
+  # Get only assigned issues
   def issues
     @projects = current_user.projects.all
     @user   = current_user
 #feeds_content_holder
-  .project-box.project-updates.ui-box.ui-box-small.ui-box-big
-    .data
-      - @issues.each do |update|
-        %a.project-update{:href => dashboard_feed_path(update.project, update)}
-          = image_tag gravatar_icon(update.author_email), :class => "left", :width => 40
-          %span.update-title
-            = truncate update.title, :length => 50
-            .right= update.project.name
-          %span.update-author
-            %strong= update.author_name
-            authored
-            = time_ago_in_words(update.created_at)
-            ago
-          .right
-            - if update.critical
-              %span.tag.high critical
-            - if update.today?
-              %span.tag.today today
+  - unless @issues.empty?
+    .project-box.project-updates.ui-box.ui-box-small.ui-box-big
+      .data
+        - @issues.each do |update|
+          %a.project-update{:href => dashboard_feed_path(update.project, update)}
+            %strong.issue-number= "##{update.id}"
+            %span.update-title
+              = truncate update.title, :length => 35
+              .right= truncate update.project.name
+            %span.update-author
+              %strong= update.author_name
+              authored
+              = time_ago_in_words(update.created_at)
+              ago
+            .right
+              - if update.critical
+                %span.tag.high critical
+              - if update.today?
+                %span.tag.today today
+  - else 
+    %h2
+      No assigned
+      %span.tag.open open
+      issues
 %h4.dash-tabs
-  = link_to "Activities", dashboard_path, :remote => true, :class => "button-small dash-button #{"active" if current_page?(dashboard_path) || current_page?(root_path) }", :id => "activities_slide"
-  = link_to "Issues", dashboard_issues_path, :remote => true, :class => "button-small dash-button #{"active" if current_page?(dashboard_issues_path)}", :id => "issues_slide"
-  = link_to "Merge Requests", dashboard_merge_requests_path, :remote => true, :class => "button-small dash-button #{"active" if current_page?(dashboard_merge_requests_path)}", :id => "merge_requests_slide"
+  = link_to "Activities", dashboard_path, :remote => true, :class => "dash-button #{"active" if current_page?(dashboard_path) || current_page?(root_path) }", :id => "activities_slide"
+  = link_to "Issues", dashboard_issues_path, :remote => true, :class => "dash-button #{"active" if current_page?(dashboard_issues_path)}", :id => "issues_slide"
+  = link_to "Merge Requests", dashboard_merge_requests_path, :remote => true, :class => "dash-button #{"active" if current_page?(dashboard_merge_requests_path)}", :id => "merge_requests_slide"
   = image_tag "ajax-loader-facebook.gif", :class => "dashboard-loader"
 :javascript 
 #feeds_content_holder
-  .project-box.project-updates.ui-box.ui-box-small.ui-box-big
-    .data
-      - @merge_requests.each do |update|
-        %a.project-update{:href => project_merge_request_path(update.project, update)}
-          = image_tag gravatar_icon(update.author_email), :class => "left", :width => 40
-          %span.update-title
-            - state = update.closed ? 'closed' : 'open'
-            %span.tag{ :class => state } #{state.capitalize}
-            = truncate update.title, :length => 70
-            .right= update.project.name
-          %span.update-author
-            %strong= update.author_name
-            authored
-            = time_ago_in_words(update.created_at)
-            ago
-          .right
-            %span.tag.commit= update.source_branch
-            &rarr;
-            %span.tag.commit= update.target_branch
+  - unless @merge_requests.empty?
+    .project-box.project-updates.ui-box.ui-box-small.ui-box-big
+      .data
+        - @merge_requests.each do |update|
+          %a.project-update{:href => project_merge_request_path(update.project, update)}
+            = image_tag gravatar_icon(update.author_email), :class => "left", :width => 40
+            %span.update-title
+              = truncate update.title, :length => 35
+              .right= truncate update.project.name
+            %span.update-author
+              %strong= update.author_name
+              authored
+              = time_ago_in_words(update.created_at)
+              ago
+            .right
+              %span.tag.commit= update.source_branch
+              &rarr;
+              %span.tag.commit= update.target_branch
+  - else 
+    %h2
+      No authored or assigned
+      %span.tag.open open
+      merge requests
1	#feeds_content_holder	1	#feeds_content_holder
2	- .project-box.project-updates.ui-box.ui-box-small.ui-box-big
3	- .data
4	- - @issues.each do \|update\|
5	- %a.project-update{:href => dashboard_feed_path(update.project, update)}
6	- = image_tag gravatar_icon(update.author_email), :class => "left", :width => 40
7	- %span.update-title
8	- = truncate update.title, :length => 50
9	- .right= update.project.name
10	- %span.update-author
11	- %strong= update.author_name
12	- authored
13	- = time_ago_in_words(update.created_at)
14	- ago
15	- .right
16	- - if update.critical
17	- %span.tag.high critical
18	- - if update.today?
19	- %span.tag.today today	2	+ - unless @issues.empty?
		3	+ .project-box.project-updates.ui-box.ui-box-small.ui-box-big
		4	+ .data
		5	+ - @issues.each do \|update\|
		6	+ %a.project-update{:href => dashboard_feed_path(update.project, update)}
		7	+ %strong.issue-number= "##{update.id}"
		8	+ %span.update-title
		9	+ = truncate update.title, :length => 35
		10	+ .right= truncate update.project.name
		11	+ %span.update-author
		12	+ %strong= update.author_name
		13	+ authored
		14	+ = time_ago_in_words(update.created_at)
		15	+ ago
		16	+ .right
		17	+ - if update.critical
		18	+ %span.tag.high critical
		19	+ - if update.today?
		20	+ %span.tag.today today
20		21
		22	+ - else
		23	+ %h2
		24	+ No assigned
		25	+ %span.tag.open open
		26	+ issues