Changeset 21877 for trunk/wp-admin/menu-header.php

Timestamp:

09/16/2012 10:51:44 PM (13 years ago)

Author:

azaozz

Message:

Admin menu: add phpdoc/inline comments describing the usage of the param $icon_url in add_menu_page(), add string 'none' to be used when a menu icon is added from CSS, switch core items in the admin menu to use 'none', fixes #21349

File:

• trunk/wp-admin/menu-header.php (modified) (1 diff)

trunk/wp-admin/menu-header.php

@@ -69 +69 @@
         $id = ! empty( $item[5] ) ? ' id="' . preg_replace( '|[^a-zA-Z0-9_:.]|', '-', $item[5] ) . '"' : '';
         $img = '';
+        // if the string 'none' (previously 'div') is passed instead of an URL, don't output the default menu image
+        // so an icon can be added to div.wp-menu-image as background with CSS.
         if ( ! empty( $item[6] ) )
-            $img = ( 'div' === $item[6] ) ? '<br />' : '<img src="' . $item[6] . '" alt="" />';
+            $img = ( 'none' === $item[6] || 'div' === $item[6] ) ? '<br />' : '<img src="' . $item[6] . '" alt="" />';
         $arrow = '<div class="wp-menu-arrow"><div></div></div>';