Hi Marc,<br><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
1. please follow Rouslan's suggestions; however, the variable, after the<br>
split, really contains a span id (to find the other footnote span)<br>
<br>
2. please add a comment just before the line containing the split,<br>
explaining that this variable contains two classes at this point (for<br>
example "footnotemarker footnote_1_1")<br>
<br>
3. also add a comment, using for example footnote_2_3 and that after the<br>
split, the variable contains 2, so will find the footnote #2<br>
<br></blockquote><div> </div><div>I did the above changes and pushed to the repo.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I find it a bit hazardous to use this split on a string which contains<br>
two classes. You never know, another class having an underscore could be<br>
added to this element in the future.<br>
<br></blockquote><div><br></div><div>I change the logic a little, and in that case it is more safe with splitting and getting the id.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">


Another thing I find unclear: in  and<br>
PMA_convertFootnotesToTooltips()  you have kept the principle of using<br>
the instance of the footnote, even though the goal of the instance was<br>
only to have unique ids. IMO you should get rid of the instance.<br>
<div><br></div></blockquote><div><br></div><div>I removed it and pushed my changes to the repo.</div><div><br></div><div>Now I find two new errors with this.One is an firebug error which says w"<span class="Apple-style-span" style="color: rgb(255, 0, 0); font-family: 'dejavu sans mono', monospace; font-size: 11px; white-space: pre-wrap; "><a href="http://localhost/phpmyadmin-gsoc/themes/pmahomme/img/ic_b_triggers" class="webkit-html-resource-link" title="http://localhost/phpmyadmin-gsoc/themes/pmahomme/img/ic_b_triggers" style="box-sizing: border-box; color: rgb(38, 38, 38); cursor: pointer; max-width: 100%; ">http://localhost/phpmyadmin-gsoc/themes/pmahomme/img/ic_b_triggers</a> 404 (Not Found)</span>" when I click on "DB->Structure->insert". It appears in non ajax scenario also. </div>
<div><br></div><div>The next one is when I go to "Sakila" database and try to insert for actor_info table it gives an error. But in ajax enabled dialog shows the json string is not parsed clearly. I cannot parse the whole data content at the "$("td.insert_table a.ajax").live('click')" action in db_structure.js file because the returned content from tbl_change.php file is not 100% an json response. That ajax response was passed at the line 604 in common.lib.php and it seems not parsing correctly. Can you please help me on this issue. Thank you. </div>
<div><br></div><div>Regards,</div><div>Thilanka. </div><div><br></div><div><br></div></div>-- <br><div><span style="border-collapse:collapse;font-family:arial, sans-serif;font-size:13px"><div style="text-align:left">L G Thilanka Kaushalya     <br>
Computer Science & Engineering,    <br><div style="color:rgb(80, 0, 80)">
University of Moratuwa,<br>Sri Lanka<span></span><br>                          <br></div></div>Connect with me at,<br>      Gmail        : <a href="mailto:lgtkaushalya@gmail.com" style="color:rgb(0, 0, 204)" target="_blank">lgtkaushalya@gmail.com</a><br>

      Twitter       : @thilanka_k<br>      Facebook  : Thilanka Kaushalya<br>      Yahoo       : <a href="mailto:lgtkaushalya@yahoo.com" style="color:rgb(0, 0, 204)" target="_blank">lgtkaushalya@yahoo.com</a></span></div>

<div>      Blog          : <a href="http://coders-view.blogspot.com/" target="_blank">http://coders-view.blogspot.com/</a></div><br><br>