Make WordPress Core

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#22447 closed defect (bug) (fixed)

Update styling for the Edit Image popup in TinyMCE

Reported by: azaozz's profile azaozz Owned by: azaozz's profile azaozz
Milestone: 3.5 Priority: high
Severity: normal Version: 3.5
Component: TinyMCE Keywords:
Focuses: Cc:

Description

As we are using this in 3.5, the css for the iframe would need some updating.

Attachments (1)

22447.patch (6.9 KB) - added by azaozz 11 years ago.

Download all attachments as: .zip

Change History (5)

@azaozz
11 years ago

#1 @azaozz
11 years ago

In 22447.patch:

  • Load buttons.css in the popup's iframe.
  • Match fonts with the rest of the admin.
  • Add 2x for the align icons.

Also removes the "views" styling from the editor content css.

#2 @nacin
11 years ago

../../../../../../wp-admin/images/align-none-2x.png works, but isn't valid. We'll need to move those alignment icons into wp-includes to use it here.

koopersmith and I discussed converting this to be a template like everything else. Enables us to reuse existing styles and pieces of HTML to provide a better experience. But, only if we have time. Unfortunately we've approached the long tail of bugs, so this might not be the best use of our time.

#3 @azaozz
11 years ago

Yes, the (old) Edit Image modal is the only place that uses these icons in 3.5. They are in wp-admin/css because of the old Upload/Insert modal that will be removed. The ../../../../../../wp-admin/images/... may look "ugly" but is the only safe way to reference the icons from wp-includes/js/tinymce/plugins/wpeditimage/css/editimage.css.

Version 0, edited 11 years ago by azaozz (next)

#4 @azaozz
11 years ago

  • Owner set to azaozz
  • Resolution set to fixed
  • Status changed from new to closed

In 22586:

TinyMCE: fix styles, add 2x background icons and load buttons.css in the Edit Image modal, remove wp-view styles from content.css, add 2x icons for the popup buttons (for editing image and gallery), fixes #22447, see #21019

Note: See TracTickets for help on using tickets.