Ticket #6811 (closed enhancement: fixed)
Unify TinyMCE image popup and insert image popup
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 2.6.1 |
| Component: | Administration | Version: | 2.5 |
| Severity: | normal | Keywords: | blessed |
| Cc: |
Description
Right now we have two methods for interacting with images, one bundled with TinyMCE and one from our "insert image" dialog.
Our insert image dialog has some better functionality around alignment and such, so we should standardize on that.
Attachments
Change History
Customized Edit Image dialog, first run. Has the functionality of both "Insert/Edit Image" and "Insert/Edit Link" popups in TinyMCE, together with several WordPress specific features (css classes, attachment rel, etc.). Also the look and feel is like the Add Media dialog.
Has an advanced tab that may need some options added or removed.
The zip file contains a TinyMCE plugin that goes to /wp-includes/js/tinymce/plugins/.
I think some strings in the plugin need to be setup for translation.
Replying to ryan:
I think some strings in the plugin need to be setup for translation.
Yes, the translation strings aren't in yet. Was wondering if this should be integrated with the "Add external image" too.
Second run. Added translation strings, RTL support, several layout improvements, image border, hspace and vspace can be used as shortcuts to add css styles, the preview is visible on both tabs, etc.
Also made a small change to ThickBox' style for the loading animation, now it's shown in a box similarly to the theme preview on http://wordpress.com.
The edit-image-popup2.patch includes a new file: wp-includes/js/tinymce/plugins/wpeditimage/css/editimage-rtl.css that didn't get created for some reason. Adding a zipped version of it.
comment:10
azaozz — 4 years ago
Fixes: align center in IE, import of color css stylesheets, potential error on submit when removing align center, few css fixes, etc. The image size setting is now on the side and is in percents of the current size.
comment:11
ryan — 4 years ago
comment:12
azaozz — 4 years ago
Several fixes: new link not created if the image has class "aligncenter" in Safari; recalculates the demo image size properly and applies any inline styles to it; if relative link is added, it's converted to absolute; "Unknown error" in IE6, although everything is working well.
comment:13
ryan — 4 years ago
comment:14
azaozz — 4 years ago
- Status changed from new to closed
- Resolution set to fixed
- Milestone changed from 2.9 to 2.6.1
Done.


See also: #6812.