<div dir="ltr">Atul, I get that, that is the normal workflow, but this is a new feature this must go to master branch when the pull is merged like it happened this time. <div>Although I have a feeling that <b>matching the values of input fields against the values they were loaded with would be a good solution to this</b>, rather than checking if length of visible input.text fields, i think rather than merging this directly to master, in pma main repo if it is merged with some other branch and other members have confirmed no bugs exist, it can be merged to master. </div>

<div><br></div><div><br></div></div><div class="gmail_extra"><br clear="all"><div><div dir="ltr"><div><br></div><font color="#666666">Minhaz</font><div><font color="#666666"><a href="http://cistoner.org" target="_blank">cistoner.org</a></font></div>

</div></div>
<br><br><div class="gmail_quote">On Tue, Apr 22, 2014 at 5:23 PM, Atul Pratap Singh <span dir="ltr"><<a href="mailto:atulpratapsingh05@gmail.com" target="_blank">atulpratapsingh05@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div class="">On Tue, Apr 22, 2014 at 5:16 PM, Minhaz A V <span dir="ltr"><<a href="mailto:minhazav@gmail.com" target="_blank">minhazav@gmail.com</a>></span> wrote:<br>



<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<div>I see the code is showing behavior other than expected behavior at certain sections.</div><div>


I was thinking this code could be modified to check if any input field was changed from the value it was loaded with if some user try to perform ajax operation or any operation that triggers onbeforeunload event.</div>


<div><br></div><div>AJAX code will go well in ajax.js while for onbeforeunload section, I'd add an event listener to it in common.js as mentioned by Atul.</div><div><br></div><div>If you guys feel this feature should not exist in pma, I'd stop here on this. Kindly let me know by night or I'll start writing :D, as I feel this feature should be there.</div>





<div><br></div><div>Also, is it possible to add a branch for this in pma, till this feature is working well or I should come up with a whole working one in my fork before I pull.</div></div><div class="gmail_extra"><br clear="all">



</div></blockquote></div><div>About a separate branch, it is not meant to be created in main PMA repository, instead you can create a new branch out of master in your local repo and push it to your github account for public review, then you can open a pull request from your branch to the main PMA repo's master branch when it's all done.</div>



</div><div class=""><br clear="all"><div><br></div>-- <br>Atul Pratap Singh<div><a href="http://blog.atulsisodia.in/" target="_blank">http://blog.atulsisodia.in/</a></div>
</div></div></div>
<br>------------------------------------------------------------------------------<br>
Start Your Social Network Today - Download eXo Platform<br>
Build your Enterprise Intranet with eXo Platform Software<br>
Java Based Open Source Intranet - Social, Extensible, Cloud Ready<br>
Get Started Now And Turn Your Intranet Into A Collaboration Platform<br>
<a href="http://p.sf.net/sfu/ExoPlatform" target="_blank">http://p.sf.net/sfu/ExoPlatform</a><br>_______________________________________________<br>
Phpmyadmin-devel mailing list<br>
<a href="mailto:Phpmyadmin-devel@lists.sourceforge.net">Phpmyadmin-devel@lists.sourceforge.net</a><br>
<a href="https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel" target="_blank">https://lists.sourceforge.net/lists/listinfo/phpmyadmin-devel</a><br>
<br></blockquote></div><br></div>