[wp-trac] [WordPress Trac] #17144: Editor API enhancement
WordPress Trac
wp-trac at lists.automattic.com
Wed Aug 3 18:42:49 UTC 2011
#17144: Editor API enhancement
-------------------------+------------------
Reporter: azaozz | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: 3.3
Component: Editor | Version:
Severity: normal | Resolution:
Keywords: |
-------------------------+------------------
Comment (by azaozz):
Replying to [comment:25 nacin]:
> The first editor could retain the existing ID. Subsequent ones can be
new IDs.
The only changed IDs are on the switching buttons, the rest are new. Of
course the quicktags toolbar ID is changed too but that's in #16695.
> The point is simple: Everyone uses it. It should stay and be a wrapper.
You mean don't deprecate the_editor()? That would make two wrappers for
the same thing :)
The arguments have changed (not in a breaking way), thinking by
deprecating it we will draw attention to that.
> There's no need for a lot of those functions to become methods... All of
these functions should still exist on their own.
Added them in the class as it's only loaded (lazily) when needed and these
functions are only used there. Can probably move wp_link_query() directly
to admin-ajax.php as it can only be usefull from there. Don't see a point
for wp_fullscreen_html() and wp_link_dialog() to be separate functions.
Furthermore they will be loaded on each run of WP (on the front end) which
is undesirable.
Actually we have quite a bit of code in wp-includes that is loaded on each
run of WP but used once in a while (1 in 10,000?). Even some functions
that cannot be run unless wp-admin is loaded. All this eats some resources
which multiplied by many millions amount to a lot :)
--
Ticket URL: <http://core.trac.wordpress.org/ticket/17144#comment:27>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list