Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JSON iframe #4

Closed
jtousek opened this issue Dec 16, 2009 · 7 comments
Closed

JSON iframe #4

jtousek opened this issue Dec 16, 2009 · 7 comments

Comments

@jtousek
Copy link

jtousek commented Dec 16, 2009

When I have some HTML in the JSON object, I get &lt; instead of < and &gt; instead of > and so on.
This modification fixed the problem.:

Line 320:
xhr.responseText = pre.textContent;

There is innerHTML instead of textContent now. But that returns something like php func htmlspecialchars - chars like < and > are encoded to their entities which is bad.

@gucki
Copy link

gucki commented May 19, 2010

This bug still exists in version: 2.43 (12-MAR-2010). I found it while working rails 3.

The suggested fix works fine for me (firefox 3.6). Only the line changed from 320 to 327.

@malsup
Copy link
Collaborator

malsup commented Aug 9, 2010

What browser does this occur on? Can you create a simple test page for me? Thanks.

@djcas9
Copy link

djcas9 commented Sep 9, 2010

Same Issue - This is a problem on Max OSX, Windows - tested with all browsers.

@jtousek
Copy link
Author

jtousek commented Sep 9, 2010

I'm not sure it's the only problem, anyway I've used <textarea> tag and it works just fine. :)

@softlion
Copy link

+1

@ulmus
Copy link

ulmus commented Dec 3, 2010

I can attest both to the problem still existing (osx and win, all major browsers) and that the patch fixed the problem for me. Was losing my mind until I saw this bug report!

@malsup
Copy link
Collaborator

malsup commented Dec 3, 2010

Fixed in v2.50. Thanks for all the feedback.

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants