From 6c29739113cb8c4dfb8cf77a9b6bb6d0112f28ca Mon Sep 17 00:00:00 2001
From: Platonides <platonides@users.mediawiki.org>
Date: Thu, 10 Feb 2011 20:36:18 +0000
Subject: [PATCH] Follow up r81914. Don't build the <li> manually.

---
 skins/MonoBook.php | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/skins/MonoBook.php b/skins/MonoBook.php
index 2711a1afc6..557abe6ca9 100644
--- a/skins/MonoBook.php
+++ b/skins/MonoBook.php
@@ -203,12 +203,6 @@ class MonoBookTemplate extends BaseTemplate {
 		<div class="pBody">
 			<ul><?php
 				foreach($this->data['content_actions'] as $key => $tab) {
-					echo '
-				 <li id="' . Sanitizer::escapeId( "ca-$key" ) . '"';
-					if( $tab['class'] ) {
-						echo ' class="'.htmlspecialchars($tab['class']).'"';
-					}
-					echo '>';
 					$linkAttribs = array( 'href' => $tab['href'] );
 					
 				 	if( isset( $tab["tooltiponly"] ) && $tab["tooltiponly"] ) {
@@ -219,8 +213,15 @@ class MonoBookTemplate extends BaseTemplate {
 				 	} else {
 						$linkAttribs += $skin->tooltipAndAccesskeyAttribs( "ca-$key" );
 				 	}
-				 	echo Html::element( 'a', $linkAttribs, $tab['text'] );
-				 	echo '</li>';
+				 	$linkHtml = Html::element( 'a', $linkAttribs, $tab['text'] );
+				 	
+				 	/* Surround with a <li> */
+				 	$liAttribs = array( 'id' => Sanitizer::escapeId( "ca-$key" ) );
+					if( $tab['class'] ) {
+						$liAttribs['class'] = $tab['class'];
+					}
+				 	echo '
+				' . Html::rawElement( 'li', $liAttribs, $linkHtml );
 				} ?>
 
 			</ul>
-- 
2.20.1