Changeset - 5258b66bf5d7
Parent rev.
Child rev.
[Not reviewed]
default
0 2 0
Mads Kiilerich - 9 years ago 2017-05-13 02:29:13
mads@kiilerich.com
pullrequests: fix "additional changes" js error - make sure all cs tables have a first column to align with (Issue #274)

A slight simplification of 16234f629cfb and fixing a JavaScript failure for
unauthenticated visits to PRs with pending available changesets.

nextFirstincell would be undefined becuase the "Current revision - no change"
line didn't have any elements in td, and nextFirstincell.offsetTop would thus
fail.

Fixed by adding a span around the text.

Also clarify that any element is fine - there is no need to check for
visibility.
2 files changed with 4 insertions and 3 deletions:
kallithea/public/js/graph.js
3
2
kallithea/templates/pullrequests/pullrequest_show.html
1
1
0 comments (0 inline, 0 general)
kallithea/public/js/graph.js
➞
Show inline comments
 
@@ -91,50 +91,51 @@ function BranchRenderer(canvas_id, conte
 

			




	
	
	
		
 
		var edge_pad = this.dot_radius + 2;

			




	
	
	
		
 
		var box_size = Math.min(18, (canvasWidth - edge_pad * 2) / lineCount);

			




	
	
	
		
 
		var base_x = canvasWidth - edge_pad;

			




	
	
	
		
 

			




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

			




	
	
	
		
 
			var row = document.getElementById(row_id_prefix+idx);

			




	
	
	
		
 
			if (row == null) {

			




	
	
	
		
 
				console.log("error: row "+row_id_prefix+idx+" not found");

			




	
	
	
		
 
				continue;

			




	
	
	
		
 
			}

			




	
	
	
		
 
			var next = document.getElementById(row_id_prefix+(idx+1));

			




	
	
	
		
 
			var extra = 0;

			




	
	
	
		
 

			




	
	
	
		
 
			cur = data[i];

			




	
	
	
		
 
			node = cur[0];

			




	
	
	
		
 
			in_l = cur[1];

			




	
	
	
		
 
			closing = cur[2];

			




	
	
	
		
 
			obsolete_node = cur[3];

			




	
	
	
		
 
			bumped_node = cur[4];

			




	
	
	
		
 
			divergent_node = cur[5];

			




	
	
	
		
 
			extinct_node = cur[6];

			




	
	
	
		
 
			unstable_node = cur[7];

			




	
	
	
		
 

			




	
	
	
		
 
			var firstincell = $(row).find('td>:visible')[0];

			




	
	
	
		
 
			var nextFirstincell = $(next).find('td>:visible')[0];

			




	
	
	
		
 
			// center dots on the first element in a td (not necessarily the first one, but there must be one)

			




	
	
	
		
 
			var firstincell = $(row).find('td>*')[0];

			




	
	
	
		
 
			var nextFirstincell = $(next).find('td>*')[0];

			




	
	
	
		
 
			var rowY = Math.floor(row.offsetTop + firstincell.offsetTop + firstincell.offsetHeight/2);

			




	
	
	
		
 
			var nextY = Math.floor((next == null) ? rowY + row.offsetHeight/2 : next.offsetTop + nextFirstincell.offsetTop + nextFirstincell.offsetHeight/2);

			




	
	
	
		
 

			




	
	
	
		
 
			for (var j in in_l) {

			




	
	
	
		
 
				line = in_l[j];

			




	
	
	
		
 
				start = line[0];

			




	
	
	
		
 
				end = line[1];

			




	
	
	
		
 
				color = line[2];

			




	
	
	
		
 
				obsolete_line = line[3];

			




	
	
	
		
 

			




	
	
	
		
 
				x = Math.floor(base_x - box_size * start);

			




	
	
	
		
 

			




	
	
	
		
 
				// figure out if this is a dead-end;

			




	
	
	
		
 
				// we want to fade away this line

			




	
	
	
		
 
				var dead_end = true;

			




	
	
	
		
 
				if (next != null) {

			




	
	
	
		
 
					nextdata = data[i+1];

			




	
	
	
		
 
					next_l = nextdata[1];

			




	
	
	
		
 
					for (var k=0; k < next_l.length; ++k) {

			




	
	
	
		
 
						if (next_l[k][0] == end) {

			




	
	
	
		
 
							dead_end = false;

			




	
	
	
		
 
							break;

			




	
	
	
		
 
						}

			




	
	
	
		
 
					}

			




        

    


    
    

    

        

                

                    kallithea/templates/pullrequests/pullrequest_show.html
                

                

                  
                      
                        

                      ➞

                      
                        
                  

                  
                      
                  
                      
                  
                      
                  
                      
                  
                  
                

                

                    Show inline comments
                    
                

        

        

            



	
	
		
 
@@ -144,49 +144,49 @@ ${self.repo_context_bar('showpullrequest

			




	
	
	
		
 
          <div class="pr-do-edit ac" style="display:none">

			




	
	
	
		
 
               ${h.text('owner', class_='form-control', value=c.pull_request.owner.username, placeholder=_('Username'))}

			




	
	
	
		
 
               <div id="owner_completion_container"></div>

			




	
	
	
		
 
          </div>

			




	
	
	
		
 
        </div>

			




	
	
	
		
 

			




	
	
	
		
 
        <div class="form-group">

			




	
	
	
		
 
          <label>${_('Next iteration')}:</label>

			




	
	
	
		
 
            <div>

			




	
	
	
		
 
              <div class="msg-div">${c.update_msg}</div>

			




	
	
	
		
 
              %if c.avail_revs:

			




	
	
	
		
 
              <div id="updaterevs" class="clearfix" style="max-height:200px; overflow-y:auto; overflow-x:hidden; margin-bottom: 10px; padding: 1px 0">

			




	
	
	
		
 
                <div style="height:0;width:40px">

			




	
	
	
		
 
                  <canvas id="avail_graph_canvas" style="width:0"></canvas>

			




	
	
	
		
 
                </div>

			




	
	
	
		
 
                <table class="table" id="updaterevs-table" style="padding-left:50px">

			




	
	
	
		
 
                  %for cnt, cs in enumerate(c.avail_cs):

			




	
	
	
		
 
                    <tr id="chg_available_${cnt+1}" class="${'mergerow' if len(cs.parents) > 1 and not (editable and cs.revision in c.avail_revs) else ''}">

			




	
	
	
		
 
                      %if c.cs_ranges and cs.revision == c.cs_ranges[-1].revision:

			




	
	
	
		
 
                        <td>

			




	
	
	
		
 
                          %if editable:

			




	
	
	
		
 
                            ${h.radio(name='updaterev', value='', checked=True)}

			




	
	
	
		
 
                          %endif

			




	
	
	
		
 
                        </td>

			




	
	
	
		
 
                        <td colspan="4">${_("Current revision - no change")}</td>

			




	
	
	
		
 
                        <td colspan="4"><span>${_("Current revision - no change")}</span></td>

			




	
	
	
		
 
                      %else:

			




	
	
	
		
 
                        <td>

			




	
	
	
		
 
                          %if editable and cs.revision in c.avail_revs:

			




	
	
	
		
 
                            ${h.radio(name='updaterev', value=cs.raw_id)}

			




	
	
	
		
 
                          %endif

			




	
	
	
		
 
                        </td>

			




	
	
	
		
 
                        <td style="width: 120px"><span data-toggle="tooltip" title="${h.age(cs.date)}">${cs.date}</span></td>

			




	
	
	
		
 
                        <td>${h.link_to(h.show_id(cs),h.url('changeset_home',repo_name=c.cs_repo.repo_name,revision=cs.raw_id), class_='changeset_hash')}</td>

			




	
	
	
		
 
                        <td>

			




	
	
	
		
 
                          <div class="pull-right" style="margin-top: -4px;">

			




	
	
	
		
 
                            %for tag in cs.tags:

			




	
	
	
		
 
                              <span class="tagtag" title="${_('Tag %s') % tag}">

			




	
	
	
		
 
                                ${h.link_to(tag,h.url('changeset_home',repo_name=c.repo_name,revision=cs.raw_id))}

			




	
	
	
		
 
                              </span>

			




	
	
	
		
 
                            %endfor

			




	
	
	
		
 
                          </div>

			




	
	
	
		
 
                          <div class="message" style="white-space:normal; height:1.1em; max-width: 500px; padding:0">${h.urlify_text(cs.message, c.repo_name)}</div>

			




	
	
	
		
 
                        </td>

			




	
	
	
		
 
                      %endif

			




	
	
	
		
 
                    </tr>

			




	
	
	
		
 
                  %endfor

			




	
	
	
		
 
                </table>

			




	
	
	
		
 
              </div>

			




	
	
	
		
 
              <div class="msg-div">(${_("Pull request iterations do not change content once created. Select a revision and save to make a new iteration.")})</div>

			




        

    



    


    



    



      

      





    
    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.