diff --git a/blocks/online_users/block_online_users.php b/blocks/online_users/block_online_users.php
index 2d71dd9cb02b2..8d3f1003b383a 100644
--- a/blocks/online_users/block_online_users.php
+++ b/blocks/online_users/block_online_users.php
@@ -113,7 +113,7 @@ function get_content() {
         //Calculate minutes
         $minutes  = floor($timetoshowusers/60);
 
-        $this->content->text = "<font size=\"-2\"><div align=center>(".get_string("periodnminutes","block_online_users",$minutes).")</div></font>";
+        $this->content->text = "<font size=\"-2\"><div align=\"center\">(".get_string("periodnminutes","block_online_users",$minutes).")</div></font>";
 
         //Now, we have in users, the list of users to show
         //Because they are online
@@ -137,7 +137,7 @@ function get_content() {
                 $table->align = array("right","left");
                 $table->cellpadding = 1;
                 $table->cellspacing = 1;
-                $table->data[] = array("<img src=\"$CFG->pixpath/i/user.gif\" height=16 width=16 alt=\"\">",$user->fullname);
+                $table->data[] = array("<img src=\"$CFG->pixpath/i/user.gif\" height=\"16\" width=\"16\" alt=\"\">",$user->fullname);
             }
             // Slightly hacky way to do it but...
             ob_start();
@@ -146,7 +146,7 @@ function get_content() {
             ob_end_clean();
 */
         } else {
-            $this->content->text .= "<font size=\"-1\"><p align=center>".get_string("none")."</p></font>";
+            $this->content->text .= "<font size=\"-1\"><p align=\"center\">".get_string("none")."</p></font>";
         }
 
         return $this->content;
diff --git a/blocks/section_links/block_section_links.php b/blocks/section_links/block_section_links.php
index a6f08a8639a4e..03043519d4736 100644
--- a/blocks/section_links/block_section_links.php
+++ b/blocks/section_links/block_section_links.php
@@ -63,7 +63,7 @@ function get_content() {
         } else {
             $link = '#';
         }
-        $text = '<font size=-1>';
+        $text = '<font size="-1">';
         for ($i = $inc; $i <= $this->course->numsections; $i += $inc) {
             $isvisible = get_field('course_sections', 'visible', 'course', $this->course->id, 'section', $i);
             if (!$isvisible and !isteacher($this->course->id)) {