WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

#17690 closed defect (bug) (fixed)

DFW: Iframe wants to load wp-admin/undefined stylesheet

Reported by: ocean90 Owned by: azaozz
Milestone: 3.2 Priority: normal
Severity: normal Version: 3.2
Component: Editor Keywords: has-patch
Focuses: Cc:

Description

From log:

GET http://wpms.local/wp-admin/undefined 404 (Not Found)


Source:

<head xmlns="http://www.w3.org/1999/xhtml">
<meta http-equiv="X-UA-Compatible" content="IE=7">
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
<link type="text/css" rel="stylesheet" href="http://1.wpms.local/wp-includes/js/tinymce/themes/advanced/skins/wp_theme/content.css">
<link type="text/css" rel="stylesheet" href="http://1.wpms.local/wp-includes/js/tinymce/plugins/spellchecker/css/content.css">
<!-- See next line. -->
<link type="text/css" rel="stylesheet" href="http://1.wpms.local/wp-admin/undefined">
<link type="text/css" rel="stylesheet" href="http://1.wpms.local/wp-includes/js/tinymce/plugins/wpfullscreen/css/wp-fullscreen.css">
<link rel="stylesheet" data-mce-href="http://1.wpms.local/wp-includes/js/tinymce/plugins/wordpress/css/content.css?ver=342-20110603" href="http://1.wpms.local/wp-includes/js/tinymce/plugins/wordpress/css/content.css?ver=342-20110603">
</head>

Seems to be after r18137.

Attachments (1)

17690.diff (789 bytes) - added by duck_ 4 years ago.

Download all attachments as: .zip

Change History (4)

@duck_4 years ago

comment:1 @duck_4 years ago

  • Keywords has-patch added; needs-patch removed

This is only a problem when a theme without a registered editor-style.css file is active. It was also present pre-r18103.

Attached patch checks s.content_css before trying to use it.

Remember that the changes have to be applied to the minified editor_plugin.js for testing since the .dev.js version is never loaded.

comment:2 @ocean904 years ago

duck_ is right. Patch works.

comment:3 @azaozz4 years ago

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

In [18151]:

DFW: don't prepend comma if content_css is empty, props duck_, fixes #17690

Note: See TracTickets for help on using tickets.