Changeset - 1f2ba96de73e
Parent rev.
Child rev.
[Not reviewed]
beta
0 3 0
Marcin Kuzminski - 14 years ago 2011-11-14 21:10:02
marcin@python-works.com
#71 code review
- inline comments are now positioned inlines
3 files changed with 92 insertions and 12 deletions:
rhodecode/public/css/style.css
54
2
rhodecode/public/js/rhodecode.js
10
5
rhodecode/templates/changeset/changeset.html
28
5
0 comments (0 inline, 0 general)
rhodecode/public/css/style.css
➞
Show inline comments
 
@@ -3232,13 +3232,13 @@ div.rst-block  pre {
 
    -webkit-border-radius: 3px;
 
    -moz-border-radius: 3px;
 
    border-radius: 3px;
 
}
 
 
 
 
/** comment main **/
 
.comments {
 
    padding:10px 20px;
 
}
 
 
.comments .comment {
 
    border: 1px solid #ddd;
 
@@ -3274,12 +3274,13 @@ div.rst-block  pre {
 
.comments .comments-number{
 
	padding:0px 0px 10px 0px;
 
	font-weight: bold;
 
	color: #666;
 
	font-size: 16px;
 
}
 
 
/** comment form **/
 
 
.comment-form .clearfix{
 
	background: #EEE;
 
    -webkit-border-radius: 4px;
 
    -moz-border-radius: 4px;
 
@@ -3391,7 +3392,58 @@ form.comment-inline-form {
 
    color: #666666;
 
    font-size: 10px;
 
}
 
 
.comment-inline-form .comment-button{
 
    padding-top:5px;
 
}
 
\ No newline at end of file
 
}
 
 
/** comment inline **/
 
.inline-comments {
 
    padding:10px 20px;
 
}
 
 
.inline-comments div.rst-block  {
 
	clear:both;
 
	overflow:hidden;
 
	margin:0;
 
	padding:0 20px 0px;
 
}
 
.inline-comments .comment {
 
    border: 1px solid #ddd;
 
    -webkit-border-radius: 4px;
 
    -moz-border-radius: 4px;
 
    border-radius: 4px;
 
    margin-left: 5px;
 
    margin-right: 5px;
 
    margin-bottom: 3px;
 
}
 
 
.inline-comments .comment .meta {
 
    background: #f8f8f8;
 
    padding: 6px;
 
    border-bottom: 1px solid #ddd;
 
}
 
 
.inline-comments .comment .meta img {
 
    vertical-align: middle;
 
}
 
 
.inline-comments .comment .meta .user {
 
    font-weight: bold;
 
}
 
 
.inline-comments .comment .meta .date {
 
    float: right;
 
}
 
 
.inline-comments .comment .text {
 
    padding: 8px 6px 6px 14px;
 
    background-color: #FAFAFA;
 
}
 
 
.inline-comments .comments-number{
 
    padding:0px 0px 10px 0px;
 
    font-weight: bold;
 
    color: #666;
 
    font-size: 16px;
 
}
rhodecode/public/js/rhodecode.js
➞
Show inline comments
 
@@ -301,20 +301,25 @@ var ajaxPOST = function(url,postData,suc
 

			




	
	
	
		
 
/** comments **/

			




	
	
	
		
 
var removeInlineForm = function(form) {

			




	
	
	
		
 
	form.parentNode.removeChild(form);

			




	
	
	
		
 
}

			




	
	
	
		
 

			




	
	
	
		
 
var createInlineForm = function(parent_tr, f_path, line) {

			




	
	
	
		
 
var tableTr = function(cls,body){

			




	
	
	
		
 
	var form = document.createElement('tr');

			




	
	
	
		
 
	YUD.addClass(form, 'comment-form-inline');

			




	
	
	
		
 
	YUD.addClass(form, cls);

			




	
	
	
		
 
	form.innerHTML = '<td class="lineno new"></td>'+

			




	
	
	
		
 
    				 '<td class="lineno old"></td>'+ 

			




	
	
	
		
 
                     '<td>{0}</td>'.format(body);

			




	
	
	
		
 
	return form;

			




	
	
	
		
 
}

			




	
	
	
		
 

			




	
	
	
		
 
var createInlineForm = function(parent_tr, f_path, line) {

			




	
	
	
		
 
	var tmpl = YUD.get('comment-inline-form-template').innerHTML;

			




	
	
	
		
 
	tmpl = tmpl.format(f_path, line);

			




	
	
	
		
 
	form.innerHTML = '<td class="lineno new"></td>'+

			




	
	
	
		
 
		             '<td class="lineno old"></td>'+ 

			




	
	
	
		
 
			         '<td>{0}</td>'.format(tmpl);

			




	
	
	
		
 
	var form = tableTr('comment-form-inline',tmpl)

			




	
	
	
		
 

			




	
	
	
		
 
	// create event for hide button

			




	
	
	
		
 
	form = new YAHOO.util.Element(form);

			




	
	
	
		
 
	var form_hide_button = new YAHOO.util.Element(form.getElementsByClassName('hide-inline-form')[0]);

			




	
	
	
		
 
	form_hide_button.on('click', function(e) {

			




	
	
	
		
 
		var newtr = e.currentTarget.parentNode.parentNode.parentNode.parentNode.parentNode;

			




        

    


    
    

    

        

                

                    rhodecode/templates/changeset/changeset.html
                

                

                  
                      
                        

                      ➞

                      
                        
                  

                  
                      
                  
                      
                  
                      
                  
                      
                  
                  
                

                

                    Show inline comments
                    
                

        

        

            



	
	
		
 
@@ -15,12 +15,18 @@

			




	
	
	
		
 
</%def>

			




	
	
	
		
 

			




	
	
	
		
 
<%def name="page_nav()">

			




	
	
	
		
 
    ${self.menu('changelog')}     

			




	
	
	
		
 
</%def>

			




	
	
	
		
 

			




	
	
	
		
 
<%def name="fid(raw_id,path)" filter="strip">

			




	
	
	
		
 
  <% 

			




	
	
	
		
 
    return 'C-%s-%s' % (h.short_id(raw_id),h.safeid(h.safe_unicode(path))) 

			




	
	
	
		
 
  %>

			




	
	
	
		
 
</%def>

			




	
	
	
		
 

			




	
	
	
		
 
<%def name="main()">

			




	
	
	
		
 
<div class="box">

			




	
	
	
		
 
    <!-- box / title -->

			




	
	
	
		
 
    <div class="title">

			




	
	
	
		
 
        ${self.breadcrumbs()}

			




	
	
	
		
 
    </div>

			




	
	
		
 
@@ -89,13 +95,13 @@

			




	
	
	
		
 
	        <span style="font-size:1.1em;font-weight: bold">

			




	
	
	
		
 
	        ${_('%s files affected with %s additions and %s deletions.') % (len(c.changeset.affected_files),c.lines_added,c.lines_deleted)}

			




	
	
	
		
 
	        </span>

			




	
	
	
		
 
	        <div class="cs_files">

			




	
	
	
		
 
	                %for change,filenode,diff,cs1,cs2,stat in c.changes:

			




	
	
	
		
 
	                    <div class="cs_${change}">

			




	
	
	
		
 
		                    <div class="node">${h.link_to(h.safe_unicode(filenode.path),h.url.current(anchor='C-%s-%s' % (h.short_id(filenode.changeset.raw_id),h.safeid(h.safe_unicode(filenode.path)))))}</div>

			




	
	
	
		
 
		                    <div class="node">${h.link_to(h.safe_unicode(filenode.path),h.url.current(anchor=self.fid(filenode.changeset.raw_id,filenode.path)))}</div>

			




	
	
	
		
 
		                    <div class="changes">${h.fancy_file_stats(stat)}</div>

			




	
	
	
		
 
	                    </div>

			




	
	
	
		
 
	                %endfor

			




	
	
	
		
 
	                % if c.cut_off:

			




	
	
	
		
 
	                  ${_('Changeset was too big and was cut off...')}

			




	
	
	
		
 
	                % endif

			




	
	
		
 
@@ -105,13 +111,13 @@

			




	
	
	
		
 
    </div>

			




	
	
	
		
 
    	

			




	
	
	
		
 
	%for change,filenode,diff,cs1,cs2,stat in c.changes:

			




	
	
	
		
 
		%if change !='removed':

			




	
	
	
		
 
		<div style="clear:both;height:10px"></div>

			




	
	
	
		
 
		<div class="diffblock  margined">

			




	
	
	
		
 
			<div id="${'C-%s-%s' % (h.short_id(filenode.changeset.raw_id),h.safeid(h.safe_unicode(filenode.path)))}" class="code-header">

			




	
	
	
		
 
			<div id="${self.fid(filenode.changeset.raw_id,filenode.path)}" class="code-header">

			




	
	
	
		
 
				<div class="changeset_header">

			




	
	
	
		
 
					<span class="changeset_file">

			




	
	
	
		
 
						${h.link_to_if(change!='removed',h.safe_unicode(filenode.path),h.url('files_home',repo_name=c.repo_name,

			




	
	
	
		
 
						revision=filenode.changeset.raw_id,f_path=h.safe_unicode(filenode.path)))}

			




	
	
	
		
 
					</span>

			




	
	
	
		
 
					&raquo; <span>${h.link_to(_('diff'),

			




	
	
		
 
@@ -139,15 +145,15 @@

			




	
	
	
		
 
    ${comment.comment_inline_form()}

			




	
	
	
		
 
    

			




	
	
	
		
 
    <div class="comments">

			




	
	
	
		
 
        <div class="comments-number">${len(c.comments)} comment(s) (${c.inline_cnt} ${_('inline')})</div>

			




	
	
	
		
 
        

			




	
	
	
		
 
        %for path, lines in c.inline_comments:

			




	
	
	
		
 
            <div class="inline-comment-placeholder" path="${path} ">

			




	
	
	
		
 
            <div style="display:none" class="inline-comment-placeholder" path="${path}" target_id="${self.fid(c.changeset.raw_id,path)}">

			




	
	
	
		
 
            % for line,comments in lines.iteritems():

			




	
	
	
		
 
                <div class="inline-comment-placeholder-line" line="${line}"> 

			




	
	
	
		
 
                <div class="inline-comment-placeholder-line" line="${line}" target_id="${h.safeid(h.safe_unicode(path))}"> 

			




	
	
	
		
 
                %for co in comments:

			




	
	
	
		
 
                    ${comment.comment_block(co)}

			




	
	
	
		
 
                %endfor

			




	
	
	
		
 
                </div>

			




	
	
	
		
 
            %endfor

			




	
	
	
		
 
            </div>

			




	
	
		
 
@@ -205,12 +211,29 @@

			




	
	
	
		
 
        	  YUD.addClass(tr,'form-open');

			




	
	
	
		
 
        	  var node = tr.parentNode.parentNode.parentNode.getElementsByClassName('full_f_path')[0];

			




	
	
	
		
 
        	  var f_path = YUD.getAttribute(node,'path');

			




	
	
	
		
 
        	  var lineno = getLineNo(tr);

			




	
	
	
		
 
        	  var form = createInlineForm(tr, f_path, lineno);

			




	
	
	
		
 
        	  YUD.insertAfter(form,tr);

			




	
	
	
		
 
          })

			




	
	
	
		
 
          });

			




	
	
	
		
 
          

			




	
	
	
		
 
          // inject comments into they proper positions

			




	
	
	
		
 
          var file_comments = YUQ('.inline-comment-placeholder');

			




	
	
	
		
 
          

			




	
	
	
		
 
          for (f in file_comments){

			




	
	
	
		
 
        	  var box = file_comments[f];

			




	
	
	
		
 
        	  var inlines = box.children;

			




	
	
	
		
 
        	  

			




	
	
	
		
 
        	  for(var i=0; i<inlines.length; i++){

			




	
	
	
		
 
        		  var inline = inlines[i];

			




	
	
	
		
 
        		  var lineno = YUD.getAttribute(inlines[i],'line');

			




	
	
	
		
 
        		  var lineid = "a{0}_{1}".format(YUD.getAttribute(inline,'target_id'),lineno);

			




	
	
	
		
 
                  var target_line = YUD.get(lineid);

			




	
	
	
		
 
                  var comments = new YAHOO.util.Element(tableTr('inline-comments',inline.innerHTML))

			




	
	
	
		
 
                  YUD.insertAfter(comments,target_line.parentNode);        		  

			




	
	
	
		
 
        	  }

			




	
	
	
		
 
          }

			




	
	
	
		
 
      })

			




	
	
	
		
 
      

			




	
	
	
		
 
    </script> 

			




	
	
	
		
 
  </div>	

			




	
	
	
		
 
</%def>

			




        

    



    


    



    



      

      





    
    0 comments (0 inline, 0 general)
    





    


  

  







  


    




    








    
        
    
    
        This site is powered by
            Kallithea 0.7.0,
        which is
        © 2010–2025 by various authors & licensed under GPLv3.