MDL-27550 workshop: fixed XHTML strictness of H3 and A combo

H3 is block level element while A is in-line element in XHTML.
Credit goes to Sam Hemelryk for spotting this during the peer-review.

diff --git a/mod/workshop/renderer.php b/mod/workshop/renderer.php
index d0053bb..955a171 100644 (file)
--- a/mod/workshop/renderer.php
+++ b/mod/workshop/renderer.php
@@ -92,14 +92,14 @@ class mod_workshop_renderer extends plugin_renderer_base {
         $o .= $this->output->container_start($classes);
         $o .= $this->output->container_start('header');
 
-        $title = $this->output->heading(format_string($submission->title), 3, 'title');
+        $title = format_string($submission->title);
 
         if ($this->page->url != $submission->url) {
-            $o .= html_writer::link($submission->url, $title);
-        } else {
-            $o .= $title;
+            $title = html_writer::link($submission->url, $title);
         }
 
+        $o .= $this->output->heading($title, 3, 'title');
+
         if (!$anonymous) {
             $author             = new stdclass();
             $author->id         = $submission->authorid;