#30949 closed enhancement (fixed)
Consider removing old DFW code form core
Reported by: | iseulde | Owned by: | iseulde |
---|---|---|---|
Milestone: | 4.3 | Priority: | normal |
Severity: | normal | Version: | 4.1 |
Component: | Editor | Keywords: | has-patch |
Focuses: | Cc: |
Description
Last cycle we decided to try this in 4.2. A plugin could copy the necessary things from 4.1 and hopefully improve it.
Attachments (1)
Change History (18)
This ticket was mentioned in Slack in #core by drew. View the logs.
10 years ago
This ticket was mentioned in Slack in #core by drew. View the logs.
10 years ago
This ticket was mentioned in Slack in #core by azaozz. View the logs.
10 years ago
#6
@
10 years ago
- Keywords has-patch added; needs-patch removed
- Milestone changed from Future Release to 4.3
In 30949.patch: remove the JS and CSS for wp-fullscreen
, remove the TinyMCE plugin, deprecate the PHP functions.
Moved the deprecated wp_ajax_wp_fullscreen_save_post()
to wp-admin/includes/deprecated.php, but maybe better to leave it in its current place in wp-admin/includes/ajax-actions.php?
#7
@
10 years ago
This looks awesome! :)
There seems to be a problem with the patch. I looks like a file is deleted twice.
The next patch would empty out the file src/wp-includes/js/tinymce/plugins/wpfullscreen/plugin.js, which is already empty! Assume -R? [n]
This ticket was mentioned in Slack in #core by obenland. View the logs.
10 years ago
This ticket was mentioned in Slack in #core-editor by azaozz. View the logs.
10 years ago
#11
@
10 years ago
Keeping this open for a bit longer in case there are bugs or we want to also deprecate wp_ajax_wp_fullscreen_save_post()
.
#14
@
9 years ago
@azaozz, any decision on deprecating wp_ajax_wp_fullscreen_save_post()
yet? Otherwise we could just close it and open bug tickets if any.
#15
@
9 years ago
- Resolution set to fixed
- Status changed from assigned to closed
Lets keep it for now. One of the items on the editor wish list is to save with AJAX. We will probably reuse this function when that is implemented.
Definitely too late for this in 4.2.