You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ofbiz.apache.org by "Mridul Pathak (JIRA)" <ji...@apache.org> on 2008/07/01 18:24:45 UTC

[jira] Created: (OFBIZ-1859) Implement Ajax.InPlaceEditor for display field in Form Widget

Implement Ajax.InPlaceEditor for display field in Form Widget
-------------------------------------------------------------

                 Key: OFBIZ-1859
                 URL: https://issues.apache.org/jira/browse/OFBIZ-1859
             Project: OFBiz
          Issue Type: New Feature
          Components: framework
    Affects Versions: SVN trunk
            Reporter: Mridul Pathak
             Fix For: SVN trunk


Add ability to In Place Edit a <display> field in Form Widget. After this enhancement a developer can edit a single/multiple line in a display field of form widget using Ajax.InPlaceEditor without a page refresh or separate Edit Form.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Re: [jira] Updated: (OFBIZ-1859) Implement Ajax.InPlaceEditor for display field in Form Widget

Posted by Mridul Pathak <mr...@hotwaxmedia.com>.
Hi Bruno,

     The patch for this fix that I have uploaded to this issue is not  
yet checked in to OFBiz trunk.  So, it won't work on the demo server.   
I tested the patch again and it is working fine with pagination.

     Try applying this patch on your local copy and see if it works  
for you.  This will be another test for the patch as well :).

--
Thanks & Regards,

Mridul Pathak
HotWax Media, Inc.
http://www.hotwaxmedia.com
mridul.pathak@hotwaxmedia.com






On Jul 26, 2008, at 12:14 PM, Bruno Busco wrote:

> Thank you Mridul,
> I see that now it is working well.
> Thank you for this great improvement I look forward to use it  
> exensively!
>
> Unfortunately, going further and tryng to use it with a paginated  
> form it
> still does not work well.
>
> Try the following:
> 1) Go to
> https://demo.hotwaxmedia.com/example/control/authview/findExampleAjax
> 2) Add more that 10 examples so that the pagination feature comes up
> 3) Press the next button
> --> The in-place-editor becomes active only on the first row and when
> clicked several editing rows are displayed.
>
> Thanks,
> Bruno
>
>
> 2008/7/25 Mridul Pathak (JIRA) <ji...@apache.org>
>
>>
>>    [
>> https://issues.apache.org/jira/browse/OFBIZ-1859?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel 
>> ]
>>
>> Mridul Pathak updated OFBIZ-1859:
>> ---------------------------------
>>
>>    Attachment: AjaxInPlaceEditor_Improvements.patch
>>
>> Fixed the bug reported by Bruno Busco in reply to the commit 678997.
>> Thanks Bruno.
>>
>>> Implement Ajax.InPlaceEditor for display field in Form Widget
>>> -------------------------------------------------------------
>>>
>>>                Key: OFBIZ-1859
>>>                URL: https://issues.apache.org/jira/browse/OFBIZ-1859
>>>            Project: OFBiz
>>>         Issue Type: New Feature
>>>         Components: framework
>>>   Affects Versions: SVN trunk
>>>           Reporter: Mridul Pathak
>>>            Fix For: SVN trunk
>>>
>>>        Attachments: AjaxInPlaceEditor.patch,  
>>> AjaxInPlaceEditor.patch,
>> AjaxInPlaceEditor.patch, AjaxInPlaceEditor_Improvements.patch,
>> AjaxInPlaceEditor_Improvements.patch,  
>> AjaxInPlaceEditor_Improvements.patch,
>> AjaxInPlaceEditor_Improvements.patch
>>>
>>>
>>> Add ability to In Place Edit a <display> field in Form Widget.  
>>> After this
>> enhancement a developer can edit a single/multiple line in a  
>> display field
>> of form widget using Ajax.InPlaceEditor without a page refresh or  
>> separate
>> Edit Form.
>>> Please refer to following document for more details about this
>> enhancement:
>> http://docs.ofbiz.org/display/OFBIZ/Enhancing+Display+Widget+to+use+Ajax.InPlaceEditor
>>
>> --
>> This message is automatically generated by JIRA.
>> -
>> You can reply to this email to add a comment to the issue online.
>>
>>


Re: [jira] Updated: (OFBIZ-1859) Implement Ajax.InPlaceEditor for display field in Form Widget

Posted by Bruno Busco <br...@gmail.com>.
Thank you Mridul,
I see that now it is working well.
Thank you for this great improvement I look forward to use it exensively!

Unfortunately, going further and tryng to use it with a paginated form it
still does not work well.

Try the following:
1) Go to
https://demo.hotwaxmedia.com/example/control/authview/findExampleAjax
2) Add more that 10 examples so that the pagination feature comes up
3) Press the next button
--> The in-place-editor becomes active only on the first row and when
clicked several editing rows are displayed.

Thanks,
Bruno


2008/7/25 Mridul Pathak (JIRA) <ji...@apache.org>

>
>     [
> https://issues.apache.org/jira/browse/OFBIZ-1859?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel]
>
> Mridul Pathak updated OFBIZ-1859:
> ---------------------------------
>
>     Attachment: AjaxInPlaceEditor_Improvements.patch
>
> Fixed the bug reported by Bruno Busco in reply to the commit 678997.
>  Thanks Bruno.
>
> > Implement Ajax.InPlaceEditor for display field in Form Widget
> > -------------------------------------------------------------
> >
> >                 Key: OFBIZ-1859
> >                 URL: https://issues.apache.org/jira/browse/OFBIZ-1859
> >             Project: OFBiz
> >          Issue Type: New Feature
> >          Components: framework
> >    Affects Versions: SVN trunk
> >            Reporter: Mridul Pathak
> >             Fix For: SVN trunk
> >
> >         Attachments: AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch,
> AjaxInPlaceEditor.patch, AjaxInPlaceEditor_Improvements.patch,
> AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch,
> AjaxInPlaceEditor_Improvements.patch
> >
> >
> > Add ability to In Place Edit a <display> field in Form Widget. After this
> enhancement a developer can edit a single/multiple line in a display field
> of form widget using Ajax.InPlaceEditor without a page refresh or separate
> Edit Form.
> > Please refer to following document for more details about this
> enhancement:
> http://docs.ofbiz.org/display/OFBIZ/Enhancing+Display+Widget+to+use+Ajax.InPlaceEditor
>
> --
> This message is automatically generated by JIRA.
> -
> You can reply to this email to add a comment to the issue online.
>
>

[jira] Updated: (OFBIZ-1859) Implement Ajax.InPlaceEditor for display field in Form Widget

Posted by "Mridul Pathak (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/OFBIZ-1859?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mridul Pathak updated OFBIZ-1859:
---------------------------------

    Description: 
Add ability to In Place Edit a <display> field in Form Widget. After this enhancement a developer can edit a single/multiple line in a display field of form widget using Ajax.InPlaceEditor without a page refresh or separate Edit Form.

Please refer to following document for more details about this enhancement: http://docs.ofbiz.org/display/OFBIZ/Enhancing+Display+Widget+to+use+Ajax.InPlaceEditor

  was:Add ability to In Place Edit a <display> field in Form Widget. After this enhancement a developer can edit a single/multiple line in a display field of form widget using Ajax.InPlaceEditor without a page refresh or separate Edit Form.


> Implement Ajax.InPlaceEditor for display field in Form Widget
> -------------------------------------------------------------
>
>                 Key: OFBIZ-1859
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1859
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Mridul Pathak
>             Fix For: SVN trunk
>
>         Attachments: AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch
>
>
> Add ability to In Place Edit a <display> field in Form Widget. After this enhancement a developer can edit a single/multiple line in a display field of form widget using Ajax.InPlaceEditor without a page refresh or separate Edit Form.
> Please refer to following document for more details about this enhancement: http://docs.ofbiz.org/display/OFBIZ/Enhancing+Display+Widget+to+use+Ajax.InPlaceEditor

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (OFBIZ-1859) Implement Ajax.InPlaceEditor for display field in Form Widget

Posted by "Anil K Patel (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/OFBIZ-1859?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12615867#action_12615867 ] 

Anil K Patel commented on OFBIZ-1859:
-------------------------------------

Mridul, thanks for improvements. Your most recent patch is in trunk rev#678979. 

> Implement Ajax.InPlaceEditor for display field in Form Widget
> -------------------------------------------------------------
>
>                 Key: OFBIZ-1859
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1859
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Mridul Pathak
>             Fix For: SVN trunk
>
>         Attachments: AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch
>
>
> Add ability to In Place Edit a <display> field in Form Widget. After this enhancement a developer can edit a single/multiple line in a display field of form widget using Ajax.InPlaceEditor without a page refresh or separate Edit Form.
> Please refer to following document for more details about this enhancement: http://docs.ofbiz.org/display/OFBIZ/Enhancing+Display+Widget+to+use+Ajax.InPlaceEditor

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (OFBIZ-1859) Implement Ajax.InPlaceEditor for display field in Form Widget

Posted by "Mridul Pathak (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/OFBIZ-1859?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mridul Pathak updated OFBIZ-1859:
---------------------------------

    Attachment: AjaxInPlaceEditor.patch

This is the update patch.  This fixes the problem in last patch where whole responseText was shown in place of the edited field.  Now the updated value is shown correctly.  Please verify my approach to fix the problem.

> Implement Ajax.InPlaceEditor for display field in Form Widget
> -------------------------------------------------------------
>
>                 Key: OFBIZ-1859
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1859
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Mridul Pathak
>             Fix For: SVN trunk
>
>         Attachments: AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch
>
>
> Add ability to In Place Edit a <display> field in Form Widget. After this enhancement a developer can edit a single/multiple line in a display field of form widget using Ajax.InPlaceEditor without a page refresh or separate Edit Form.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (OFBIZ-1859) Implement Ajax.InPlaceEditor for display field in Form Widget

Posted by "Mridul Pathak (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/OFBIZ-1859?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mridul Pathak updated OFBIZ-1859:
---------------------------------

    Attachment: AjaxInPlaceEditor_Improvements.patch

Uploading the updated patch for improvements with just a small change.  Moved example for InPlaceEditor from partymgr to example component.

> Implement Ajax.InPlaceEditor for display field in Form Widget
> -------------------------------------------------------------
>
>                 Key: OFBIZ-1859
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1859
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Mridul Pathak
>             Fix For: SVN trunk
>
>         Attachments: AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch
>
>
> Add ability to In Place Edit a <display> field in Form Widget. After this enhancement a developer can edit a single/multiple line in a display field of form widget using Ajax.InPlaceEditor without a page refresh or separate Edit Form.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (OFBIZ-1859) Implement Ajax.InPlaceEditor for display field in Form Widget

Posted by "Bruno Busco (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/OFBIZ-1859?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12617156#action_12617156 ] 

Bruno Busco commented on OFBIZ-1859:
------------------------------------

BTW...
...do you think it is possible to have the highlight removed when the mouse leaves the field (more standard) and not after a fixed time?
I know may be asking too much but...who knows... ;-)

> Implement Ajax.InPlaceEditor for display field in Form Widget
> -------------------------------------------------------------
>
>                 Key: OFBIZ-1859
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1859
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Mridul Pathak
>             Fix For: SVN trunk
>
>         Attachments: AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch
>
>
> Add ability to In Place Edit a <display> field in Form Widget. After this enhancement a developer can edit a single/multiple line in a display field of form widget using Ajax.InPlaceEditor without a page refresh or separate Edit Form.
> Please refer to following document for more details about this enhancement: http://docs.ofbiz.org/display/OFBIZ/Enhancing+Display+Widget+to+use+Ajax.InPlaceEditor

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (OFBIZ-1859) Implement Ajax.InPlaceEditor for display field in Form Widget

Posted by "Bruno Busco (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/OFBIZ-1859?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12617139#action_12617139 ] 

Bruno Busco commented on OFBIZ-1859:
------------------------------------

I have tested your patch locally and it is ok. Thank you!

BTW here is a finding I think it is not strictly related to this but more to https://issues.apache.org/jira/browse/OFBIZ-1889.
I write it here because on the ML I had (wrongly) reported it as related to your patch but I do not think so anymore:

I have a "multi" tipe form in which I use a in-place-edit field, the _rowSubmit checkboxes, and the Ajax pagination...wow ! ;-)
(popUps coming soon... ;-) )

Well, in this case when the pagination is activated (and used with the next button) the "Select all" checkbox disappears from the table header AND the in-field-edit starts working on the first row only.


> Implement Ajax.InPlaceEditor for display field in Form Widget
> -------------------------------------------------------------
>
>                 Key: OFBIZ-1859
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1859
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Mridul Pathak
>             Fix For: SVN trunk
>
>         Attachments: AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch
>
>
> Add ability to In Place Edit a <display> field in Form Widget. After this enhancement a developer can edit a single/multiple line in a display field of form widget using Ajax.InPlaceEditor without a page refresh or separate Edit Form.
> Please refer to following document for more details about this enhancement: http://docs.ofbiz.org/display/OFBIZ/Enhancing+Display+Widget+to+use+Ajax.InPlaceEditor

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (OFBIZ-1859) Implement Ajax.InPlaceEditor for display field in Form Widget

Posted by "Mridul Pathak (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/OFBIZ-1859?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mridul Pathak updated OFBIZ-1859:
---------------------------------

    Attachment: AjaxInPlaceEditor_Improvements.patch

The same bug that was reported for list forms is applicable to multi forms as well.  Uploading the patch for the fix.
Thanks Bruno for drawing my attention to that.

Hi Jacques,  just when you were committing the previous patch I was testing this bug fix for multi form,  if I could have done it early you could have committed a single patch for both the fixes. :)

> Implement Ajax.InPlaceEditor for display field in Form Widget
> -------------------------------------------------------------
>
>                 Key: OFBIZ-1859
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1859
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Mridul Pathak
>             Fix For: SVN trunk
>
>         Attachments: AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch
>
>
> Add ability to In Place Edit a <display> field in Form Widget. After this enhancement a developer can edit a single/multiple line in a display field of form widget using Ajax.InPlaceEditor without a page refresh or separate Edit Form.
> Please refer to following document for more details about this enhancement: http://docs.ofbiz.org/display/OFBIZ/Enhancing+Display+Widget+to+use+Ajax.InPlaceEditor

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (OFBIZ-1859) Implement Ajax.InPlaceEditor for display field in Form Widget

Posted by "Bruno Busco (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/OFBIZ-1859?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12617155#action_12617155 ] 

Bruno Busco commented on OFBIZ-1859:
------------------------------------

Thank you Mridul,
your patch makes my "multi" form work now.

> Implement Ajax.InPlaceEditor for display field in Form Widget
> -------------------------------------------------------------
>
>                 Key: OFBIZ-1859
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1859
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Mridul Pathak
>             Fix For: SVN trunk
>
>         Attachments: AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch
>
>
> Add ability to In Place Edit a <display> field in Form Widget. After this enhancement a developer can edit a single/multiple line in a display field of form widget using Ajax.InPlaceEditor without a page refresh or separate Edit Form.
> Please refer to following document for more details about this enhancement: http://docs.ofbiz.org/display/OFBIZ/Enhancing+Display+Widget+to+use+Ajax.InPlaceEditor

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (OFBIZ-1859) Implement Ajax.InPlaceEditor for display field in Form Widget

Posted by "Mridul Pathak (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/OFBIZ-1859?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mridul Pathak updated OFBIZ-1859:
---------------------------------

    Attachment: AjaxInPlaceEditor.patch

Uploaded the updated patch with few corrections.

> Implement Ajax.InPlaceEditor for display field in Form Widget
> -------------------------------------------------------------
>
>                 Key: OFBIZ-1859
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1859
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Mridul Pathak
>             Fix For: SVN trunk
>
>         Attachments: AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch
>
>
> Add ability to In Place Edit a <display> field in Form Widget. After this enhancement a developer can edit a single/multiple line in a display field of form widget using Ajax.InPlaceEditor without a page refresh or separate Edit Form.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (OFBIZ-1859) Implement Ajax.InPlaceEditor for display field in Form Widget

Posted by "Mridul Pathak (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/OFBIZ-1859?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mridul Pathak updated OFBIZ-1859:
---------------------------------

    Attachment: AjaxInPlaceEditor.patch

Here is the patch for this enhancement.  Currently it supports single line and multiple line InPlaceEditor.  Please suggest any improvements.

> Implement Ajax.InPlaceEditor for display field in Form Widget
> -------------------------------------------------------------
>
>                 Key: OFBIZ-1859
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1859
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Mridul Pathak
>             Fix For: SVN trunk
>
>         Attachments: AjaxInPlaceEditor.patch
>
>
> Add ability to In Place Edit a <display> field in Form Widget. After this enhancement a developer can edit a single/multiple line in a display field of form widget using Ajax.InPlaceEditor without a page refresh or separate Edit Form.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (OFBIZ-1859) Implement Ajax.InPlaceEditor for display field in Form Widget

Posted by "Jacques Le Roux (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/OFBIZ-1859?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12617142#action_12617142 ] 

Jacques Le Roux commented on OFBIZ-1859:
----------------------------------------

Hi Mridul,

It works well but I found a weird behaviour in FF3 and Opera 9.02 (not in IE7). Let me explain what happened to me.

I 1st tried to to create a new line while having still an in place editor open in FF3. I used to way for that, hitting enter and clicking on the Create button. In both case I got a dialog box to save acreateExampleAjax file which contained
{"javax.servlet.request.cipher_suite":"TLS_DHE_RSA_WITH_AES_128_CBC_SHA","targetRequestUri":"/createExampleAjax","javax.servlet.request.key_size":128,"exampleId":"10014","thisRequestUri":"createExampleAjax","multiPartMap":{},"_CONTROL_PATH_":"/example/control","_CONTEXT_ROOT_":"D:\\workspace\\ofbizRun\\framework\\example\\webapp\\example\\","javax.servlet.request.ssl_session":"488aeb7ea3bbeb574646e869f357e8ea365b0b9a297b4cb0cd4038d1aa1937d1","_SERVER_ROOT_URL_":"https://localhost:18443"}

Then I tried on IE7, nothing happened with the enter key and Create button worked fine. 

With Opera 9.02 only the enter key proposed to create the file but even without editing with the in place editor.

Then I retried with FF3 and I got the same behaviour than with IE7 (ie OK), really weird :D

It's really a cool feature so I commited this fix in trunk rev. 679975

> Implement Ajax.InPlaceEditor for display field in Form Widget
> -------------------------------------------------------------
>
>                 Key: OFBIZ-1859
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1859
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Mridul Pathak
>             Fix For: SVN trunk
>
>         Attachments: AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch
>
>
> Add ability to In Place Edit a <display> field in Form Widget. After this enhancement a developer can edit a single/multiple line in a display field of form widget using Ajax.InPlaceEditor without a page refresh or separate Edit Form.
> Please refer to following document for more details about this enhancement: http://docs.ofbiz.org/display/OFBIZ/Enhancing+Display+Widget+to+use+Ajax.InPlaceEditor

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (OFBIZ-1859) Implement Ajax.InPlaceEditor for display field in Form Widget

Posted by "Jacques Le Roux (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/OFBIZ-1859?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12617162#action_12617162 ] 

Jacques Le Roux commented on OFBIZ-1859:
----------------------------------------

Last patch in trunk rev. 679980  


> Implement Ajax.InPlaceEditor for display field in Form Widget
> -------------------------------------------------------------
>
>                 Key: OFBIZ-1859
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1859
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Mridul Pathak
>             Fix For: SVN trunk
>
>         Attachments: AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch
>
>
> Add ability to In Place Edit a <display> field in Form Widget. After this enhancement a developer can edit a single/multiple line in a display field of form widget using Ajax.InPlaceEditor without a page refresh or separate Edit Form.
> Please refer to following document for more details about this enhancement: http://docs.ofbiz.org/display/OFBIZ/Enhancing+Display+Widget+to+use+Ajax.InPlaceEditor

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (OFBIZ-1859) Implement Ajax.InPlaceEditor for display field in Form Widget

Posted by "Mridul Pathak (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/OFBIZ-1859?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12617159#action_12617159 ] 

Mridul Pathak commented on OFBIZ-1859:
--------------------------------------

Bruno,

    This has to do with Ajax.InPlaceEditor class.  Though I checked and found that it enters hover (highlighted) on "mouseover" event and leaves hover (highlight removed) on "mouseout" event.

> Implement Ajax.InPlaceEditor for display field in Form Widget
> -------------------------------------------------------------
>
>                 Key: OFBIZ-1859
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1859
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Mridul Pathak
>             Fix For: SVN trunk
>
>         Attachments: AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch
>
>
> Add ability to In Place Edit a <display> field in Form Widget. After this enhancement a developer can edit a single/multiple line in a display field of form widget using Ajax.InPlaceEditor without a page refresh or separate Edit Form.
> Please refer to following document for more details about this enhancement: http://docs.ofbiz.org/display/OFBIZ/Enhancing+Display+Widget+to+use+Ajax.InPlaceEditor

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (OFBIZ-1859) Implement Ajax.InPlaceEditor for display field in Form Widget

Posted by "Mridul Pathak (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/OFBIZ-1859?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mridul Pathak updated OFBIZ-1859:
---------------------------------

    Attachment: AjaxInPlaceEditor_Improvements.patch

Fixed the bug reported by Bruno Busco in reply to the commit 678997.  Thanks Bruno.

> Implement Ajax.InPlaceEditor for display field in Form Widget
> -------------------------------------------------------------
>
>                 Key: OFBIZ-1859
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1859
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Mridul Pathak
>             Fix For: SVN trunk
>
>         Attachments: AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch
>
>
> Add ability to In Place Edit a <display> field in Form Widget. After this enhancement a developer can edit a single/multiple line in a display field of form widget using Ajax.InPlaceEditor without a page refresh or separate Edit Form.
> Please refer to following document for more details about this enhancement: http://docs.ofbiz.org/display/OFBIZ/Enhancing+Display+Widget+to+use+Ajax.InPlaceEditor

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (OFBIZ-1859) Implement Ajax.InPlaceEditor for display field in Form Widget

Posted by "Jacques Le Roux (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/OFBIZ-1859?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12617160#action_12617160 ] 

Jacques Le Roux commented on OFBIZ-1859:
----------------------------------------

No worries Mridul,

Just keep up your good work ;o)

> Implement Ajax.InPlaceEditor for display field in Form Widget
> -------------------------------------------------------------
>
>                 Key: OFBIZ-1859
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1859
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Mridul Pathak
>             Fix For: SVN trunk
>
>         Attachments: AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch
>
>
> Add ability to In Place Edit a <display> field in Form Widget. After this enhancement a developer can edit a single/multiple line in a display field of form widget using Ajax.InPlaceEditor without a page refresh or separate Edit Form.
> Please refer to following document for more details about this enhancement: http://docs.ofbiz.org/display/OFBIZ/Enhancing+Display+Widget+to+use+Ajax.InPlaceEditor

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (OFBIZ-1859) Implement Ajax.InPlaceEditor for display field in Form Widget

Posted by "Mridul Pathak (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/OFBIZ-1859?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12617146#action_12617146 ] 

Mridul Pathak commented on OFBIZ-1859:
--------------------------------------

Hi jacques,

I'll try to reproduce this behavior and find any possible solution for that.

> Implement Ajax.InPlaceEditor for display field in Form Widget
> -------------------------------------------------------------
>
>                 Key: OFBIZ-1859
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1859
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Mridul Pathak
>             Fix For: SVN trunk
>
>         Attachments: AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch
>
>
> Add ability to In Place Edit a <display> field in Form Widget. After this enhancement a developer can edit a single/multiple line in a display field of form widget using Ajax.InPlaceEditor without a page refresh or separate Edit Form.
> Please refer to following document for more details about this enhancement: http://docs.ofbiz.org/display/OFBIZ/Enhancing+Display+Widget+to+use+Ajax.InPlaceEditor

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (OFBIZ-1859) Implement Ajax.InPlaceEditor for display field in Form Widget

Posted by "Mridul Pathak (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/OFBIZ-1859?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mridul Pathak updated OFBIZ-1859:
---------------------------------

    Attachment: AjaxInPlaceEditor_Improvements.patch

Uploading the updated patch for Improvements and correction.  Used proper widget-style  for display field.

> Implement Ajax.InPlaceEditor for display field in Form Widget
> -------------------------------------------------------------
>
>                 Key: OFBIZ-1859
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1859
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Mridul Pathak
>             Fix For: SVN trunk
>
>         Attachments: AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch, AjaxInPlaceEditor_Improvements.patch
>
>
> Add ability to In Place Edit a <display> field in Form Widget. After this enhancement a developer can edit a single/multiple line in a display field of form widget using Ajax.InPlaceEditor without a page refresh or separate Edit Form.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (OFBIZ-1859) Implement Ajax.InPlaceEditor for display field in Form Widget

Posted by "Anil K Patel (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/OFBIZ-1859?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12612614#action_12612614 ] 

Anil K Patel commented on OFBIZ-1859:
-------------------------------------

Mridul, thanks for your help with implementation. Your patch dated July 10 is in trunk rev# 675688. I am keeping this issue open for tracking further improvements to the implementation. 

> Implement Ajax.InPlaceEditor for display field in Form Widget
> -------------------------------------------------------------
>
>                 Key: OFBIZ-1859
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1859
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Mridul Pathak
>             Fix For: SVN trunk
>
>         Attachments: AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch
>
>
> Add ability to In Place Edit a <display> field in Form Widget. After this enhancement a developer can edit a single/multiple line in a display field of form widget using Ajax.InPlaceEditor without a page refresh or separate Edit Form.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Updated: (OFBIZ-1859) Implement Ajax.InPlaceEditor for display field in Form Widget

Posted by "Mridul Pathak (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/OFBIZ-1859?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mridul Pathak updated OFBIZ-1859:
---------------------------------

    Attachment: AjaxInPlaceEditor_Improvements.patch

Patch with some improvements and corrections.
Improvement:  Introduced a new option "updateAfterRequestCall" in Ajax.InPlaceEditor to sustain its original functionality when it call Ajax.Request (when htmlResponse = false).  Corresponding changes in ModelFormField.InPlaceEditor class and HtmlFormRenderer.renderDisplayField() method.

> Implement Ajax.InPlaceEditor for display field in Form Widget
> -------------------------------------------------------------
>
>                 Key: OFBIZ-1859
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1859
>             Project: OFBiz
>          Issue Type: New Feature
>          Components: framework
>    Affects Versions: SVN trunk
>            Reporter: Mridul Pathak
>             Fix For: SVN trunk
>
>         Attachments: AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor.patch, AjaxInPlaceEditor_Improvements.patch
>
>
> Add ability to In Place Edit a <display> field in Form Widget. After this enhancement a developer can edit a single/multiple line in a display field of form widget using Ajax.InPlaceEditor without a page refresh or separate Edit Form.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.