<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Jun 16, 2014 at 7:14 PM, Isaac Bennetch <span dir="ltr"><<a href="mailto:bennetch@gmail.com" target="_blank">bennetch@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello,<br>
<div class=""><br>
On 6/16/14 8:09 AM, Chirayu Chiripal wrote:<br>
> Hello Isaac Sir,<br>
><br>
> Please review this branch: <a href="https://github.com/D-storm/phpmyadmin/tree/FR-755" target="_blank">https://github.com/D-storm/phpmyadmin/tree/FR-755</a><br>
><br>
> Tasks to be done yet (Will be pushed soon):<br>
><br>
> 1) Fix failing test cases and improve test cases.<br>
<br>
</div>Yes, this is quite good but I have a few minor suggestions.<br>
<br>
1) There should be some way to inform the user once they've selected<br>
this option. I have a few ideas about this but don't prefer one over the<br>
other at the moment.<br>
<br>
What do you think of adding a checkbox in front of the text "Change<br>
Database/Table/Column names". When the user clicks the checkbox or the<br>
text, the dialog appears; once the dialog is closed the checkbox is checked.<br>
<br>
An alternative is making an icon appear in front of the text (a green<br>
checkbox, probably) once the dialog is closed.<br>
<br>
I think it's important to give visual feedback that the feature is going<br>
to be active.<br></blockquote><div><br></div><div>I think I'll go with green checkbox.<br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
2) I suggest adding a "reset" button to the dialog which would clear all<br>
values. I also think the upper-right X (close button) should abandon any<br>
changes made; right now it saves those changes.<br></blockquote><div><br></div><div>Actually, there is nothing like save here, those input fields (initially hidden) are just displayed when dialog opens and hides when dialog closes but I'll see how can it be done. Should it reset every field  i.e. all databases/tables/columns inputs?<br>
</div><div>Also, then we should change "close" button to "save & close".<br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
3) I think we should call this "Rename exported database/column/table<br>
names" -- but I'm open to other suggestions.<br></blockquote><div><br></div><div>This sounds good enough. I will change it soon.<br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
4) I've tested with many columns and the dialog properly adjusts<br>
downward, so this is good behavior.<br>
<br>
I'll make some comments in GitHub as well.<br>
<br>
Thanks.<br>
<div class=""><br>
<br>
> --<br>
> Regards,<br>
> Chirayu Chiripal<br>
> phpMyAdmin Intern - Google Summer of Code 2014<br>
> <a href="https://chirayuchiripal.wordpress.com/" target="_blank">https://chirayuchiripal.wordpress.com/</a><br>
</div></blockquote></div><br></div></div>