From 5c7bcd40463eb01234be5d5c95ba77472d41f907 Mon Sep 17 00:00:00 2001
From: Trevor Parscal <tparscal@users.mediawiki.org>
Date: Fri, 17 Jul 2009 19:09:44 +0000
Subject: [PATCH] Changed the simple search button from an input tag to a
 button tag to provide better styling support, especially in older versions of
 Safari.

---
 skins/Vector.php          | 2 +-
 skins/vector/main-ltr.css | 4 ++--
 skins/vector/main-rtl.css | 4 ++--
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/skins/Vector.php b/skins/Vector.php
index c49d95c1ff..58b3e48b09 100644
--- a/skins/Vector.php
+++ b/skins/Vector.php
@@ -779,7 +779,7 @@ class VectorTemplate extends QuickTemplate {
 		<?php if ( $wgVectorUseSimpleSearch ): ?>
 		<div id="simpleSearch">
 			<input id="searchInput" name="search" type="text" <?php echo $this->skin->tooltipAndAccesskey( 'search' ); ?> <?php if( isset( $this->data['search'] ) ): ?> value="<?php $this->text( 'search' ) ?>"<?php endif; ?> />
-			<input id="searchButton" type='submit' name='fulltext' value="&nbsp;" src="<?php echo $wgStylePath ?>/vector/images/search.png" <?php echo $this->skin->tooltipAndAccesskey( 'search-fulltext' ); ?> />
+			<button id="searchButton" type='submit' name='fulltext' <?php echo $this->skin->tooltipAndAccesskey( 'search-fulltext' ); ?>>&nbsp;</button>
 		</div>
 		<?php else: ?>
 		<input id="searchInput" name="search" type="text" <?php echo $this->skin->tooltipAndAccesskey( 'search' ); ?> <?php if( isset( $this->data['search'] ) ): ?> value="<?php $this->text( 'search' ) ?>"<?php endif; ?> />
diff --git a/skins/vector/main-ltr.css b/skins/vector/main-ltr.css
index 6cddd9f828..93e6d3df45 100644
--- a/skins/vector/main-ltr.css
+++ b/skins/vector/main-ltr.css
@@ -311,7 +311,7 @@ div#head h5 {
 		background-color: transparent;
 	}
 	/* OVERRIDDEN BY COMPLIANT BROWSERS */
-	div#p-search div#simpleSearch input#searchButton {
+	div#p-search div#simpleSearch button#searchButton {
 		position: absolute;
 		top: 0;
 		right: 0;
@@ -327,7 +327,7 @@ div#head h5 {
 		background-repeat: no-repeat;
 	}
 	/* IGNORED BY IE6 */
-	div#p-search div#simpleSearch > input#searchButton {
+	div#p-search div#simpleSearch > button#searchButton {
 		height: 100%;
 	}
 /* Panel */
diff --git a/skins/vector/main-rtl.css b/skins/vector/main-rtl.css
index 6ce13c2125..d6a027e890 100644
--- a/skins/vector/main-rtl.css
+++ b/skins/vector/main-rtl.css
@@ -311,7 +311,7 @@ div#head h5 {
 		background-color: transparent;
 	}
 	/* OVERRIDDEN BY COMPLIANT BROWSERS */
-	div#p-search div#simpleSearch input#searchButton {
+	div#p-search div#simpleSearch button#searchButton {
 		position: absolute;
 		top: 0;
 		left: 0;
@@ -327,7 +327,7 @@ div#head h5 {
 		background-repeat: no-repeat;
 	}
 	/* IGNORED BY IE6 */
-	div#p-search div#simpleSearch > input#searchButton {
+	div#p-search div#simpleSearch > button#searchButton {
 		height: 100%;
 	}
 /* Panel */
-- 
2.20.1