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

tinymce 4.0b1 fails on pages served as xhtml #614

Closed
TinyMCE-User opened this issue Apr 22, 2013 · 2 comments
Closed

tinymce 4.0b1 fails on pages served as xhtml #614

TinyMCE-User opened this issue Apr 22, 2013 · 2 comments

Comments

@TinyMCE-User
Copy link

Description of problem:

Latest tinymce beta does not work when serving pages as application/xhtml+xml

Steps to reproduce:
1.Serve page with content-type: application/xhtml+xml
2.init tinymce with the same parameters as the basic example in javascript
3.fails
4.Serve page with content-type: text/html and repeat
5.works

Expected result: tinymce editor

Actual result: Blank area where editor should be.

Legacy information imported from TinyMCE bug tracker:

#T5824 posted by ramsey

Tags: [4x]
Status: Open
Resolution: None
Attached URL: none

@TinyMCE-User
Copy link
Author

Same issue with 4.0.20. The issue is that the HTML produced for the toolbar is not XML compliant, and therefore fails (on line 6137) when the innerHTML method is called.

I forked the repo in GitHub but it did not have the TinyMCE 4 branch in it for some reason. Anyway the fix was fairly simple. Replace all attributes that specify hidefocus without a value with hidefocus="1", and there is also a " " on line 31240 that needs removing.

Like I say fairly simple fix, I am happy to create the path properly if someone can help, or I can test if someone else wants to submit the patch.

Posted by hexydec

@TinyMCE-User
Copy link
Author

Seems to be fixed in 4.0.21. Thanks.

Posted by hexydec

@spocke spocke removed none labels Oct 29, 2015
@fyrkant fyrkant closed this as completed Jan 30, 2017
spocke added a commit that referenced this issue May 8, 2018
* commit '6d0967d836cf9da62e494c2b2a0c918aa3aed0ab':
  TINY-1649: added support for preserving whitespace in pre inline editors
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

3 participants