changeset 6278:af5eef651c77

style: don't use div.action_button for PR reviewers This is an odd button. The remaining action buttons are more regular. Based on work by Dominik Ruf.
author Mads Kiilerich <madski@unity3d.com>
date Thu, 20 Oct 2016 19:54:19 +0200
parents 4304595d246c
children 20830dfe3ed8
files kallithea/public/js/base.js kallithea/templates/pullrequests/pullrequest_show.html
diffstat 2 files changed, 6 insertions(+), 6 deletions(-) [+]
line wrap: on
line diff
--- a/kallithea/public/js/base.js	Mon Oct 24 21:45:09 2016 +0200
+++ b/kallithea/public/js/base.js	Thu Oct 20 19:54:19 2016 +0200
@@ -1313,17 +1313,17 @@
     var element = (
         '     <li id="reviewer_{2}">\n'+
         '       <div class="reviewers_member">\n'+
-        '           <div class="reviewer_status tooltip" title="not_reviewed">\n'+
+        '         <div class="reviewer_status tooltip" title="not_reviewed">\n'+
         '             <i class="icon-circle changeset-status-not_reviewed"></i>\n'+
-        '           </div>\n'+
+        '         </div>\n'+
         (gravatarelm ?
         '         <div class="reviewer_gravatar gravatar">{0}</div>\n' :
         '')+
         '         <div style="float:left;">{1}</div>\n'+
         '         <input type="hidden" value="{2}" name="review_members" />\n'+
-        '         <div class="reviewer_member_remove action_button" onclick="removeReviewMember({2})">\n'+
+        '         <a href="#" class="reviewer_member_remove" onclick="removeReviewMember({2})">\n'+
         '             <i class="icon-minus-circled"></i>\n'+
-        '         </div> (add not saved)\n'+
+        '         </a> (add not saved)\n'+
         '       </div>\n'+
         '     </li>\n'
         ).format(gravatarelm, displayname, id);
--- a/kallithea/templates/pullrequests/pullrequest_show.html	Mon Oct 24 21:45:09 2016 +0200
+++ b/kallithea/templates/pullrequests/pullrequest_show.html	Thu Oct 20 19:54:19 2016 +0200
@@ -247,9 +247,9 @@
                   </div>
                   <input type="hidden" value="${member.user_id}" name="review_members" />
                   %if editable:
-                  <div class="reviewer_member_remove action_button" onclick="removeReviewMember(${member.user_id})" title="${_('Remove reviewer')}">
+                  <a href="#" class="reviewer_member_remove" onclick="removeReviewMember(${member.user_id})" title="${_('Remove reviewer')}">
                       <i class="icon-minus-circled"></i>
-                  </div>
+                  </a>
                   %endif
                 </div>
               </li>