Re: Javascript errors when clicking expand buttons in diff view

Mon, 15 Feb 2016 18:28:44 -0800 

No worries Christian.

So I followed your instructions to get the fragment HTML and it strangely 
seems to include the tbody element as you are expecting (see attachment). 
I've also attached the html of the entire #diffs element after this 
fragment is inserted and you can see that the tbody elements do not make it 
through. My best guess is that whatever javascript handles inserting these 
fragments into the diff container is stripping the tbody element.

Thanks again,
Travis

-- 
Supercharge your Review Board with Power Pack: 
https://www.reviewboard.org/powerpack/
Want us to host Review Board for you? Check out RBCommons: 
https://rbcommons.com/
Happy user? Let us know! https://www.reviewboard.org/users/
--- 
You received this message because you are subscribed to the Google Groups 
"reviewboard" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
35 lines
36 
volutpat. Nulla id sollicitudin odio. Aliquam elementum sollicitudin
36 
volutpat. Nulla id sollicitudin odio. Aliquam elementum sollicitudin
37 
nisl, sit amet sagittis augue. Vivamus accumsan erat sed sem
37 
nisl, sit amet sagittis augue. Vivamus accumsan erat sed sem
38 
convallis, sed bibendum sapien accumsan. Quisque pretium vestibulum
38 
convallis, sed bibendum sapien accumsan. Quisque pretium vestibulum
39 
risus et ultricies. Fusce turpis erat, dignissim id imperdiet ac,
39 
risus et ultricies. Fusce turpis erat, dignissim id imperdiet ac,
40 
pulvinar a eros. Nullam mattis lacus et augue feugiat, eleifend
40 
pulvinar a eros. Nullam mattis lacus et augue feugiat, eleifend
41 
blandit odio aliquet. Fusce tincidunt urna turpis, fermentum accumsan
41 
blandit odio aliquet. Fusce tincidunt urna turpis, fermentum accumsan
42 
ipsum facilisis commodo. Pellentesque pretium lectus volutpat sapien
42 
ipsum facilisis commodo. Pellentesque pretium lectus volutpat sapien
43 
ultrices tristique.
43 
ultrices tristique.
44 


   

   44
   

    

   
  

  

   45
   


    
Duis interdum diam eu elit lacinia auctor. Donec sed metus a risus


   

   45
   

    
Duis interdum diam eu elit lacinia auctor. Donec sed metus a risus

   
  

  

   46
   


    
maximus ullamcorper. Aenean faucibus viverra nulla, sed malesuada mi


   

   46
   

    
maximus ullamcorper. Aenean faucibus viverra nulla, sed malesuada mi

   
  

  

   47
   


    
pulvinar non. Vestibulum dolor erat, pharetra id nisi sed, vulputate


   

   47
   

    
pulvinar non. Vestibulum dolor erat, pharetra id nisi sed, vulputate

   
  

  

   48
   


    
iaculis mi. Suspendisse quis purus at nisl bibendum bibendum sit amet


   

   48
   

    
iaculis mi. Suspendisse quis purus at nisl bibendum bibendum sit amet

   
  

  

   49
   


    
sed leo. Nullam elementum hendrerit purus at dictum. Morbi auctor mi


   

   49
   

    
sed leo. Nullam elementum hendrerit purus at dictum. Morbi auctor mi

   
  

  

   50
   


    
diam, eu viverra urna dignissim at. Fusce pellentesque, eros a


   

   50
   

    
diam, eu viverra urna dignissim at. Fusce pellentesque, eros a

   
  

  

   51
   


    
malesuada tempor, neque nisi feugiat metus, id volutpat justo orci eu


   

   51
   

    
malesuada tempor, neque nisi feugiat metus, id volutpat justo orci eu

   
  

  

   52
   


    
libero. Aliquam vitae turpis et enim tempus aliquet et in sem. Nam


   

   52
   

    
libero. Aliquam vitae turpis et enim tempus aliquet et in sem. Nam

   
  

  

   53
   


    
purus orci, ultrices id vulputate ac, molestie vitae dui. Vestibulum


   

   53
   

    
purus orci, ultrices id vulputate ac, molestie vitae dui. Vestibulum

   
  

  

   54
   


    
rhoncus est vitae lorem pharetra euismod. Nam tempor non orci id


   

   54
   

    
rhoncus est vitae lorem pharetra euismod. Nam tempor non orci id

   
  

  

   55
   


    
luctus. Aenean ultricies turpis id ex ornare, quis dictum sapien


   

   55
   

    
luctus. Aenean ultricies turpis id ex ornare, quis dictum sapien

   
  

 

 
















 


35 lines


 




















   36



 

  

   

  


    
volutpat. Nulla id sollicitudin odio. Aliquam elementum sollicitudin







   36


    
volutpat. Nulla id sollicitudin odio. Aliquam elementum sollicitudin






   37



    
nisl, sit amet sagittis augue. Vivamus accumsan erat sed sem




   37


    
nisl, sit amet sagittis augue. Vivamus accumsan erat sed sem






   38



    
convallis, sed bibendum sapien accumsan. Quisque pretium vestibulum




   38


    
convallis, sed bibendum sapien accumsan. Quisque pretium vestibulum






   39



    
risus et ultricies. Fusce turpis erat, dignissim id imperdiet ac,




   39


    
risus et ultricies. Fusce turpis erat, dignissim id imperdiet ac,






   40



    
pulvinar a eros. Nullam mattis lacus et augue feugiat, eleifend




   40


    
pulvinar a eros. Nullam mattis lacus et augue feugiat, eleifend






   41



    
blandit odio aliquet. Fusce tincidunt urna turpis, fermentum accumsan




   41


    
blandit odio aliquet. Fusce tincidunt urna turpis, fermentum accumsan






   42



    
ipsum facilisis commodo. Pellentesque pretium lectus volutpat sapien




   42


    
ipsum facilisis commodo. Pellentesque pretium lectus volutpat sapien






   43



    
ultrices tristique.




   43


    
ultrices tristique.






   44



    




   44


    






   45



    
Duis interdum diam eu elit lacinia auctor. Donec sed metus a risus




   45


    
Duis interdum diam eu elit lacinia auctor. Donec sed metus a risus






   46



    
maximus ullamcorper. Aenean faucibus viverra nulla, sed malesuada mi




   46


    
maximus ullamcorper. Aenean faucibus viverra nulla, sed malesuada mi






   47



    
pulvinar non. Vestibulum dolor erat, pharetra id nisi sed, vulputate




   47


    
pulvinar non. Vestibulum dolor erat, pharetra id nisi sed, vulputate






   48



    
iaculis mi. Suspendisse quis purus at nisl bibendum bibendum sit amet




   48


    
iaculis mi. Suspendisse quis purus at nisl bibendum bibendum sit amet






   49



    
sed leo. Nullam elementum hendrerit purus at dictum. Morbi auctor mi




   49


    
sed leo. Nullam elementum hendrerit purus at dictum. Morbi auctor mi






   50



    
diam, eu viverra urna dignissim at. Fusce pellentesque, eros a




   50


    
diam, eu viverra urna dignissim at. Fusce pellentesque, eros a






   51



    
malesuada tempor, neque nisi feugiat metus, id volutpat justo orci eu




   51


    
malesuada tempor, neque nisi feugiat metus, id volutpat justo orci eu






   52



    
libero. Aliquam vitae turpis et enim tempus aliquet et in sem. Nam




   52


    
libero. Aliquam vitae turpis et enim tempus aliquet et in sem. Nam






   53



    
purus orci, ultrices id vulputate ac, molestie vitae dui. Vestibulum




   53


    
purus orci, ultrices id vulputate ac, molestie vitae dui. Vestibulum






   54



    
rhoncus est vitae lorem pharetra euismod. Nam tempor non orci id




   54


    
rhoncus est vitae lorem pharetra euismod. Nam tempor non orci id






   55



    
luctus. Aenean ultricies turpis id ex ornare, quis dictum sapien




   55


    
luctus. Aenean ultricies turpis id ex ornare, quis dictum sapien

 

  
  

  
  
 
 
  


   
    

    lorem_ipsum.txt
  

  

  



   
   

    

    Revision 80e3fca3ee8f282c83103fded5755fca4ba2c0a0
    

   

   

    

    New Change
   


  

 


























































 

  


   56
   


    
bibendum.


   		

   56
   

    
bibendum.

   		
  


  


   57
   


    


   		

   57
   

    

   		
  


  


   58
   


    
Duis tincidunt aliquam lorem vitae accumsan. Vestibulum blandit purus


   		

   58
   

    
Duis tincidunt aliquam lorem vitae accumsan. Vestibulum blandit purus

   		
  


  


   59
   


    
quis odio semper, quis rhoncus leo blandit. Integer tristique


   		

   59
   

    
quis odio semper, quis rhoncus leo blandit. Integer tristique

   		
  


  


   60
   


    
porttitor sem vitae porttitor. Suspendisse pellentesque, turpis et


   		

   60
   

    
porttitor sem vitae porttitor. Suspendisse pellentesque, turpis et

   		
  


 



 

  


   
 
61
   


    
commodo faucibus, nulla diam ornare enim, in eleifend lorem mauris


   		

   61
   

    
commodo faucibus, Test change please ignore nulla diam ornare enim, in eleifend lorem mauris

   		
  


 



 

  


   62
   


    
vitae mauris. Aliquam a mi erat. Donec enim lorem, consectetur id


   		

   62
   

    
vitae mauris. Aliquam a mi erat. Donec enim lorem, consectetur id

   		
  


  


   63
   


    
hendrerit ac, tempor id mi. Mauris consectetur ipsum vitae arcu


   		

   63
   

    
hendrerit ac, tempor id mi. Mauris consectetur ipsum vitae arcu

   		
  


  


   64
   


    
eleifend, non sollicitudin erat tincidunt. Nam sagittis suscipit nibh


   		

   64
   

    
eleifend, non sollicitudin erat tincidunt. Nam sagittis suscipit nibh

   		
  


  


   65
   


    
sit amet ullamcorper. Praesent fringilla quis nulla a ornare. Sed ac


   		

   65
   

    
sit amet ullamcorper. Praesent fringilla quis nulla a ornare. Sed ac

   		
  


  


   66
   


    
posuere lacus. Nam auctor lobortis metus faucibus faucibus. Maecenas


   		

   66
   

    
posuere lacus. Nam auctor lobortis metus faucibus faucibus. Maecenas

   		
  


 



 
  

   




 








   		
   




 


44 lines




   		
  


 








 



























					Reply via email to