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

events made easy and tinymce #152

Closed
herrvigg opened this issue Jun 22, 2018 · 5 comments
Closed

events made easy and tinymce #152

herrvigg opened this issue Jun 22, 2018 · 5 comments
Labels
bug Something isn't working, reproducible legacy issue Legacy issue imported from original repo

Comments

@herrvigg
Copy link
Collaborator

Issue by liedekef
Tuesday May 05, 2015 at 22:52 GMT
Originally opened as qTranslate-Team/qtranslate-x#152


Hi,

the plugin is working fine with the combo qtranslate-x and EME, but there seems to be an issue with tinymce:
https://wordpress.org/support/topic/bug-when-switch-languages-on-visual-editor
I tried it on my local install and it seems to be the case for me too. Although in the generated html output I see some qtranxs-translatable class being added to the content div, the language switch only works for the text-mode. Maybe the sequence of loading the javascript is wrong (just a thought)?

Franky

@herrvigg herrvigg added bug Something isn't working, reproducible legacy issue Legacy issue imported from original repo labels Jun 22, 2018
@herrvigg
Copy link
Collaborator Author

Comment by johnclause
Thursday May 07, 2015 at 00:13 GMT


@liedekef Franky: this is indeed strange, it all looks very similar to standard post page, but somehow it does not work. Can you think of a difference why event page could behave differently? Do you do anything non-wp-standard to load TinyMCE? I believe, if everything is done in the same way as on a standard post page '/wp-admin/post.php', it will work.

@herrvigg
Copy link
Collaborator Author

Comment by liedekef
Thursday May 07, 2015 at 10:14 GMT


I haven't changed the code to add tinymce in quite a while. I use the standard wp call wp_editor in eme_events.php:

wp_editor($event['event_notes'],"content");

@herrvigg
Copy link
Collaborator Author

Comment by johnclause
Saturday May 16, 2015 at 17:54 GMT


Hi Franky, would you have time to try the latest beta https://github.com/qTranslate-Team/qtranslate-x/releases/tag/3.3.2 ? I think this issue has disappeared, could you confirm?

@herrvigg
Copy link
Collaborator Author

Comment by liedekef
Saturday May 16, 2015 at 23:00 GMT


That version seems to be working. May I ask what it is that is special for the EME tinymce editor?

@herrvigg
Copy link
Collaborator Author

Comment by johnclause
Sunday May 17, 2015 at 02:47 GMT


I did not specifically worked on this issue, decided to take a look after the last round of changes to implement Integration Framework, and when I tried eme again, it worked. It is kind of hard to trace back now what was wrong, and maybe it is not important at this point, the only thing that comes in mind is that I moved line "qtx.addContentHooksTinyMCE();" to be executed always and not only on post.php. Ah, that is it, yes, you use admin.php to edit events, so that line was not fired for you before, which means it never worked, we missed it ;) Yes, normally, plugins would use post.php for any kind of posts, and "event" is one of them, but I guess you implemented your own framework.

I am closing it now, we can still write to a closed issue or may reopen it if needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working, reproducible legacy issue Legacy issue imported from original repo
Projects
None yet
Development

No branches or pull requests

1 participant