From 48fac1852412110852a970e2855160dcd3678e2e Mon Sep 17 00:00:00 2001 From: Alex Monk Date: Thu, 13 Sep 2012 23:01:37 +0100 Subject: [PATCH] (bug 40213) border=0 is pointless, 0 is the default. Change-Id: I6212915d098466e1317b0c21e7be12d40d467a7d --- includes/SkinLegacy.php | 2 +- includes/specials/SpecialMovepage.php | 2 +- includes/specials/SpecialPrefixindex.php | 4 ++-- includes/specials/SpecialSearch.php | 4 ++-- includes/specials/SpecialUndelete.php | 2 +- includes/specials/SpecialUserrights.php | 4 ++-- skins/CologneBlue.php | 4 ++-- skins/Standard.php | 2 +- tests/parser/parserTests.txt | 6 +++--- 9 files changed, 15 insertions(+), 15 deletions(-) diff --git a/includes/SkinLegacy.php b/includes/SkinLegacy.php index 1ae1cb3cce..e695ba6ca2 100644 --- a/includes/SkinLegacy.php +++ b/includes/SkinLegacy.php @@ -120,7 +120,7 @@ class LegacyTemplate extends BaseTemplate { } $s .= "\n
\n
\n" . - "\n\n"; + "
\n\n"; if ( $this->getSkin()->qbSetting() == 0 ) { $s .= "
\n" . diff --git a/includes/specials/SpecialMovepage.php b/includes/specials/SpecialMovepage.php index 3ece917b81..af3dbf3e44 100644 --- a/includes/specials/SpecialMovepage.php +++ b/includes/specials/SpecialMovepage.php @@ -258,7 +258,7 @@ class MovePageForm extends UnlistedSpecialPage { Xml::openElement( 'form', array( 'method' => 'post', 'action' => $this->getTitle()->getLocalURL( 'action=submit' ), 'id' => 'movepage' ) ) . Xml::openElement( 'fieldset' ) . Xml::element( 'legend', null, $this->msg( 'move-page-legend' )->text() ) . - Xml::openElement( 'table', array( 'border' => '0', 'id' => 'mw-movepage-table' ) ) . + Xml::openElement( 'table', array( 'id' => 'mw-movepage-table' ) ) . "
" . $this->msg( 'movearticle' )->escaped() . diff --git a/includes/specials/SpecialPrefixindex.php b/includes/specials/SpecialPrefixindex.php index c37bb1d310..7740b32033 100644 --- a/includes/specials/SpecialPrefixindex.php +++ b/includes/specials/SpecialPrefixindex.php @@ -188,7 +188,7 @@ class SpecialPrefixindex extends SpecialAllpages { $n = 0; if( $res->numRows() > 0 ) { - $out = Xml::openElement( 'table', array( 'border' => '0', 'id' => 'mw-prefixindex-list-table' ) ); + $out = Xml::openElement( 'table', array( 'id' => 'mw-prefixindex-list-table' ) ); while( ( $n < $this->maxPerPage ) && ( $s = $res->fetchObject() ) ) { $t = Title::makeTitle( $s->page_namespace, $s->page_title ); @@ -227,7 +227,7 @@ class SpecialPrefixindex extends SpecialAllpages { } else { $nsForm = $this->namespacePrefixForm( $namespace, $prefix, $hideredirects ); $self = $this->getTitle(); - $out2 = Xml::openElement( 'table', array( 'border' => '0', 'id' => 'mw-prefixindex-nav-table' ) ) . + $out2 = Xml::openElement( 'table', array( 'id' => 'mw-prefixindex-nav-table' ) ) . '
' . $nsForm . diff --git a/includes/specials/SpecialSearch.php b/includes/specials/SpecialSearch.php index 85365f5dd9..5f5b6b4da5 100644 --- a/includes/specials/SpecialSearch.php +++ b/includes/specials/SpecialSearch.php @@ -300,7 +300,7 @@ class SpecialSearch extends SpecialPage { ) ); $out->addHtml( - Xml::openElement( 'table', array( 'id'=>'mw-search-top-table', 'border'=>0, 'cellpadding'=>0, 'cellspacing'=>0 ) ) . + Xml::openElement( 'table', array( 'id' => 'mw-search-top-table', 'cellpadding' => 0, 'cellspacing' => 0 ) ) . Xml::openElement( 'tr' ) . Xml::openElement( 'td' ) . "\n" . $this->shortDialog( $term ) . @@ -868,7 +868,7 @@ class SpecialSearch extends SpecialPage { for( $i = 0; $i < $numRows; $i += 4 ) { $namespaceTables .= Xml::openElement( 'table', - array( 'cellpadding' => 0, 'cellspacing' => 0, 'border' => 0 ) + array( 'cellpadding' => 0, 'cellspacing' => 0 ) ); for( $j = $i; $j < $i + 4 && $j < $numRows; $j++ ) { $namespaceTables .= Xml::tags( 'tr', null, $rows[$j] ); diff --git a/includes/specials/SpecialUndelete.php b/includes/specials/SpecialUndelete.php index 6dde6b9a58..d8e0b97c45 100644 --- a/includes/specials/SpecialUndelete.php +++ b/includes/specials/SpecialUndelete.php @@ -908,7 +908,7 @@ class SpecialUndelete extends SpecialPage { $diffEngine->showDiffStyle(); $this->getOutput()->addHTML( "
" . - "" . + "
" . "" . "" . "" . diff --git a/includes/specials/SpecialUserrights.php b/includes/specials/SpecialUserrights.php index 59d983ff6e..9f5a48a530 100644 --- a/includes/specials/SpecialUserrights.php +++ b/includes/specials/SpecialUserrights.php @@ -449,7 +449,7 @@ class UserrightsPage extends SpecialPage { $this->msg( 'userrights-groups-help', $user->getName() )->parse() . $grouplist . Xml::tags( 'p', null, $this->groupCheckboxes( $groups, $user ) ) . - Xml::openElement( 'table', array( 'border' => '0', 'id' => 'mw-userrights-table-outer' ) ) . + Xml::openElement( 'table', array( 'id' => 'mw-userrights-table-outer' ) ) . "\n"; foreach( $columns as $name => $column ) { if( $column === array() ) diff --git a/skins/CologneBlue.php b/skins/CologneBlue.php index 913c19bfa3..1823d8823a 100644 --- a/skins/CologneBlue.php +++ b/skins/CologneBlue.php @@ -80,7 +80,7 @@ class CologneBlueTemplate extends LegacyTemplate { $mainPageObj = Title::newMainPage(); $s = "\n
\n
" . - '
" . Xml::label( $this->msg( 'userrights-reason' )->text(), 'wpReason' ) . @@ -534,7 +534,7 @@ class UserrightsPage extends SpecialPage { } # Build the HTML table - $ret .= Xml::openElement( 'table', array( 'border' => '0', 'class' => 'mw-userrights-groups' ) ) . + $ret .= Xml::openElement( 'table', array( 'class' => 'mw-userrights-groups' ) ) . "
'; + '
'; $s .= '
'; $s .= ''; @@ -122,7 +122,7 @@ class CologneBlueTemplate extends LegacyTemplate { $s = "\n
\n"; $s .= "\n