# HG changeset patch
# User Marcin Kuzminski <marcin@rhodecode.com>
# Date 2018-01-18 18:04:48
# Node ID 590df01217bb4db68edf753c65525e31ff1d2282
# Parent  3775edd6faa28cee8a341bec8d827666419b9291

reviewers: colorify the groups

diff --git a/rhodecode/public/js/src/rhodecode/utils/colorgenerator.js b/rhodecode/public/js/src/rhodecode/utils/colorgenerator.js
--- a/rhodecode/public/js/src/rhodecode/utils/colorgenerator.js
+++ b/rhodecode/public/js/src/rhodecode/utils/colorgenerator.js
@@ -22,7 +22,8 @@
 *usage::
 *  var CG = new ColorGenerator();
 *  var col = CG.getColor(key); //returns array of RGB
-*  'rgb({0})'.format(col.join(',')
+*  CG.asRGB(col) === 'rgb({0})'.format(col.join(',')
+*
 *
 * @returns {ColorGenerator}
 */
@@ -47,6 +48,9 @@ ColorGenerator.prototype = {
       return this.cacheColorMap[key];
     }
   },
+  asRGB:function(color_tuple) {
+    return 'rgb({0})'.format(color_tuple.join(','));
+  },
   _hsvToRgb:function(h,s,v){
     if (s === 0.0)
         return [v, v, v];
diff --git a/rhodecode/templates/ejs_templates/templates.html b/rhodecode/templates/ejs_templates/templates.html
--- a/rhodecode/templates/ejs_templates/templates.html
+++ b/rhodecode/templates/ejs_templates/templates.html
@@ -26,11 +26,24 @@ if (show_disabled) {
 
 </script>
 
+<script>
+var CG = new ColorGenerator();
+</script>
 
 <script id="ejs_reviewMemberEntry" type="text/template" class="ejsTemplate">
 
 <li id="reviewer_<%= member.user_id %>" class="reviewer_entry">
-    <div class="reviewers_member">
+        <%
+
+            if (member.user_group && member.user_group.vote_rule) {
+                var groupStyle = 'border-left: 1px solid '+CG.asRGB(CG.getColor(member.user_group.vote_rule));
+            } else {
+                var groupStyle = 'border-left: 1px solid white';
+            }
+
+        %>
+
+    <div class="reviewers_member" style="<%= groupStyle%>" >
         <div class="reviewer_status tooltip" title="<%= review_status_label %>">
           <div class="flag_status <%= review_status %> pull-left reviewer_member_status"></div>
         </div>
@@ -41,15 +54,6 @@ if (show_disabled) {
             </div>
         <% } %>
 
-        <%if (member.user_group && member.user_group.vote_rule) {%>
-            <div style="float:right">
-              <%if (member.user_group.vote_rule == -1) {%>
-                Min votes: ALL
-              <%} else {%>
-                Min votes: <%= member.user_group.vote_rule %>
-              <%}%>
-            </div>
-        <%}%>
         <%-
           renderTemplate('gravatarWithUser', {
             'size': 16,
@@ -62,6 +66,18 @@ if (show_disabled) {
 
       <input type="hidden" name="__start__" value="reviewer:mapping">
 
+
+        <%if (member.user_group && member.user_group.vote_rule) {%>
+            <div class="reviewer_reason">
+
+              <%if (member.user_group.vote_rule == -1) {%>
+               - group votes required: ALL
+              <%} else {%>
+               - group votes required: <%= member.user_group.vote_rule %>
+              <%}%>
+            </div>
+        <%}%>
+
         <input type="hidden" name="__start__" value="reasons:sequence">
           <% for (var i = 0; i < reasons.length; i++) { %>
               <% var reason = reasons[i] %>
@@ -106,4 +122,4 @@ if (show_disabled) {
     registerTemplates();
 </script>
 
-</%text>
\ No newline at end of file
+</%text>