AnanthaKN / jquery-in-place-editor

Automatically exported from code.google.com/p/jquery-in-place-editor
Other
0 stars 0 forks source link

editInPlace field does not work within a form element #72

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. The editInPlace field is within a form element
2. Onclick of the editInPlace field will remove the text
3. It's not longer possible to use the editInPlace field

What is the expected output? What do you see instead?
Input field, but the current text will be removed

What version of the product are you using? On what operating system?
FireFox 3.6.10 + jQuery 1.4.2 + editinplace2.x

Please provide any additional information below.
Solution for current trunk version:
on line 258
-- this.dom.html('<form class="inplace_form" style="display: inline; margin: 0; 
padding: 0;"></form>')
++ this.dom.html('').append('<form class="inplace_form" style="display: inline; 
margin: 0; padding: 0;"></form>')

Original issue reported on code.google.com by stefanga...@gmail.com on 28 Sep 2010 at 10:18

Attachments:

GoogleCodeExporter commented 8 years ago
Confirmed this is an issue when using ASP.net.  The clicked text is removed, 
but the form and new elements are not added to the document.

Sample of error can be found here :

http://jsfiddle.net/coreydownie/4VCaZ/1/

Original comment by corey.do...@gmail.com on 27 Jan 2011 at 5:02

GoogleCodeExporter commented 8 years ago
I'm not quite sure what to make of this - the demo you provided seems to do 
what it should? Can you please elaborate?

Original comment by mhaec...@gmail.com on 27 Jan 2011 at 8:50

GoogleCodeExporter commented 8 years ago
My mistake, I linked to the incorrect revision

http://jsfiddle.net/coreydownie/4VCaZ/

Worth noting, the fix provided above resolved the issue for me.

Original comment by corey.do...@gmail.com on 27 Jan 2011 at 8:55

GoogleCodeExporter commented 8 years ago
Just thought I'd mention IE8 and FireFox4 or Safari 5.0.4 do not exhibit the 
problem, but Chrome 10.0.648.204 does (not sure on previous versions). Also as 
noted, stefanga's fix above fixes the issue in Chrome.

Original comment by jmina...@gmail.com on 1 Apr 2011 at 3:35

GoogleCodeExporter commented 8 years ago
A "little" old issue, but I'm using the current version (2.2.1) and the problem 
is still there.

I've just used the code fix above and now works!

Original comment by romulom...@gmail.com on 18 Oct 2011 at 7:16

GoogleCodeExporter commented 8 years ago
That fix worked for me as well. However you came up with that... thank you.

Original comment by marc.st...@gmail.com on 15 Feb 2012 at 1:30

GoogleCodeExporter commented 8 years ago
The fix also worked for me....Thank you.

Original comment by anitabha...@gmail.com on 8 May 2012 at 1:24

GoogleCodeExporter commented 8 years ago
Had the same problem, guys, please apply patch above, people are suffering.

Original comment by kuri...@gmail.com on 21 Aug 2012 at 6:10

GoogleCodeExporter commented 8 years ago
This bug still exists. I was able to reproduce it with the latest version of 
Chromium. 

Version 25.0.1364.160 Ubuntu 12.04 (25.0.1364.160-0ubuntu0.12.04.1)

Used the following code (around line 619):

    this.dom.html('').append('<form class="inplace_form" style="display: inline; margin: 0; padding: 0;"></form>')

Problem was resolved.

Original comment by dave.jar...@gmail.com on 10 Apr 2013 at 4:07

GoogleCodeExporter commented 8 years ago
Thanks , now it is working in Chrome

Original comment by anu.mish...@gmail.com on 29 Aug 2013 at 9:28

GoogleCodeExporter commented 8 years ago
Thank you, the diff above worked correctly across all browsers for my 
application (current Chrome, FF, & IE). The line numbers were different for me, 
but they were close enough (253).

Original comment by aa...@espcredit.no on 11 Sep 2014 at 8:41