[vtk-developers] Approving a gitlab topic
    Ben Boeckel 
    ben.boeckel at kitware.com
       
    Fri Apr 17 08:56:30 EDT 2015
    
    
  
On Wed, Apr 15, 2015 at 22:20:04 -0400, David Cole via vtk-developers wrote:
> Buttons, labelled "Approve" and "Reject"
> 
> Not to be overly pedantic, but buttons should indicate the action you
> are doing by clicking them, not the action you did once they have been
> clicked. (i.e. -- make it an action verb, not a past tense "did that"
> thing...)
> 
> +2 for buttons!
Here are some bookmarklets you can add to your browser add put in the
bookmark toolbar:
    Ack:
        javascript:(function(){jQuery("textarea.note_text:visible").val("+1")})()
    Review:
        javascript:(function(){jQuery("textarea.note_text:visible").val("+2")})()
    Tested:
        javascript:(function(){jQuery("textarea.note_text:visible").val("+3")})()
    Reject:
        javascript:(function(){jQuery("textarea.note_text:visible").val("-1")})()
    Buildbot:
        javascript:(function(){jQuery("textarea.note_text:visible").val("@buildbot test")})()
    Merge:
        javascript:(function(){jQuery("textarea.note_text:visible").val("\n\nDo: merge")})()
Note that these will *overwrite* any text currently in the comment box.
--Ben
    
    
More information about the vtk-developers
mailing list