<br><br><div class="gmail_quote">On Fri, Jul 8, 2011 at 1:38 AM, Marc Delisle <span dir="ltr"><<a href="mailto:marc@infomarc.info">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;">
Le 2011-07-07 09:56, Marc Delisle a écrit :<br>
<div class="im">> Le 2011-07-06 13:36, Ammar Yasir a écrit :<br>
>> On Wed, Jul 6, 2011 at 1:36 AM, Marc Delisle<<a href="mailto:marc@infomarc.info">marc@infomarc.info</a>>  wrote:<br>
>><br>
>>> Le 2011-07-05 08:50, Ammar Yasir a écrit :<br>
>>><br>
>>>>      * The link for 'How to use?' / instructions to use is available<br>
>>>>        again by using PMA_ajaxShowMessage<br>
>>><br>
>>> OK but I don't have the time to read the full message; I must click<br>
>>> again and this is annoying. Maybe leave it in place until I click again?<br>
>>><br>
>>> Maybe just increase the time to 10 sec? or it may be 10 sec is too long ?<br>
><br>
> 10 sec would be ok; but can the message disappear if I click?<br>
</div>Ammar,<br>
I looked at your repo having commit<br>
57dc8ce4a595498a7de06ee34c7aa5129da7f934.<br>
<br>
Message is fine now. I suggest replacing<br>
  Click data points to view the data row.<br>
with<br>
  Click a data point to view and possibly edit the data row.<br>
<div class="im"><br>
<br></div></blockquote><div>Changed. <br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div class="im">
<br>
>>> It would be so much simpler if the interface was always in Edit mode. If<br>
>>> I don't want to edit, I just don't edit.<br>
>>><br>
>>> Ok, I'll remove the feature.<br>
><br>
> Thanks.<br>
<br>
<br>
</div>Works great now. I have a suggestion: when someone submits an edit,<br>
since the row's data remains visible, the corresponding data point<br>
should remain selected.<br>
<br></blockquote><div>Fixed.<br><br>I've done a basic version of edit functionality. I added a unique identifier for each row in the query result data by using the PMA_getUniqueCondition() from the common.lib.php file (where_clause). I then generate the update query in the edit submit event using this where clause and post it to sql.php. The return data is added to the  <sqlqueryresult> div to show the query executed and the message. The plot also updates on submit if one of the axes is updated. Is my logic is okay?<br>
I've pushed my work and you can try scenarios on the film_actor table in the sakila database.<br> <br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

Please please please: use shorter commits. Your comment says "Removed<br>
mode feature, appended a missing configuration directive and changed<br>
help link duration to 10sec"; splitting these in 3 commits would have<br>
made understanding each change easier.<br>
<div><div></div><div class="h5"><br>
><br>
>>><br>
>>> Another thing: why remove this line from config.default.php?<br>
>>> $cfg['ShowDisplayDir'] = false;<br>
>>><br>
>>> Reverted the change.<br>
><br>
> Thanks.<br>
><br>
> I'll have a look at your tree later today.<br>
><br>
>><br>
>>>><br>
>>>> Now working on to update query generation on edit mode. Also, will check<br>
>>>> out the code snippet for panning tyron suggested.<br>
>>>> I'll look into the default label problem in some time (testing using the<br>
>>>> pma configuration storage).<br>
><br>
<br>
<br>
--<br>
Marc Delisle<br>
<a href="http://infomarc.info" target="_blank">http://infomarc.info</a><br>
<br>
------------------------------------------------------------------------------<br>
All of the data generated in your IT infrastructure is seriously valuable.<br>
Why? It contains a definitive record of application performance, security<br>
threats, fraudulent activity, and more. Splunk takes this data and makes<br>
sense of it. IT sense. And common sense.<br>
<a href="http://p.sf.net/sfu/splunk-d2d-c2" target="_blank">http://p.sf.net/sfu/splunk-d2d-c2</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>
</div></div></blockquote></div><br>