Add tests for r96562. HttpTest still has 5 failing test (which is why it's marked...
[lhc/web/wiklou.git] / skins / CologneBlue.php
index c395e6a..66255ad 100644 (file)
@@ -19,42 +19,44 @@ class SkinCologneBlue extends SkinLegacy {
        var $skinname = 'cologneblue', $stylename = 'cologneblue',
                $template = 'CologneBlueTemplate';
 
+       /**
+        * @param $out OutputPage
+        */
        function setupSkinUserCss( OutputPage $out ){
                parent::setupSkinUserCss( $out );
                $out->addModuleStyles( 'skins.cologneblue' );
 
-               global $wgContLang;
                $qb = $this->qbSetting();
                $rules = array();
 
                if ( 2 == $qb ) { # Right
-                       $rules[] = "#quickbar { position: absolute; right: 4px; }";
-                       $rules[] = "#article { margin-left: 4px; margin-right: 148px; }";
+                       $rules[] = "/* @noflip */#quickbar { position: absolute; right: 4px; }";
+                       $rules[] = "/* @noflip */#article { margin-left: 4px; margin-right: 148px; }";
                } elseif ( 1 == $qb ) {
-                       $rules[] = "#quickbar { position: absolute; left: 4px; }";
-                       $rules[] = "#article { margin-left: 148px; margin-right: 4px; }";
+                       $rules[] = "/* @noflip */#quickbar { position: absolute; left: 4px; }";
+                       $rules[] = "/* @noflip */#article { margin-left: 148px; margin-right: 4px; }";
                } elseif ( 3 == $qb ) { # Floating left
-                       $rules[] = "#quickbar { position:absolute; left:4px }";
-                       $rules[] = "#topbar { margin-left: 148px }";
-                       $rules[] = "#article { margin-left:148px; margin-right: 4px; }";
-                       $rules[] = "body>#quickbar { position:fixed; left:4px; top:4px; overflow:auto ;bottom:4px;}"; # Hides from IE
+                       $rules[] = "/* @noflip */#quickbar { position:absolute; left:4px }";
+                       $rules[] = "/* @noflip */#topbar { margin-left: 148px }";
+                       $rules[] = "/* @noflip */#article { margin-left:148px; margin-right: 4px; }";
+                       $rules[] = "/* @noflip */body>#quickbar { position:fixed; left:4px; top:4px; overflow:auto; bottom:4px;}"; # Hides from IE
                } elseif ( 4 == $qb ) { # Floating right
-                       $rules[] = "#quickbar { position: fixed; right: 4px; }";
-                       $rules[] = "#topbar { margin-right: 148px }";
-                       $rules[] = "#article { margin-right: 148px; margin-left: 4px; }";
-                       $rules[] = "body>#quickbar { position: fixed; right: 4px; top: 4px; overflow: auto ;bottom:4px;}"; # Hides from IE
+                       $rules[] = "/* @noflip */#quickbar { position: fixed; right: 4px; }";
+                       $rules[] = "/* @noflip */#topbar { margin-right: 148px }";
+                       $rules[] = "/* @noflip */#article { margin-right: 148px; margin-left: 4px; }";
+                       $rules[] = "/* @noflip */body>#quickbar { position: fixed; right: 4px; top: 4px; overflow: auto; bottom:4px;}"; # Hides from IE
                }
                $style = implode( "\n", $rules );
-               if ( $wgContLang->getDir() === 'rtl' ) {
-                       $style = CSSJanus::transform( $style, true, false );
-               }
-               $out->addInlineStyle( $style );
+               $out->addInlineStyle( $style, /* flip css if RTL */true );
        }
 
 }
 
 class CologneBlueTemplate extends LegacyTemplate {
 
+       /**
+        * @return string
+        */
        function doBeforeContent() {
                $mainPageObj = Title::newMainPage();
 
@@ -75,7 +77,7 @@ class CologneBlueTemplate extends LegacyTemplate {
 
                $s .= '<font size="-1"><span id="langlinks">';
                $s .= str_replace( '<br />', '', $this->otherLanguages() );
-               $cat = $this->getSkin()->getCategoryLinks();
+               $cat = '<div id="catlinks" class="catlinks">' . $this->getSkin()->getCategoryLinks() . '</div>';
                if( $cat ) {
                        $s .= "<br />$cat\n";
                }
@@ -95,6 +97,9 @@ class CologneBlueTemplate extends LegacyTemplate {
                return $s;
        }
 
+       /**
+        * @return string
+        */
        function doAfterContent(){
                global $wgLang;
 
@@ -136,6 +141,9 @@ class CologneBlueTemplate extends LegacyTemplate {
                return $s;
        }
 
+       /**
+        * @return string
+        */
        function sysLinks() {
                global $wgUser, $wgLang;
                $li = SpecialPage::getTitleFor( 'Userlogin' );
@@ -194,6 +202,8 @@ class CologneBlueTemplate extends LegacyTemplate {
        /**
         * Compute the sidebar
         * @access private
+        *
+        * @return string
         */
        function quickBar(){
                global $wgOut, $wgUser;
@@ -225,10 +235,12 @@ class CologneBlueTemplate extends LegacyTemplate {
                                }
                                $s .= "\n<h6>" . htmlspecialchars( $h ) . "</h6>";
                        }
-                       foreach ( $browseLinks as $link ) {
-                               if ( $link['text'] != '-' ) {
-                                       $s .= "<a href=\"{$link['href']}\">" .
-                                               htmlspecialchars( $link['text'] ) . '</a>' . $sep;
+                       if( is_array( $browseLinks ) ) {
+                               foreach ( $browseLinks as $link ) {
+                                       if ( $link['text'] != '-' ) {
+                                               $s .= "<a href=\"{$link['href']}\">" .
+                                                       htmlspecialchars( $link['text'] ) . '</a>' . $sep;
+                                       }
                                }
                        }
                        $barnumber++;
@@ -347,11 +359,19 @@ class CologneBlueTemplate extends LegacyTemplate {
                return $s;
        }
 
+       /**
+        * @param $key string
+        * @return string
+        */
        function menuHead( $key ) {
                $s = "\n<h6>" . wfMsg( $key ) . "</h6>";
                return $s;
        }
 
+       /**
+        * @param $label string
+        * @return string
+        */
        function searchForm( $label = '' ) {
                global $wgRequest, $wgUseTwoButtonsSearchForm;