tweaks to page history

diff --git a/includes/PageHistory.php b/includes/PageHistory.php
index e95ad39..27ff210 100644 (file)
--- a/includes/PageHistory.php
+++ b/includes/PageHistory.php
@@ -77,9 +77,9 @@ class PageHistory {
                        $this->mTitle->getPrefixedText(),
                        "action=history", $atend );
                $s = $numbar;
-               $this->submitbuttonhtml = ( $this->linesonpage > 1) ? '<li class="histsubmitli">
-               <input class="historysubmit" type="submit" accesskey="'.wfMsg('accesskey-compareselectedversions').
-                       '" title="'.wfMsg('tooltip-compareselectedversions').'" value="'.wfMsg('compareselectedversions')."\" /></li>" :
+               $this->submitbuttonhtml = ( $this->linesonpage > 1) ? '<input class="historysubmit" 
+               type="submit" accesskey="'.wfMsg('accesskey-compareselectedversions').
+                       '" title="'.wfMsg('tooltip-compareselectedversions').'" value="'.wfMsg('compareselectedversions')."\" />" :
                        '';
                $s .= $this->beginHistoryList();
                $counter = 1;
@@ -114,10 +114,10 @@ class PageHistory {
                global $wgTitle;
                $this->lastdate = $this->lastline = "";
                $s = "\n<p>" . wfMsg( "histlegend" ).'</p>'; 
-               $s .="\n<form id=\"pagehistory\" name=\"pagehistory\" action=\"" . $wgTitle->getFullURL("-") . "\" method=\"get\">";
+               $s .="\n<form action=\"" . $wgTitle->getFullURL("-") . "\" method=\"get\">";
                $s .= "<input type=\"hidden\" name=\"title\" value=\"".htmlspecialchars($wgTitle->getPrefixedDbKey())."\"/>\n";
-               $s .= "" . "\n<ul>";
                $s .= !empty($this->submitbuttonhtml) ? $this->submitbuttonhtml."\n":'';
+               $s .= "" . "\n<ul id=\"pagehistory\" >";
                return $s;
        }
 
@@ -126,8 +126,9 @@ class PageHistory {
                $last = wfMsg( "last" );
 
                $s = $skip ? "" : preg_replace( "/!OLDID![0-9]+!/", $last, $this->lastline );
+               $s .= "</ul>";
                $s .= !empty($this->submitbuttonhtml) ? $this->submitbuttonhtml."\n":'';
-               $s .= "</ul></form>\n";
+               $s .= "</form>\n";
                return $s;
        }
 

diff --git a/stylesheets/common.css b/stylesheets/common.css
index 7061340..712e4fe 100644 (file)
--- a/stylesheets/common.css
+++ b/stylesheets/common.css
@@ -77,7 +77,6 @@ div.tleft {
     background-color:#f9f9f9;
     border:1px dashed #aaaaaa;
 }
-#pagehistory li.histsubmitli { list-style: none }
 
 table.diff {
         background:white;

diff --git a/stylesheets/monobook/main.css b/stylesheets/monobook/main.css
index 4247920..f1bcf9f 100644 (file)
--- a/stylesheets/monobook/main.css
+++ b/stylesheets/monobook/main.css
@@ -157,7 +157,6 @@ textarea {
 /* hide this from ie/mac and konq2.2 */
 @media All {
     head:first-child+body input {
-        font-family: "Bitstream Vera Sans", "Lucida Grande", Verdana, Lucida, Helvetica, Arial, sans-serif;
         visibility: visible;
         border: 1px solid #2f6fab;  
         color: Black;
@@ -165,11 +164,13 @@ textarea {
         vertical-align: middle;
         padding: 0.2em;
     }
-    head:first-child+body input.historysubmit {
-        height: 1.9em !important;
-        padding: none !important;
-        vertical-align: middle;
-    }
+}
+input.historysubmit {
+    padding: 0 0.3em 0.3em 0.3em !important;
+    font-size: 94%;
+    cursor: pointer;
+    height: 1.7em !important;
+    margin-left: 1.6em;
 }
 input[type="radio"],
 input[type="checkbox"] { border:none; }
@@ -421,7 +422,6 @@ div.townBox dl dd {
     background-color:#f9f9f9;
     border:1px dashed #aaaaaa;
 }
-#pagehistory li.histsubmitli { list-style: none }
 /*
 ** Diff rendering
 */
@@ -822,17 +822,17 @@ fieldset.operaprefsection { margin-left: 15em }
 */
 
 /* tabs: border on the a, not the div */
-*>html #p-cactions li { border:none; }
-*>html #p-cactions li a {
+* > html #p-cactions li { border:none; }
+* > html #p-cactions li a {
     border: 1px solid #aaaaaa;
     border-bottom: none;
 }
-*>html #p-cactions li.selected a { border-color: #fabd23; }
+* > html #p-cactions li.selected a { border-color: #fabd23; }
 /* footer icons need a fixed width */
-*>html #f-poweredbyico,
-*>html #f-copyrightico { width: 88px; }
-*>html #bodyContent,
-*>html #bodyContent pre {
+* > html #f-poweredbyico,
+* > html #f-copyrightico { width: 88px; }
+* > html #bodyContent,
+* > html #bodyContent pre {
     overflow-x: auto;
     width: 100%;
     padding-bottom: 25px;
@@ -846,6 +846,7 @@ fieldset.operaprefsection { margin-left: 15em }
     margin-bottom: 0;
 }
 * html div.editsection { font-size: smaller; }
+#pagehistory li.selected { position: relative; }
 
 /* debugging tool.. */
 /*div{ border:1px solid #000000;}*/

diff --git a/stylesheets/wikibits.js b/stylesheets/wikibits.js
index bab8d8a..9d22a09 100644 (file)
--- a/stylesheets/wikibits.js
+++ b/stylesheets/wikibits.js
@@ -62,7 +62,7 @@ function toggleVisibility( _levelId, _otherId, _linkId) {
 function histrowinit () {
     hf = document.getElementById('pagehistory');
     if(!hf) return;
-    lis = hf.getElementsByTagName('li');
+    lis = hf.getElementsByTagName('LI');
     for (i=0;i<lis.length;i++) {
         inputs=lis[i].getElementsByTagName('INPUT');
         if(inputs[0] && inputs[1]) {
@@ -78,7 +78,7 @@ function diffcheck() {
     var oli = false; // the li where the oldid radio is checked
     hf = document.getElementById('pagehistory');
     if(!hf) return;
-    lis = hf.getElementsByTagName('li');
+    lis = hf.getElementsByTagName('LI');
     for (i=0;i<lis.length;i++) {
         inputs=lis[i].getElementsByTagName('INPUT');
         if(inputs[1] && inputs[0]) {