<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
"http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head><meta http-equiv="content-type" content="text/html; charset=utf-8" /><style type="text/css"><!--
#msg dl { border: 1px #006 solid; background: #369; padding: 6px; color: #fff; }
#msg dt { float: left; width: 6em; font-weight: bold; }
#msg dt:after { content:':';}
#msg dl, #msg dt, #msg ul, #msg li, #header, #footer { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; }
#msg dl a { font-weight: bold}
#msg dl a:link { color:#fc3; }
#msg dl a:active { color:#ff0; }
#msg dl a:visited { color:#cc6; }
h3 { font-family: verdana,arial,helvetica,sans-serif; font-size: 10pt; font-weight: bold; }
#msg pre { overflow: auto; background: #ffc; border: 1px #fc0 solid; padding: 6px; }
#msg ul, pre { overflow: auto; }
#header, #footer { color: #fff; background: #636; border: 1px #300 solid; padding: 6px; }
#patch { width: 100%; }
#patch h4 {font-family: verdana,arial,helvetica,sans-serif;font-size:10pt;padding:8px;background:#369;color:#fff;margin:0;}
#patch .propset h4, #patch .binary h4 {margin:0;}
#patch pre {padding:0;line-height:1.2em;margin:0;}
#patch .diff {width:100%;background:#eee;padding: 0 0 10px 0;overflow:auto;}
#patch .propset .diff, #patch .binary .diff {padding:10px 0;}
#patch span {display:block;padding:0 10px;}
#patch .modfile, #patch .addfile, #patch .delfile, #patch .propset, #patch .binary, #patch .copfile {border:1px solid #ccc;margin:10px 0;}
#patch ins {background:#dfd;text-decoration:none;display:block;padding:0 10px;}
#patch del {background:#fdd;text-decoration:none;display:block;padding:0 10px;}
#patch .lines, .info {color:#888;background:#fff;}
--></style>
<title>[12541] trunk/wp-admin: Some style tweaks to theme editor, fixes #11563.</title>
</head>
<body>
<div id="msg">
<dl>
<dt>Revision</dt> <dd><a href="http://trac.wordpress.org/changeset/12541">12541</a></dd>
<dt>Author</dt> <dd>matt</dd>
<dt>Date</dt> <dd>2009-12-25 03:16:25 +0000 (Fri, 25 Dec 2009)</dd>
</dl>
<h3>Log Message</h3>
<pre>Some style tweaks to theme editor, fixes <a href="http://trac.wordpress.org/ticket/11563">#11563</a>. Hat tip: Simek.
I adjusted the padding to be a little heavier and then have a negative margin, so the text stays aligned and the highlight floats around it. As an added tweak I removed the extraneous "Theme Files" heading which didn't really convey any useful information.
"Perfection is achieved perfection not when there is nothing left to add, but when there is nothing left to take away" -- Antoine de Saint-Exupery (and Merry Christmas everybody. :))</pre>
<h3>Modified Paths</h3>
<ul>
<li><a href="#trunkwpadmincssthemeeditorcss">trunk/wp-admin/css/theme-editor.css</a></li>
<li><a href="#trunkwpadmincssthemeeditordevcss">trunk/wp-admin/css/theme-editor.dev.css</a></li>
<li><a href="#trunkwpadminthemeeditorphp">trunk/wp-admin/theme-editor.php</a></li>
</ul>
</div>
<div id="patch">
<h3>Diff</h3>
<a id="trunkwpadmincssthemeeditorcss"></a>
<div class="modfile"><h4>Modified: trunk/wp-admin/css/theme-editor.css (12540 => 12541)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/wp-admin/css/theme-editor.css        2009-12-25 02:46:40 UTC (rev 12540)
+++ trunk/wp-admin/css/theme-editor.css        2009-12-25 03:16:25 UTC (rev 12541)
</span><span class="lines">@@ -1 +1 @@
</span><del>-#template textarea{font-family:Consolas,Monaco,Courier,monospace;font-size:12px;width:97%;}#template p{width:97%;}#templateside{float:right;width:190px;word-wrap:break-word;}#templateside h3,#postcustomstuff p.submit{margin:0;}#templateside h4{margin:1em 0 0;}#templateside ol,#templateside ul{margin:.5em;padding:0;}#templateside li{margin:4px 0;}.nonessential{font-size:small;}.highlight{padding:1px;}div.tablenav{margin-right:210px;}#documentation{margin-top:10px;}#documentation label{line-height:22px;vertical-align:top;font-weight:bold;}.fileedit-sub{padding:10px 0 8px;line-height:180%;}
</del><span class="cx">\ No newline at end of file
</span><ins>+#template textarea{font-family:Consolas,Monaco,Courier,monospace;font-size:12px;width:97%;}#template p{width:97%;}#templateside{float:right;width:190px;word-wrap:break-word;}#templateside h3,#postcustomstuff p.submit{margin:0;}#templateside h4{margin:1em 0 0;}#templateside ol,#templateside ul{margin:.5em;padding:0;}#templateside li{margin:4px 0;}.nonessential{font-size:11px;font-style:italic;padding-left:12px;}.highlight{padding:3px 3px 3px 12px;margin-left:-12px;font-weight:bold;-moz-border-radius:8px;-khtml-border-radius:8px;-webkit-border-radius:8px;border-radius:8px;}div.tablenav{margin-right:210px;}#documentation{margin-top:10px;}#documentation label{line-height:22px;vertical-align:top;font-weight:bold;}.fileedit-sub{padding:10px 0 8px;line-height:180%;}
</ins><span class="cx">\ No newline at end of file
</span></span></pre></div>
<a id="trunkwpadmincssthemeeditordevcss"></a>
<div class="modfile"><h4>Modified: trunk/wp-admin/css/theme-editor.dev.css (12540 => 12541)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/wp-admin/css/theme-editor.dev.css        2009-12-25 02:46:40 UTC (rev 12540)
+++ trunk/wp-admin/css/theme-editor.dev.css        2009-12-25 03:16:25 UTC (rev 12541)
</span><span class="lines">@@ -34,11 +34,19 @@
</span><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> .nonessential {
</span><del>-        font-size: small;
</del><ins>+        font-size: 11px;
+        font-style: italic;
+        padding-left: 12px;
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> .highlight {
</span><del>-        padding: 1px;
</del><ins>+        padding: 3px 3px 3px 12px;
+        margin-left: -12px;
+        font-weight: bold;
+        -moz-border-radius: 8px;
+        -khtml-border-radius: 8px;
+        -webkit-border-radius: 8px;
+        border-radius: 8px;
</ins><span class="cx"> }
</span><span class="cx">
</span><span class="cx"> div.tablenav {
</span></span></pre></div>
<a id="trunkwpadminthemeeditorphp"></a>
<div class="modfile"><h4>Modified: trunk/wp-admin/theme-editor.php (12540 => 12541)</h4>
<pre class="diff"><span>
<span class="info">--- trunk/wp-admin/theme-editor.php        2009-12-25 02:46:40 UTC (rev 12540)
+++ trunk/wp-admin/theme-editor.php        2009-12-25 03:16:25 UTC (rev 12541)
</span><span class="lines">@@ -142,12 +142,11 @@
</span><span class="cx"> </div>
</span><span class="cx"> <br class="clear" />
</span><span class="cx">         <div id="templateside">
</span><del>-        <h3><?php _e("Theme Files"); ?></h3>
</del><span class="cx">
</span><span class="cx"> <?php
</span><span class="cx"> if ($allowed_files) :
</span><span class="cx"> ?>
</span><del>-        <h4><?php _e('Templates'); ?></h4>
</del><ins>+        <h3><?php _e('Templates'); ?></h3>
</ins><span class="cx">         <ul>
</span><span class="cx"> <?php
</span><span class="cx">         $template_mapping = array();
</span><span class="lines">@@ -155,8 +154,8 @@
</span><span class="cx">         foreach ( $themes[$theme]['Template Files'] as $template_file ) {
</span><span class="cx">                 $description = trim( get_file_description($template_file) );
</span><span class="cx">                 $template_show = basename($template_file);
</span><del>-                $filedesc = ( $description != $template_file ) ? "$description <span class='nonessential'>($template_show)</span>" : $description;
-                $filedesc = ( $template_file == $file ) ? "<span class='highlight'>$description <span class='nonessential'>($template_show)</span></span>" : $filedesc;
</del><ins>+                $filedesc = ( $description != $template_file ) ? "$description<br /><span class='nonessential'>($template_show)</span>" : "$description";
+                $filedesc = ( $template_file == $file ) ? "<div class='highlight'>$description<br /><span class='nonessential'>($template_show)</span></div>" : $filedesc;
</ins><span class="cx">
</span><span class="cx">                 // If we have two files of the same name prefer the one in the Template Directory
</span><span class="cx">                 // This means that we display the correct files for child themes which overload Templates as well as Styles
</span><span class="lines">@@ -174,7 +173,7 @@
</span><span class="cx">                 <li><a href="theme-editor.php?file=<?php echo "$template_file"; ?>&amp;theme=<?php echo urlencode($theme) ?>&amp;dir=theme"><?php echo $filedesc ?></a></li>
</span><span class="cx"> <?php endwhile; ?>
</span><span class="cx">         </ul>
</span><del>-        <h4><?php /* translators: Theme stylesheets in theme editor */ echo _x('Styles', 'Theme stylesheets in theme editor'); ?></h4>
</del><ins>+        <h3><?php /* translators: Theme stylesheets in theme editor */ echo _x('Styles', 'Theme stylesheets in theme editor'); ?></h3>
</ins><span class="cx">         <ul>
</span><span class="cx"> <?php
</span><span class="cx">         $template_mapping = array();
</span><span class="lines">@@ -182,8 +181,8 @@
</span><span class="cx">         foreach ( $themes[$theme]['Stylesheet Files'] as $style_file ) {
</span><span class="cx">                 $description = trim( get_file_description($style_file) );
</span><span class="cx">                 $style_show = basename($style_file);
</span><del>-                $filedesc = ( $description != $style_file ) ? "$description <span class='nonessential'>($style_show)</span>" : "$description";
-                $filedesc = ( $style_file == $file ) ? "<span class='highlight'>$description <span class='nonessential'>($style_show)</span></span>" : $filedesc;
</del><ins>+                $filedesc = ( $description != $style_file ) ? "$description<br /><span class='nonessential'>($style_show)</span>" : "$description";
+                $filedesc = ( $style_file == $file ) ? "<div class='highlight'>$description<br /><span class='nonessential'>($style_show)</span></div>" : $filedesc;
</ins><span class="cx">                 $template_mapping[ $description ] = array( _get_template_edit_filename($style_file, $stylesheet_dir), $filedesc );
</span><span class="cx">         }
</span><span class="cx">         ksort( $template_mapping );
</span></span></pre>
</div>
</div>
</body>
</html>