[wp-trac] [WordPress Trac] #17144: Editor API enhancement
WordPress Trac
wp-trac at lists.automattic.com
Tue Dec 13 14:26:43 UTC 2011
#17144: Editor API enhancement
----------------------------+---------------------
Reporter: azaozz | Owner:
Type: task (blessed) | Status: closed
Priority: normal | Milestone: 3.3
Component: Editor | Version:
Severity: normal | Resolution: fixed
Keywords: needs-patch |
----------------------------+---------------------
Comment (by sushkov):
Regarding the hooks:
https://github.com/scholarpress/buddypress-
courseware/blob/master/courses/courses.class.php#L397 (To load enqueues)
https://github.com/scholarpress/buddypress-
courseware/blob/master/groups/templates/edit_course.php#L13 (To render the
editor, mind that now, `wp_tiny_mce()` gives a fatal error because it
looks like `deprecated.php` is not loaded to properly inside the files
that were containing it before)
What about DFW, the plugin uses hard-coded stuff like:
https://github.com/WordPress/WordPress/blob/master/wp-
includes/js/tinymce/plugins/wpfullscreen/editor_plugin.dev.js#L37
Where you can use something like:
`ed = (tinyMCE.editors.length > 0) ? tinyMCE.editors[0] :
tinymce.get('content');`
Another problem is even DFW is passed as `true` the proper CSS is not
loaded.
So back to my initial argument, I can't see the real benefit of
`wp_editor()` changes, because as an older developer of `wp-editor` API I
still have to hack around to get things done!
Another question is why would you decide to change code that was already
working?!
--
Ticket URL: <http://core.trac.wordpress.org/ticket/17144#comment:108>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list