DifferenceEngine: Use CSS classes instead of inline styles

Change-Id: Iee0e5e7eb73023a6f57c1f462e71cbb0e771d1f2
This commit is contained in:
Bartosz Dziewoński 2018-03-05 19:24:37 +01:00
parent 4afd9e566c
commit 99e8325cc1
2 changed files with 11 additions and 5 deletions

View file

@ -1294,7 +1294,7 @@ class DifferenceEngine extends ContextSource {
if ( !$diff && !$otitle ) {
$header .= "
<tr style=\"vertical-align: top;\" lang=\"{$userLang}\">
<tr class=\"diff-title\" lang=\"{$userLang}\">
<td class=\"diff-ntitle\">{$ntitle}</td>
</tr>";
$multiColspan = 1;
@ -1313,7 +1313,7 @@ class DifferenceEngine extends ContextSource {
}
if ( $otitle || $ntitle ) {
$header .= "
<tr style=\"vertical-align: top;\" lang=\"{$userLang}\">
<tr class=\"diff-title\" lang=\"{$userLang}\">
<td colspan=\"$colspan\" class=\"diff-otitle\">{$otitle}</td>
<td colspan=\"$colspan\" class=\"diff-ntitle\">{$ntitle}</td>
</tr>";
@ -1321,12 +1321,12 @@ class DifferenceEngine extends ContextSource {
}
if ( $multi != '' ) {
$header .= "<tr><td colspan=\"{$multiColspan}\" style=\"text-align: center;\" " .
$header .= "<tr><td colspan=\"{$multiColspan}\" " .
"class=\"diff-multi\" lang=\"{$userLang}\">{$multi}</td></tr>";
}
if ( $notice != '' ) {
$header .= "<tr><td colspan=\"{$multiColspan}\" style=\"text-align: center;\" " .
"lang=\"{$userLang}\">{$notice}</td></tr>";
$header .= "<tr><td colspan=\"{$multiColspan}\" " .
"class=\"diff-notice\" lang=\"{$userLang}\">{$notice}</td></tr>";
}
return $header . $diff . "</table>";

View file

@ -33,6 +33,12 @@
word-wrap: break-word;
}
.diff-title {
vertical-align: top;
}
.diff-notice,
.diff-multi,
.diff-otitle,
.diff-ntitle {
text-align: center;