reviewers: fixed UI issues on reviewers for big screens.

rhodecode/templates/ejs_templates/templates.html

@@ -65,6 +65,7 b" var data_hovercard_url = pyroutes.url('h"
 
 <tr id="reviewer_<%= member.user_id %>" class="reviewer_entry" tooltip="Review Group" data-reviewer-user-id="<%= member.user_id %>">
 
+    <% if (!create) { %>
     <td style="width: 20px">
         <div class="tooltip presence-state" style="display: none; position: absolute; left: 2px" title="This users is currently at this page">
           <i class="icon-eye" style="color: #0ac878"></i>
@@ -79,6 +80,24 b" var data_hovercard_url = pyroutes.url('h"
         </div>
         <% } %>
     </td>
+    <% } %>
+
+    <% if (mandatory) { %>
+    <td style="text-align: right;width: 10px;">
+        <div class="reviewer_member_mandatory tooltip" title="Mandatory reviewer">
+            <i class="icon-lock"></i>
+        </div>
+    </td>
+
+    <% } else { %>
+    <td style="text-align: right;width: 10px;">
+        <% if (allowed_to_update) { %>
+          <div class="<%=role %>_member_remove" onclick="reviewersController.removeMember(<%= member.user_id %>, true)" style="visibility: <%= edit_visibility %>;">
+              <i class="icon-remove" style="color: #e85e4d;"></i>
+          </div>
+        <% } %>
+    </td>
+    <% } %>
 
     <td>
       <div id="reviewer_<%= member.user_id %>_name" class="reviewer_name">
@@ -94,9 +113,6 b" var data_hovercard_url = pyroutes.url('h"
             })
         %>
       </div>
-    </td>
-
-    <td style="width: 10px">
     <% if (reviewGroup !== null) { %>
         <span class="tooltip" title="Member of review group from rule: `<%= member.user_group.name %>`" style="color: <%= reviewGroupColor %>">
         <%- reviewGroup %>
@@ -104,23 +120,6 b" var data_hovercard_url = pyroutes.url('h"
     <% } %>
     </td>
 
-    <% if (mandatory) { %>
-    <td style="text-align: right;width: 10px;">
-        <div class="reviewer_member_mandatory tooltip" title="Mandatory reviewer">
-            <i class="icon-lock"></i>
-        </div>
-    </td>
-
-    <% } else { %>
-    <td style="text-align: right;width: 10px;">
-        <% if (allowed_to_update) { %>
-          <div class="<%=role %>_member_remove" onclick="reviewersController.removeMember(<%= member.user_id %>, true)" style="visibility: <%= edit_visibility %>;">
-              <i class="icon-remove"></i>
-          </div>
-        <% } %>
-    </td>
-    <% } %>
-
 </tr>
 
 <tr id="reviewer_<%= member.user_id %>_rules">