From dbe35378e61bcae80ef4712c298648fa819f737a Mon Sep 17 00:00:00 2001
From: azaozz <azaozz@1a063a9b-81f0-0310-95a4-ce76da25c4cd>
Date: Mon, 8 Sep 2008 06:01:33 +0000
Subject: [PATCH] CSS and HTML validation fixes, props Simek, fixes #7703

git-svn-id: http://svn.automattic.com/wordpress/trunk@8845 1a063a9b-81f0-0310-95a4-ce76da25c4cd
---
 wp-admin/inbox.php              |  6 +++---
 wp-admin/includes/dashboard.php | 28 +---------------------------
 wp-admin/includes/template.php  |  4 ++--
 wp-admin/plugin-editor.php      |  6 +++---
 wp-admin/theme-editor.php       |  2 +-
 wp-admin/user-edit.php          |  2 +-
 wp-admin/users.php              |  2 +-
 wp-admin/wp-admin.css           | 30 ++++++++++++++++++++++++++++++
 8 files changed, 42 insertions(+), 38 deletions(-)

diff --git a/wp-admin/inbox.php b/wp-admin/inbox.php
index 3ac7b6e0ed..9f068d21a7 100644
--- a/wp-admin/inbox.php
+++ b/wp-admin/inbox.php
@@ -23,7 +23,7 @@ endif;
 <form id="inbox-filter" action="" method="get">
 <h2><?php _e('Inbox'); ?></h2>
 <ul class="subsubsub">
-<li><a href="#" class="current"><?php _e('Messages') ?></a></li> | <li><a href="#"><?php echo sprintf(__('Archived') . ' (%s)', '42'); ?></a></li>
+<li><a href="#" class="current"><?php _e('Messages') ?></a> | </li><li><a href="#"><?php echo sprintf(__('Archived') . ' (%s)', '42'); ?></a></li>
 </ul>
 <div class="tablenav">
 <div class="alignleft">
@@ -39,7 +39,7 @@ endif;
 <table class="widefat">
 	<thead>
 	<tr>
-	<th scope="col" class="check-column"><input type="checkbox"/></th>
+	<th scope="col" class="check-column"><input type="checkbox" /></th>
 	<th scope="col"><?php _e('Message'); ?></th>
 	<th scope="col"><?php _e('Date'); ?></th>
 	<th scope="col"><?php _e('From'); ?></th>
@@ -50,7 +50,7 @@ endif;
 <?php $crazy_posts = array( '', 'some post', 'a post', 'my cool post' ); foreach ( wp_get_inbox_items() as $k => $item ) : // crazyhorse ?>
 	
 	<tr id="message-<?php echo $k; ?>">
-		<th scope="col" class="check-column"><input type="checkbox" name="messages[]" value="<?php echo $k; ?>" /></td>
+		<th scope="col" class="check-column"><input type="checkbox" name="messages[]" value="<?php echo $k; ?>" /></th>
 		<td><?php
 			if ( $item->href )
 				echo "<a href='$item->href' class='no-crazy'>";
diff --git a/wp-admin/includes/dashboard.php b/wp-admin/includes/dashboard.php
index 55ccdcbbad..a3f1e3c0ef 100644
--- a/wp-admin/includes/dashboard.php
+++ b/wp-admin/includes/dashboard.php
@@ -382,6 +382,7 @@ function wp_dashboard_quick_press( $sidebar_args ) {
 
 function wp_dashboard_quick_press_js() {
 ?>
+
 <script type="text/javascript">
 /* <![CDATA[ */
 var quickPressLoad = function($) {
@@ -429,33 +430,6 @@ function wp_dashboard_inbox( $sidebar_args ) {
 
 ?>
 
-	<style type="text/css">
-		#inbox-filter ul {
-			list-style: none;
-			margin: 0;
-			padding: 0;
-			position: relative;
-		}
-		#inbox-filter ul li {
-			position: relative;
-			padding-right: 20px;
-		}
-		#inbox-filter ul input.checkbox {
-			float: left;
-		}
-		#inbox-filter ul p {
-			float: left;
-			margin: 0;
-			margin-left: -40px;
-			width: 100%;
-			position: relative;
-			left: 50px;
-		}
-		#inbox-message {
-			margin:0 0 0 7.5em;
-			padding:5px;
-		}
-	</style>
 	<script type="text/javascript">
 		jQuery( function($) {
 			$('#inbox-filter').submit( function() { return false; } )
diff --git a/wp-admin/includes/template.php b/wp-admin/includes/template.php
index c617723b10..3661d582af 100644
--- a/wp-admin/includes/template.php
+++ b/wp-admin/includes/template.php
@@ -115,7 +115,7 @@ function link_cat_row( $category ) {
 
 	$name = ( $name_override ? $name_override : $category->name );
 	if ( current_user_can( 'manage_categories' ) ) {
-		$edit = "<a class='row-title' href='link-category.php?action=edit&amp;cat_ID=$category->term_id' title='" . attribute_escape(sprintf(__('Edit "%s"'), $category->name)) . "' class='edit'>$name</a>";
+		$edit = "<a class='row-title' href='link-category.php?action=edit&amp;cat_ID=$category->term_id' title='" . attribute_escape(sprintf(__('Edit "%s"'), $category->name)) . "'>$name</a>";
 		$default_cat_id = (int) get_option( 'default_link_category' );
 	} else {
 		$edit = $name;
@@ -302,7 +302,7 @@ function _tag_row( $tag, $class = '' ) {
 		$out .= '<tr id="tag-' . $tag->term_id . '"' . $class . '>';
 		$out .= '<th scope="row" class="check-column"> <input type="checkbox" name="delete_tags[]" value="' . $tag->term_id . '" /></th>';
 		$out .= '<td><strong><a class="row-title" href="edit-tags.php?action=edit&amp;tag_ID=' . $tag->term_id . '" title="' . attribute_escape(sprintf(__('Edit "%s"'), $name)) . '">' .
-			$name . '</a></td>';
+			$name . '</a></strong></td>';
 
 		$out .= "<td class='num'>$count</td>";
 		$out .= '</tr>';
diff --git a/wp-admin/plugin-editor.php b/wp-admin/plugin-editor.php
index 80bfcb06f0..856f065cc7 100644
--- a/wp-admin/plugin-editor.php
+++ b/wp-admin/plugin-editor.php
@@ -95,13 +95,13 @@ default:
 	<?php } ?>
 </div>
 <?php endif; ?>
- <div class="wrap">
+<div class="wrap">
 <div class="bordertitle">
 	<h2><?php _e('Plugin Editor'); ?></h2>
 </div>
 <div class="tablenav">
 <div class="alignleft">
-<big><strong><?php
+<big><?php
 	if ( is_plugin_active($file) ) {
 		if ( is_writeable($real_file) )
 			echo sprintf(__('Editing <strong>%s</strong> (active)'), $file);
@@ -113,7 +113,7 @@ default:
 		else
 			echo sprintf(__('Browsing <strong>%s</strong> (inactive)'), $file);
 	}
-	?></strong></big>
+	?></big>
 </div>
 <br class="clear" />
 </div>
diff --git a/wp-admin/theme-editor.php b/wp-admin/theme-editor.php
index 30e408e17f..3cd2d46d81 100644
--- a/wp-admin/theme-editor.php
+++ b/wp-admin/theme-editor.php
@@ -101,7 +101,7 @@ $desc_header = ( $description != $file_show ) ? "$description</strong> (%s)" : "
 <div class="wrap">
 <div class="bordertitle">
 	<h2><?php _e('Theme Editor'); ?></h2>
-	<form id="themeselector" name="theme" action="theme-editor.php" method="post">
+	<form id="themeselector" action="theme-editor.php" method="post">
 		<strong><label for="theme"><?php _e('Select theme to edit:'); ?> </label></strong>
 		<select name="theme" id="theme">
 <?php
diff --git a/wp-admin/user-edit.php b/wp-admin/user-edit.php
index b6da3fb441..ad5e2731a7 100644
--- a/wp-admin/user-edit.php
+++ b/wp-admin/user-edit.php
@@ -171,7 +171,7 @@ include ('admin-header.php');
 <div class="wrap" id="profile-page">
 <h2><?php $is_profile_page? _e('Your Profile and Personal Options') : _e('Edit User'); ?></h2>
 
-<form name="profile" id="your-profile" action="" method="post">
+<form id="your-profile" action="" method="post">
 <?php wp_nonce_field('update-user_' . $user_id) ?>
 <?php if ( $wp_http_referer ) : ?>
 	<input type="hidden" name="wp_http_referer" value="<?php echo clean_url($wp_http_referer); ?>" />
diff --git a/wp-admin/users.php b/wp-admin/users.php
index 4af7c77648..2f5ac5f0b0 100644
--- a/wp-admin/users.php
+++ b/wp-admin/users.php
@@ -318,7 +318,7 @@ unset($role_links);
 <option value="delete"><?php _e('Delete'); ?></option>
 </select>
 <input type="submit" value="<?php _e('Apply'); ?>" name="doaction" class="button-secondary action" />
-<label class="hidden" for="new_role"><?php _e('Change role to&hellip;') ?></label><select name="new_role" id="new_role"><option value=''><?php _e('Change role to&hellip;') ?></option>"<?php wp_dropdown_roles(); ?></select>
+<label class="hidden" for="new_role"><?php _e('Change role to&hellip;') ?></label><select name="new_role" id="new_role"><option value=''><?php _e('Change role to&hellip;') ?></option><?php wp_dropdown_roles(); ?></select>
 <input type="submit" value="<?php _e('Change'); ?>" name="changeit" class="button-secondary" />
 <?php wp_nonce_field('bulk-users'); ?>
 </div>
diff --git a/wp-admin/wp-admin.css b/wp-admin/wp-admin.css
index 7f2a8f86a2..34fc53c7a2 100644
--- a/wp-admin/wp-admin.css
+++ b/wp-admin/wp-admin.css
@@ -1859,6 +1859,36 @@ p#post-search-prep {
 	padding: 2px;
 }
 
+#inbox-filter ul {
+	list-style: none;
+	margin: 0;
+	padding: 0;
+	position: relative;
+}
+
+#inbox-filter ul li {
+	position: relative;
+}
+
+#inbox-filter ul input.checkbox {
+	float: left;
+}
+
+#inbox-filter ul p {
+	float: left;
+	margin: 0;
+	margin-left: -40px;
+	width: 100%;
+	position: relative;
+	left: 50px;
+}
+
+#inbox-message {
+	margin: 0 0 0 7.5em;
+	padding: 5px;
+}
+
+
 /* media popup 0819 */
 #sidemenu {
 	margin: -30px 15px 0 315px;