: $nothing;
$f .= $bot ? '<span class="bot">' . $this->message['boteditletter'] . '</span>' : $nothing;
$f .= $patrolled ? '<span class="unpatrolled">!</span>' : $nothing;
- return "<tt>$f</tt>";
+ return $f;
}
/**
$this->insertDiffHist($s, $rc, $unpatrolled);
# M, N, b and ! (minor, new, bot and unpatrolled)
- $s .= ' ' . $this->recentChangesFlags( $rc_type == RC_NEW, $rc_minor, $unpatrolled, '', $rc_bot );
+ $s .= '<tt> '.$this->recentChangesFlags( $rc_type == RC_NEW, $rc_minor, $unpatrolled, '', $rc_bot ).'</tt>';
$this->insertArticleLink($s, $rc, $unpatrolled, $watched);
wfProfileOut($fname.'-page');
*/
function recentChangesBlockGroup( $block ) {
global $wgLang, $wgContLang, $wgRCShowChangedSize;
- $r = '<table cellpadding="0" cellspacing="0" border="0"><tr>';
+ $r = '<table cellpadding="0" cellspacing="0" border="1"><tr>';
# Collate list of users
$isnew = false;
$toggleLink = "javascript:toggleVisibility('$rci','$rcm','$rcl')";
$tl = '<span id="'.$rcm.'"><a href="'.$toggleLink.'">' . $this->sideArrow() . '</a></span>';
$tl .= '<span id="'.$rcl.'" style="display:none"><a href="'.$toggleLink.'">' . $this->downArrow() . '</a></span>';
- $r .= '<td valign="top">'.$tl;
+ $r .= '<td valign="top">'.$tl.'</td>';
# Main line
- $r .= ' '.$this->recentChangesFlags( $isnew, false, $unpatrolled, ' ', $bot );
+ $r .= '<td valign="top"><tt> '.$this->recentChangesFlags( $isnew, false, $unpatrolled, ' ', $bot );
# Timestamp
- $r .= ' <tt>'.$block[0]->timestamp.' </tt></td><td>';
+ $r .= ' '.$block[0]->timestamp.' </tt></td><td>';
# Article link
if ( $namehidden )
$r .= "</td></tr></table>\n";
# Sub-entries
- $r .= '<div id="'.$rci.'" style="display:none;"><table cellpadding="0" cellspacing="0" border="0">';
+ $r .= '<div id="'.$rci.'" style="display:none;"><table cellpadding="0" cellspacing="0" border="1">';
foreach( $block as $rcObj ) {
# Get rc_xxxx variables
// FIXME: Would be good to replace this extract() call with something that explicitly initializes local variables.
extract( $rcObj->mAttribs );
#$r .= '<tr><td valign="top">'.$this->spacerArrow();
- $r .= '<tr><td valign="top">'.$this->spacerIndent();
- $r .= '<tt> </tt>';
+ $r .= '<tr><td valign="top">';
+ $r .= '<tt>'.$this->spacerIndent() . $this->spacerIndent();
$r .= $this->recentChangesFlags( $rc_new, $rc_minor, $rcObj->unpatrolled, ' ', $rc_bot );
- $r .= ' </td><td valign="top">';
+ $r .= ' </tt></td><td valign="top">';
$o = '';
if( $rc_this_oldid != 0 ) {
return '<td width="12"></td>';
}
- // Adds a few spaces
+ /**
+ * Add a set of spaces
+ * @return string HTML <td> tag
+ * @access private
+ */
function spacerIndent() {
return ' ';
}
extract( $rcObj->mAttribs );
$curIdEq = 'curid='.$rc_cur_id;
- $r = '<table cellspacing="0" cellpadding="0" border="0"><tr>';
+ $r = '<table cellspacing="0" cellpadding="0" border="1"><tr>';
# spacerArrow() causes issues in FF
$r .= $this->spacerColumn();
$r .= '<td valign="top">';
# Flag and Timestamp
+ $r .= '<tt> ';
if( $rc_type == RC_MOVE || $rc_type == RC_MOVE_OVER_REDIRECT ) {
- $r .= ' ';
+ $r .= ' '; // 4 flags -> 4 spaces
} else {
- $r .= ' '.$this->recentChangesFlags( $rc_type == RC_NEW, $rc_minor, $rcObj->unpatrolled, ' ', $rc_bot );
+ $r .= $this->recentChangesFlags( $rc_type == RC_NEW, $rc_minor, $rcObj->unpatrolled, ' ', $rc_bot );
}
- $r .= ' <tt>'.$rcObj->timestamp.'</tt> </td><td>';
+ $r .= ' '.$rcObj->timestamp.' </tt></td><td>';
# Article link
if ( $rc_log_type !='' ) {