<br><br><div class="gmail_quote">On Sat, Jun 25, 2011 at 9:01 PM, Ammar Yasir <span dir="ltr"><<a href="mailto:ammaryasir.88@gmail.com">ammaryasir.88@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br><br><div class="gmail_quote"><div><div></div><div class="h5">On Tue, Jun 21, 2011 at 6:06 PM, Marc Delisle <span dir="ltr"><<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Ammar Yasir a écrit :<br>
><br>
><br>
<div>> On Mon, Jun 20, 2011 at 5:43 PM, Marc Delisle <<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a><br>
</div><div>> <mailto:<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>>> wrote:<br>
><br>
</div><div>>     Ammar Yasir a écrit :<br>
>      ><br>
>      ><br>
>      > On Sat, Jun 18, 2011 at 4:42 PM, Marc Delisle <<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a><br>
>     <mailto:<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>><br>
</div><div>>      > <mailto:<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a> <mailto:<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>>>> wrote:<br>

>      ><br>
>      >     Le 2011-06-17 18:16, Ammar Yasir a écrit :<br>
>      >      > On Fri, Jun 17, 2011 at 11:49 AM, Marc Delisle<br>
>      >     <<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a> <mailto:<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>><br>
</div><div>>     <mailto:<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a> <mailto:<a href="mailto:marc@infomarc.info" target="_blank">marc@infomarc.info</a>>>> wrote:<br>
>      >      ><br>
>      >      >> Ammar,<br>
>      >      >> it works well so far!<br>
>      >      >><br>
>      >      >> Suggestions:<br>
>      >      >><br>
>      >      >> - As the default data label, use the display field if it<br>
>     exists<br>
>      >     (from<br>
>      >      >> libraries/relation.lib.php, PMA_getDisplayField() )<br>
>      >      >><br>
>      >      >> - For the title, instead of "Scatter Plot", give some<br>
>      >     indications (how<br>
>      >      >> to zoom, how to click, etc) or give a link to get such<br>
>     indications<br>
>      >      >><br>
>      >      >> - Is it possible to enlarge the size of the working zone?<br>
>      >     ideally, to<br>
>      >      >> drag it by one of its corners?<br>
>      >      >><br>
>      ><br>
>      > I used the jquery-ui plugin and implemented this.<br>
><br>
>     Ammar,<br>
>     please push your repo.<br>
><br></div></blockquote></div></div><div><br>Hi,<br>
<br>
I was implementing the edit functionality for a data point. What I have 
done is that user selects the mode(browse / edit), mouse cursor changes 
accordingly to indicate what mode is on and in browse mode onClick, for 
now I just prompt/alert the data row. <br>
For the edit mode, I create a popup using jquery, where each field will 
be displayed just like the tbl_search page ( minus the field type, 
collation, operator elements) and the value field I will update from the
 clicked data row. Should I go through with this (creating a popup like tbl_search page) ? I've organized the tbl_search code into libraries so code wont be duplicated much.<br> </div><div class="im"><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204, 204, 204);padding-left:1ex">

<div>
> I made some changes and pushed my work.<br>
>   - Put a limit on the number of rows retrieved (500).<br>
<br>
</div>This should not be hardcoded; please create a new configuration directive.<br>
<div><br></div></blockquote></div><div>Is it fine to put it in the config.default.php file? It says 'do not edit' so just confirming.  <br></div><div><div></div><div class="h5"><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204, 204, 204);padding-left:1ex">

<div><br></div></blockquote></div></div></div></blockquote><div>'Do not edit' there is meant for the end users, not for the developers. End users are expected to change config.inc.php for the configurations.</div>
<div><br></div></div>-- <br>Thanks and Regards,<div><br></div><div>Madhura Jayaratne<br><div><br></div></div><br>