Add indication to merge request list item that MR cannot be merged automatically

app/assets/stylesheets/framework/lists.scss

@@ -124,6 +124,24 @@ ul.content-list {
         padding: 10px 14px;
       }
     }
+
+    ul.controls {
+      list-style: none;
+
+      li {
+        float: left;
+        padding-right: 10px;
+
+        .author_link {
+          display: inline-block;
+
+          .avatar-inline {
+            margin-left: 0;
+            margin-right: 0;
+          }
+        }
+      }
+    }
   }
 }
 

app/models/merge_request.rb

@@ -484,4 +484,8 @@ class MergeRequest < ActiveRecord::Base
       source_project.ci_commit(last_commit.id)
     end
   end
+
+  def broken?
+    self.commits.blank? || branch_missing? || cannot_be_merged?
+  end
 end

app/views/projects/issues/_issue.html.haml

@@ -6,23 +6,26 @@
   .issue-title
     %span.issue-title-text
       = link_to_gfm issue.title, issue_path(issue), class: "row_title"
-    .pull-right.light
+    %ul.controls.light
       - if issue.closed?
-        %span
+        %li
           CLOSED
+
       - if issue.assignee
+        %li
           = link_to_member(@project, issue.assignee, name: false, title: "Assigned to :name")
+
       - note_count = issue.notes.user.count
       - if note_count > 0
-         
+        %li
           = link_to issue_path(issue) + "#notes" do
             = icon('comments')
             = note_count
       - else
-         
+        %li
           = link_to issue_path(issue) + "#notes", class: "issue-no-comments" do
             = icon('comments')
-          = 0
+            = notes_count
 
   .issue-info
     #{issue.to_reference} ·

app/views/projects/merge_requests/_merge_request.html.haml

@@ -3,31 +3,40 @@
   .merge-request-title
     %span.merge-request-title-text
       = link_to_gfm merge_request.title, merge_request_path(merge_request), class: "row_title"
-    .pull-right.light
-      - if ci_commit
-        = render_ci_status(ci_commit)
+    %ul.controls.light
       - if merge_request.merged?
-        %span
+        %li
           = icon('check')
           MERGED
       - elsif merge_request.closed?
-        %span
+        %li
           = icon('ban')
           CLOSED
-      - note_count = merge_request.mr_and_commit_notes.user.count
+
+      - if ci_commit
+        %li
+          = render_ci_status(ci_commit)
+
+      - if merge_request.open? && merge_request.broken?
+        %li
+          = link_to merge_request_path(merge_request), class: "has_tooltip", title: "Cannot be merged automatically", data: {container: 'body'} do
+            = icon('exclamation-triangle')
+
       - if merge_request.assignee
-         
+        %li
           = link_to_member(merge_request.source_project, merge_request.assignee, name: false, title: "Assigned to :name")
+
+      - note_count = merge_request.mr_and_commit_notes.user.count
       - if note_count > 0
-         
+        %li
           = link_to merge_request_path(merge_request) + "#notes" do
             = icon('comments')
             = note_count
       - else
-         
+        %li
           = link_to merge_request_path(merge_request) + "#notes", class: "merge-request-no-comments" do
             = icon('comments')
-          = 0
+            = note_count
 
   .merge-request-info
     \##{merge_request.iid} ·