-
- 5,888 Posts
Would be glad, someone would find a solution for this issues with MIGX and tinyMCE.
Personally, I don't use tinyMCE in MIGX, if possible.
For text-formating I use textile-markup and the textile-output-filter.
For images or links in between text I use another MIGX-TV and add placeholders into the textarea for them, as done here:
http://rtfm.modx.com/display/ADDON/MIGX.Fancybox-images+with+seperate+placeholders+in+Richtext-Content
-
- 863 Posts
Thanks Bruno - I'm really going to have a good attempt at fixing this. Do you have any ideas where to start? Is it anything to do with inserting images in tinyMCE, for example?
-
- 5,888 Posts
for your issue:
can you test, if its fixed (not storing values), after adding this to components/migx/elements/tv/migx.tpl
this part:
if (typeof(Tiny) != 'undefined') {
tinyMCE.triggerSave();
}
Ext.extend(MODx.window.UpdateTvItem,Ext.Window,{
submit: function() {
if (typeof(Tiny) != 'undefined') {
tinyMCE.triggerSave();
}
var v = this.fp.getForm().getValues();
-
- 433 Posts
No problem, glad your problem is solved!
Maybe it's related to my problem, just to be sure I'll try the fix and see if it solves the disappearing top navigation bar problem as well.
-
- 5,888 Posts
I don't think this has something todo with your top-navigation-issue. (Never had seen this issue)
The fix only affects not saving the richtext-content in some situations.
-
- 863 Posts
...and without getting too far ahead of myself, the fix seems to have worked for several people using different browsers. Would definitely be worth including in your next update!
-
- 14 Posts
Hi guys,
thanks so much for that quick fix. In my case, it was working fine most of the time with Chrome (still had that problem where it randomly did not save the tinyMCE content), but was completely failing with some versions of Firefox. The quick fix made everything works fine, and definitely saved me days of investigation to fix it.
Is there a pull request for that ?
(and sorry michelle, hijacking your topic as well).
