##// END OF EJS Templates
pull requests: fix issue when close button was inactive even if status...
pull requests: fix issue when close button was inactive even if status was approved or rejected already.

File last commit:

r4055:9314f264 default
r4055:9314f264 default
Show More
changeset_file_comment.html
251 lines | 10.2 KiB | text/html | HtmlLexer
/ rhodecode / templates / changeset / changeset_file_comment.html
White-space cleanup
r1888 ## -*- coding: utf-8 -*-
## usage:
## <%namespace name="comment" file="/changeset/changeset_file_comment.html"/>
## ${comment.comment_block(co)}
##
<%def name="comment_block(co)">
#415: Adding comment to changeset causes reload...
r2187 <div class="comment" id="comment-${co.comment_id}" line="${co.line_no}">
White-space cleanup
r1888 <div class="comment-wrapp">
Mads Kiilerich
html: don't use tabs
r3197 <div class="meta">
Show changes of status inside comments...
r2286 <div style="float:left"> <img src="${h.gravatar_url(co.author.email, 20)}" /> </div>
Mads Kiilerich
html: don't use tabs
r3197 <div class="user">
${co.author.username}
</div>
<div class="date">
Implemented preview for comments
r3695 ${h.age(co.modified_at)}
Mads Kiilerich
html: don't use tabs
r3197 </div>
Show pull request link also on regular comments that are now attached to separate changesets
r3190
Implemented preview for comments
r3695 <div style="float:left;padding:4px 0px 0px 5px">
Show pull request link also on regular comments that are now attached to separate changesets
r3190 <span class="">
%if co.pull_request:
<a href="${h.url('pullrequest_show',repo_name=co.pull_request.other_repo.repo_name,pull_request_id=co.pull_request.pull_request_id)}">
%if co.status_change:
Mads Kiilerich
pull requests: make the difference between changeset comments with status and PR comments with votes more obvious
r4051 ${_('Vote on pull request #%s') % co.pull_request.pull_request_id}:
Show pull request link also on regular comments that are now attached to separate changesets
r3190 %else:
Mads Kiilerich
comments: "Comment on pull request" feels more natural
r3608 ${_('Comment on pull request #%s') % co.pull_request.pull_request_id}
Show pull request link also on regular comments that are now attached to separate changesets
r3190 %endif
whitespace cleanup
r3216 </a>
Mads Kiilerich
comments: clarify in ui whether comments are for changeset or PR
r4049 %else:
%if co.status_change:
${_('Status change on changeset')}:
%else:
${_('Comment on changeset')}
%endif
Show pull request link also on regular comments that are now attached to separate changesets
r3190 %endif
</span>
</div>
Mads Kiilerich
comments: clarify in ui whether comments are for changeset or PR
r4049
%if co.status_change:
<div style="float:left" class="changeset-status-container">
<div style="float:left;padding:10px 2px 0px 2px"></div>
<div title="${_('Changeset status')}" class="changeset-status-lbl"> ${co.status_change[0].status_lbl}</div>
<div class="changeset-status-ico"><img src="${h.url(str('/images/icons/flag_status_%s.png' % co.status_change[0].status))}" /></div>
</div>
%endif
Implemented preview for comments
r3695 <a class="permalink" href="#comment-${co.comment_id}">&para;</a>
White-space cleanup
r1888 %if h.HasPermissionAny('hg.admin', 'repository.admin')() or co.author.user_id == c.rhodecode_user.user_id:
Implemented preview for comments
r3695 <div onClick="deleteComment(${co.comment_id})" class="buttons delete-comment ui-btn small">${_('Delete')}</div>
White-space cleanup
r1888 %endif
Mads Kiilerich
html: don't use tabs
r3197 </div>
<div class="text">
${h.rst_w_mentions(co.text)|n}
</div>
White-space cleanup
r1888 </div>
</div>
</%def>
Enabled inline comments in pull-requests
r2489 <%def name="comment_inline_form()">
White-space cleanup
r1888 <div id='comment-inline-form-template' style="display:none">
Autocomplete fixes...
r2369 <div class="comment-inline-form ac">
White-space cleanup
r1888 %if c.rhodecode_user.username != 'default':
#415: Adding comment to changeset causes reload...
r2187 <div class="overlay"><div class="overlay-text">${_('Submitting...')}</div></div>
Enabled inline comments in pull-requests
r2489 ${h.form('#', class_='inline-form')}
Implemented preview for comments
r3695 <div id="edit-container_{1}" class="clearfix">
Vincent Duvert
Fixed i18n of the second comment help block.
r2307 <div class="comment-help">${_('Commenting on line {1}.')}
Added h.tooltip to all places that tooltip_title is used....
r2427 ${(_('Comments parsed using %s syntax with %s support.') % (
('<a href="%s">RST</a>' % h.url('rst_help')),
Mads Kiilerich
html: don't use tabs
r3197 ('<span style="color:#003367" class="tooltip" title="%s">@mention</span>' % _('Use @username inside this text to send notification to this RhodeCode user'))
Added h.tooltip to all places that tooltip_title is used....
r2427 )
)|n
}
Mads Kiilerich
Fix inconsistent casing
r3753 <div id="preview-btn_{1}" class="preview-btn ui-btn small">${_('Preview')}</div>
Added mentions autocomplete into main comments form...
r2368 </div>
<div class="mentions-container" id="mentions_container_{1}"></div>
Implemented preview for comments
r3695 <textarea id="text_{1}" name="text" class="comment-block-ta yui-ac-input"></textarea>
</div>
<div id="preview-container_{1}" class="clearfix" style="display:none">
<div class="comment-help">
Mads Kiilerich
Fix inconsistent casing
r3753 ${_('Comment preview')}
<div id="edit-btn_{1}" class="edit-btn ui-btn small">${_('Edit')}</div>
Implemented preview for comments
r3695 </div>
<div id="preview-box_{1}" class="preview-box"></div>
White-space cleanup
r1888 </div>
<div class="comment-button">
<input type="hidden" name="f_path" value="{0}">
<input type="hidden" name="line" value="{1}">
#415: Adding comment to changeset causes reload...
r2187 ${h.submit('save', _('Comment'), class_='ui-btn save-inline-form')}
Mads Kiilerich
changeset comment: new uncommitted comments can be cancelled - not just hidden
r3254 ${h.reset('hide-inline-form', _('Cancel'), class_='ui-btn hide-inline-form')}
White-space cleanup
r1888 </div>
${h.end_form()}
%else:
${h.form('')}
<div class="clearfix">
<div class="comment-help">
Vincent Duvert
Improved i18n of the changeset inline comment section.
r2302 ${_('You need to be logged in to comment.')} <a href="${h.url('login_home',came_from=h.url.current())}">${_('Login now')}</a>
White-space cleanup
r1888 </div>
</div>
<div class="comment-button">
${h.reset('hide-inline-form', _('Hide'), class_='ui-btn hide-inline-form')}
</div>
${h.end_form()}
%endif
</div>
</div>
</%def>
Refactoring of changeset_file_comments for more generic usage. In both It enables sharing code between changeset, and pull requests discussions
r2439 ## generates inlines taken from c.comments var
<%def name="inlines()">
Vincent Duvert
Improved i18n for the comment count (use of ngettext for pluralisation).
r2310 <div class="comments-number">${ungettext("%d comment", "%d comments", len(c.comments)) % len(c.comments)} ${ungettext("(%d inline)", "(%d inline)", c.inline_cnt) % c.inline_cnt}</div>
White-space cleanup
r1888 %for path, lines in c.inline_comments:
% for line,comments in lines.iteritems():
#415: Adding comment to changeset causes reload...
r2187 <div style="display:none" class="inline-comment-placeholder" path="${path}" target_id="${h.safeid(h.safe_unicode(path))}">
White-space cleanup
r1888 %for co in comments:
${comment_block(co)}
%endfor
</div>
%endfor
%endfor
white space cleanup
r2188
#415: Adding comment to changeset causes reload...
r2187 </%def>
White-space cleanup
r1888
Added option to close pull requests, in future that will be close & merge
r2608 ## generate inline comments and the main ones
added a flag for generating pull-request comments, and force display on pull-requests
r3229 <%def name="generate_comments(include_pr=False)">
#415: Adding comment to changeset causes reload...
r2187 <div class="comments">
<div id="inline-comments-container">
Refactoring of changeset_file_comments for more generic usage. In both It enables sharing code between changeset, and pull requests discussions
r2439 ## generate inlines for this changeset
${inlines()}
#415: Adding comment to changeset causes reload...
r2187 </div>
white space cleanup
r2188
White-space cleanup
r1888 %for co in c.comments:
fixed main comments, prevent from sending inline comments if text is empty
r2189 <div id="comment-tr-${co.comment_id}">
remove comments associated to pull request from single changesets, it's too confusing....
r3200 ## only render comments that are not from pull request, or from
## pull request and a status change
added a flag for generating pull-request comments, and force display on pull-requests
r3229 %if not co.pull_request or (co.pull_request and co.status_change) or include_pr:
fixed main comments, prevent from sending inline comments if text is empty
r2189 ${comment_block(co)}
remove comments associated to pull request from single changesets, it's too confusing....
r3200 %endif
fixed main comments, prevent from sending inline comments if text is empty
r2189 </div>
White-space cleanup
r1888 %endfor
white space cleanup
r2673 </div>
Added option to close pull requests, in future that will be close & merge
r2608 </%def>
## MAIN COMMENT FORM
Mads Kiilerich
pull requests: make the difference between changeset comments with status and PR comments with votes more obvious
r4051 <%def name="comments(post_url, cur_status, is_pr=False, change_status=True)">
Added option to close pull requests, in future that will be close & merge
r2608
<div class="comments">
White-space cleanup
r1888 %if c.rhodecode_user.username != 'default':
Added mentions autocomplete into main comments form...
r2368 <div class="comment-form ac">
Refactoring of changeset_file_comments for more generic usage. In both It enables sharing code between changeset, and pull requests discussions
r2439 ${h.form(post_url)}
Implemented preview for comments
r3695 <div id="edit-container" class="clearfix">
White-space cleanup
r1888 <div class="comment-help">
Vincent Duvert
Fixed i18n of the second comment help block.
r2307 ${(_('Comments parsed using %s syntax with %s support.') % (('<a href="%s">RST</a>' % h.url('rst_help')),
Mads Kiilerich
html: don't use tabs
r3197 '<span style="color:#003367" class="tooltip" title="%s">@mention</span>' %
_('Use @username inside this text to send notification to this RhodeCode user')))|n}
Implemented #670 Implementation of Roles in Pull Request...
r3104 %if change_status:
Mads Kiilerich
pull requests: make the difference between changeset comments with status and PR comments with votes more obvious
r4051 | <a id="show_changeset_link" onClick="change_status_show();">
%if is_pr:
${_('Vote for pull request status')}
%else:
${_('Change changeset status')}
%endif
</a>
Mads Kiilerich
pullrequest: use simple link for 'Change style'...
r3593 <input id="show_changeset_status_box" type="checkbox" name="change_changeset_status" style="display: none;" />
Implemented #670 Implementation of Roles in Pull Request...
r3104 %endif
Mads Kiilerich
Fix inconsistent casing
r3753 <div id="preview-btn" class="preview-btn ui-btn small">${_('Preview')}</div>
White-space cleanup
r1888 </div>
Implemented #670 Implementation of Roles in Pull Request...
r3104 %if change_status:
Implemented initial code-review status of changesets
r2217 <div id="status_block_container" class="status-block" style="display:none">
%for status,lbl in c.changeset_statuses:
<div class="">
white space cleanup
r3149 <img src="${h.url('/images/icons/flag_status_%s.png' % status)}" /> <input ${'checked="checked"' if status == cur_status else ''}" type="radio" class="status_change_radio" name="changeset_status" id="${status}" value="${status}">
fixed changeset status labels, they now select radio buttons
r3102 <label for="${status}">${lbl}</label>
white space cleanup
r2478 </div>
Implemented initial code-review status of changesets
r2217 %endfor
Mads Kiilerich
pull requests: make the difference between changeset comments with status and PR comments with votes more obvious
r4051 %if is_pr and change_status:
Mads Kiilerich
pull request comments: implement 'close' as a checkbox that always is visible but sometimes disabled
r4048 <input id="save_close" type="checkbox" name="save_close" disabled="${'1' if cur_status in ['not_reviewed','under_review'] else ''}">
<label id="save_close_label" for="save_close" class="${'disabled' if cur_status in ['not_reviewed','under_review'] else ''}">${_("Close (when approved or rejected)")}</label>
%endif
white space cleanup
r2478 </div>
Implemented #670 Implementation of Roles in Pull Request...
r3104 %endif
merge with beta
r2391 <div class="mentions-container" id="mentions_container"></div>
Implemented preview for comments
r3695 ${h.textarea('text', class_="comment-block-ta")}
White-space cleanup
r1888 </div>
Implemented preview for comments
r3695
<div id="preview-container" class="clearfix" style="display:none">
<div class="comment-help">
Mads Kiilerich
Fix inconsistent casing
r3753 ${_('Comment preview')}
<div id="edit-btn" class="edit-btn ui-btn small">${_('Edit')}</div>
Implemented preview for comments
r3695 </div>
<div id="preview-box" class="preview-box"></div>
</div>
White-space cleanup
r1888 <div class="comment-button">
Added option to close pull requests, in future that will be close & merge
r2608 ${h.submit('save', _('Comment'), class_="ui-btn large")}
White-space cleanup
r1888 </div>
${h.end_form()}
</div>
%endif
</div>
Added mentions autocomplete into main comments form...
r2368 <script>
Mads Kiilerich
pullrequest: use simple link for 'Change style'...
r3593 var change_status_show = function(){
var show = ! YUD.get('show_changeset_status_box').checked;
YUD.get('show_changeset_status_box').checked = show;
YUD.setStyle('status_block_container', 'display', show?'':'none');
};
Added mentions autocomplete into main comments form...
r2368 YUE.onDOMReady(function () {
Autocomplete fixes...
r2369 MentionsAutoComplete('text', 'mentions_container', _USERS_AC_DATA, _GROUPS_AC_DATA);
white space cleanup
r2478
pull requests: fix issue when close button was inactive even if status...
r4055 var set_close_button_state = function(cur_val){
if (cur_val == 'approved' || cur_val == 'rejected') {
Mads Kiilerich
pull request comments: implement 'close' as a checkbox that always is visible but sometimes disabled
r4048 YUD.removeClass('save_close_label', 'disabled');
YUD.get('save_close').disabled = false;
Mads Kiilerich
html: don't use tabs
r3197 }else{
Mads Kiilerich
pull request comments: implement 'close' as a checkbox that always is visible but sometimes disabled
r4048 YUD.addClass('save_close_label', 'disabled');
YUD.get('save_close').disabled = true;
Mads Kiilerich
html: don't use tabs
r3197 }
pull requests: fix issue when close button was inactive even if status...
r4055 }
YUE.on(YUQ('.status_change_radio'), 'change',function(e){
var val = e.currentTarget.value;
set_close_button_state(val)
implements #677: Don't allow to close pull requests when they are under-review state
r3103 })
pull requests: fix issue when close button was inactive even if status...
r4055 // if currect checked status is approved or rejected we should unlock the close button
var cur_val = YUQ('.status_change_radio[checked]')[0].value;
set_close_button_state(cur_val)
Implemented preview for comments
r3695 YUE.on('preview-btn', 'click', function(e){
var _text = YUD.get('text').value;
if(!_text){
return
}
var post_data = {'text': _text};
YUD.addClass('preview-box', 'unloaded');
YUD.get('preview-box').innerHTML = _TM['Loading ...'];
YUD.setStyle('edit-container', 'display', 'none');
YUD.setStyle('preview-container', 'display', '');
var url = pyroutes.url('changeset_comment_preview', {'repo_name': '${c.repo_name}'});
ajaxPOST(url,post_data,function(o){
YUD.get('preview-box').innerHTML = o.responseText;
YUD.removeClass('preview-box', 'unloaded');
})
})
YUE.on('edit-btn', 'click', function(e){
YUD.setStyle('edit-container', 'display', '');
YUD.setStyle('preview-container', 'display', 'none');
})
white space cleanup
r2478
Added mentions autocomplete into main comments form...
r2368 });
</script>
White-space cleanup
r1888 </%def>