WordPress.org

Make WordPress Core

Opened 17 months ago

Closed 17 months ago

Last modified 17 months ago

#22447 closed defect (bug) (fixed)

Update styling for the Edit Image popup in TinyMCE

Reported by: azaozz Owned by: 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 17 months ago.

Download all attachments as: .zip

Change History (5)

azaozz17 months ago

comment:1 azaozz17 months 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.

comment:2 nacin17 months 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.

comment:3 azaozz17 months 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 17 months ago by azaozz (next)

comment:4 azaozz17 months 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.