Thijs/wiki.techinc.nl
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

AllMessagesTablePager: Avoid double class mw-datatable

Browse source 
This change replaces in the HTML of Special:AllMessages
  <table class="mw-datatable mw-datatable" id="mw-allmessagestable">
by
  <table class="mw-datatable" id="mw-allmessagestable">

Remove unnecessary space at the start of the value of the class
attribute in TablePager.

Inline the one-time used variable
  $tableClass = $this->getTableClass()

Change-Id: I4353d47bda3136cd3a8341677a0288f59895d5c6
This commit is contained in:
Fomafix 2021-10-09 16:48:30 +00:00
parent 2bcc75c9b4
commit 33720e7683
2 changed files with 2 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
includes/pager/TablePager.php
View file

@ -160,9 +160,8 @@ abstract class TablePager extends IndexPager {
}
}
$tableClass = $this->getTableClass();
$ret = Html::openElement( 'table', [
'class' => " $tableClass" ]
'class' => $this->getTableClass() ]
);
$ret .= Html::rawElement( 'thead', [], Html::rawElement( 'tr', [], "\n" . $s . "\n" ) );
$ret .= Html::openElement( 'tbody' ) . "\n";

3
includes/specials/pagers/AllMessagesTablePager.php
View file

@ -246,9 +246,8 @@ class AllMessagesTablePager extends TablePager {
}
protected function getStartBody() {
$tableClass = $this->getTableClass();
return Xml::openElement( 'table', [
'class' => "mw-datatable $tableClass",
'class' => $this->getTableClass(),
'id' => 'mw-allmessagestable'
] ) .
"\n" .