From c2bb2a8fb458456a322716e82d5ed20c9451d5a0 Mon Sep 17 00:00:00 2001 From: jrobson Date: Mon, 1 Jul 2013 13:50:14 -0700 Subject: [PATCH] Render dates as headings Not doing this leads to unnecessary styling in mobile and is more semantically correct Bug: 49269 Change-Id: Icbb390b6c807118c1b775aa4f962de06b5f7c091 --- special/SpecialNotifications.php | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/special/SpecialNotifications.php b/special/SpecialNotifications.php index 5b48eea89..1d19e4cb3 100644 --- a/special/SpecialNotifications.php +++ b/special/SpecialNotifications.php @@ -56,7 +56,9 @@ class SpecialNotifications extends SpecialPage { // Output the date header if it has not been displayed if ( $dateHeader !== $row['timestamp']['date'] ) { $dateHeader = $row['timestamp']['date']; - $notices .= Html::rawElement( 'li', array( 'class' => 'mw-echo-date-section' ), $dateHeader ); + $notices .= Html::openElement( 'li' ); + $notices .= Html::element( 'h2', array( 'class' => 'mw-echo-date-section' ), $dateHeader ); + $notices .= Html::closeElement( 'li' ); } $class = 'mw-echo-notification';