<div dir="ltr">I %100 agree with Brad.<div><br></div><div>-berk</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Apr 16, 2015 at 10:47 AM, Brad King <span dir="ltr"><<a href="mailto:brad.king@kitware.com" target="_blank">brad.king@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 04/15/2015 10:20 PM, David Cole via vtk-developers wrote:<br>
> Buttons, labelled "Approve" and "Reject"<br>
><br>
> Not to be overly pedantic, but buttons should indicate the action you<br>
> are doing by clicking them, not the action you did once they have been<br>
> clicked. (i.e. -- make it an action verb, not a past tense "did that"<br>
> thing...)<br>
><br>
> +2 for buttons!<br>
<br>
</span>Adding buttons will require modification to GitLab, which we've<br>
ruled must be kept to a minimum.  We already have a few mods in<br>
use to get the event web hooks we need.  Only after we've gotten<br>
those upstreamed successfully should we consider further mods<br>
locally.  We don't want to end up forked like we did with Gerrit.<br>
<span class="HOEnZb"><font color="#888888"><br>
-Brad<br>
<br>
</font></span></blockquote></div><br></div>