Move the text direction class from <html> to <body>, HTML doesn't allow class on...
authorBrion Vibber <brion@users.mediawiki.org>
Thu, 9 Feb 2006 06:02:10 +0000 (06:02 +0000)
committerBrion Vibber <brion@users.mediawiki.org>
Thu, 9 Feb 2006 06:02:10 +0000 (06:02 +0000)
(Wouldn't be needed if the MSIE developers would actually make attribute selectors work on attributes that I actually want to use...)

skins/MonoBook.php
skins/monobook/IE70Fixes.css

index d8447fd..8202709 100644 (file)
@@ -51,7 +51,7 @@ class MonoBookTemplate extends QuickTemplate {
                wfSuppressWarnings();
 
 ?><!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
-<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="<?php $this->text('lang') ?>" lang="<?php $this->text('lang') ?>" dir="<?php $this->text('dir') ?>" class="<?php $this->text('dir') ?>">
+<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="<?php $this->text('lang') ?>" lang="<?php $this->text('lang') ?>" dir="<?php $this->text('dir') ?>">
        <head>
                <meta http-equiv="Content-Type" content="<?php $this->text('mimetype') ?>; charset=<?php $this->text('charset') ?>" />
                <?php $this->html('headlinks') ?>
@@ -62,7 +62,7 @@ class MonoBookTemplate extends QuickTemplate {
                <!--[if lt IE 5.5000]><style type="text/css">@import "<?php $this->text('stylepath') ?>/<?php $this->text('stylename') ?>/IE50Fixes.css";</style><![endif]-->
                <!--[if IE 5.5000]><style type="text/css">@import "<?php $this->text('stylepath') ?>/<?php $this->text('stylename') ?>/IE55Fixes.css";</style><![endif]-->
                <!--[if IE 6]><style type="text/css">@import "<?php $this->text('stylepath') ?>/<?php $this->text('stylename') ?>/IE60Fixes.css";</style><![endif]-->
-               <!--[if IE 7]><style type="text/css">@import "<?php $this->text('stylepath') ?>/<?php $this->text('stylename') ?>/IE70Fixes.css";</style><![endif]-->
+               <!--[if IE 7]><style type="text/css">@import "<?php $this->text('stylepath') ?>/<?php $this->text('stylename') ?>/IE70Fixes.css?1";</style><![endif]-->
                <!--[if lt IE 7]><script type="<?php $this->text('jsmimetype') ?>" src="<?php $this->text('stylepath') ?>/common/IEFixes.js"></script>
                <meta http-equiv="imagetoolbar" content="no" /><![endif]-->
 <?php  if($this->data['jsvarurl'  ]) { ?>
@@ -85,7 +85,7 @@ class MonoBookTemplate extends QuickTemplate {
        </head>
 <body <?php if($this->data['body_ondblclick']) { ?>ondblclick="<?php $this->text('body_ondblclick') ?>"<?php } ?>
 <?php if($this->data['body_onload'    ]) { ?>onload="<?php     $this->text('body_onload')     ?>"<?php } ?>
-<?php if($this->data['nsclass'        ]) { ?>class="<?php      $this->text('nsclass')         ?>"<?php } ?>>
+ class="<?php $this->text('nsclass') ?> <?php $this->text('dir') ?>">
        <div id="globalWrapper">
                <div id="column-content">
        <div id="content">
index 4866e08..c15e185 100644 (file)
@@ -9,13 +9,13 @@
        height: 1%;
 }
 
-html.rtl #column-one {
+.rtl #column-one {
        /* For some reason it tries to inherit the padding-top into every div,
         * and I can't figure out how to get it back off.
         */
        padding-top: 0;
 }
-html.rtl #column-one #p-navigation {
+.rtl #column-one #p-navigation {
        margin-top: 160px;
 }