Make WordPress Core

Opened 17 years ago

Closed 17 years ago

#6811 closed enhancement (fixed)

Unify TinyMCE image popup and insert image popup

Reported by: matt's profile matt Owned by: azaozz's profile azaozz
Milestone: 2.6.1 Priority: normal
Severity: normal Version: 2.5
Component: Administration Keywords: blessed
Focuses: 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 (6)

wpeditimage.zip (14.6 KB) - added by azaozz 17 years ago.
edit-image-popup.patch (6.3 KB) - added by azaozz 17 years ago.
edit-image-popup2.patch (32.4 KB) - added by azaozz 17 years ago.
editimage-rtl.zip (455 bytes) - added by azaozz 17 years ago.
edit-image-popup3.patch (25.2 KB) - added by azaozz 17 years ago.
edit-image-popup4.patch (9.2 KB) - added by azaozz 17 years ago.

Download all attachments as: .zip

Change History (20)

#1 @matt
17 years ago

See also: #6812.

@azaozz
17 years ago

#2 @azaozz
17 years ago

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/.

#3 @ryan
17 years ago

(In [8066]) Unify TinyMCE image popup and insert image popup. Props azaozz. see #6811

#4 follow-up: @ryan
17 years ago

I think some strings in the plugin need to be setup for translation.

#5 in reply to: ↑ 4 @azaozz
17 years ago

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.

#6 @azaozz
17 years ago

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.

#7 @ryan
17 years ago

(In [8083]) Image popup improvements from azaozz. 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. see #6811

#8 @azaozz
17 years ago

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.

@azaozz
17 years ago

#10 @azaozz
17 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.

#11 @ryan
17 years ago

(In [8167]) Edit image updates from azaozz. see #6811

#12 @azaozz
17 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.

#13 @ryan
17 years ago

(In [8191]) Edit image fixes from azaozz: : 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. see #6811

#14 @azaozz
17 years ago

  • Milestone changed from 2.9 to 2.6.1
  • Resolution set to fixed
  • Status changed from new to closed

Done.

Note: See TracTickets for help on using tickets.