MDL-37009 Display course info as a page instead of popup
authorMarina Glancy <marina@moodle.com>
Thu, 28 Mar 2013 01:20:44 +0000 (12:20 +1100)
committerMarina Glancy <marina@moodle.com>
Mon, 1 Apr 2013 23:58:24 +0000 (10:58 +1100)
course/info.php
course/renderer.php
theme/afterburner/style/afterburner_styles.css
theme/base/style/course.css
theme/magazine/style/core.css

index f2707fa..cce2227 100644 (file)
         print_error('coursehidden', '', $CFG->wwwroot .'/');
     }
 
-    $PAGE->set_context($context);
-    $PAGE->set_pagelayout('popup');
+    $PAGE->set_course($course);
+    $PAGE->set_pagelayout('course');
     $PAGE->set_url('/course/info.php', array('id' => $course->id));
     $PAGE->set_title(get_string("summaryof", "", $course->fullname));
     $PAGE->set_heading(get_string('courseinfo'));
-    $PAGE->set_course($course);
     $PAGE->navbar->add(get_string('summary'));
 
     echo $OUTPUT->header();
index 56318fe..0416893 100644 (file)
@@ -1069,13 +1069,12 @@ class core_course_renderer extends plugin_renderer_base {
 
         // If we display course in collapsed form but the course has summary or course contacts, display the link to the info page.
         $content .= html_writer::start_tag('div', array('class' => 'moreinfo'));
-        if ($course->has_summary() || $course->has_course_contacts()) {
-            if ($chelper->get_show_courses() < self::COURSECAT_SHOW_COURSES_EXPANDED) {
+        if ($chelper->get_show_courses() < self::COURSECAT_SHOW_COURSES_EXPANDED) {
+            if ($course->has_summary() || $course->has_course_contacts()) {
                 $url = new moodle_url('/course/info.php', array('id' => $course->id));
                 $image = html_writer::empty_tag('img', array('src' => $this->output->pix_url('i/info'),
                     'alt' => $this->strings->summary));
-                $content .= $this->action_link($url, $image, new popup_action('click', $url, 'courseinfo'),
-                        array('title' => $this->strings->summary));
+                $content .= html_writer::link($url, $image, array('title' => $this->strings->summary));
             }
         }
         $content .= html_writer::end_tag('div'); // .moreinfo
index b317e3d..46052a0 100644 (file)
@@ -224,7 +224,7 @@ textarea {
 /*
 Courses and categories lists
 ----------------------------*/
-.courses .coursebox .summary {
+.coursebox .summary {
     width: 100%;
     float: left;
 }
index cac1c9f..2cf1807 100644 (file)
@@ -123,7 +123,7 @@ li.section.hidden span.commands a.editing_show {cursor:default;}
 .coursebox .enrolmenticons img,
 .coursebox .moreinfo img {margin:0 .2em;}
 .coursebox .content {clear:both;}
-.courses .coursebox .summary,
+.coursebox .summary,
 .coursebox .coursecat {float:right;width: 55%;}
 .coursebox .coursecat {text-align:right;clear:right;}
 .coursebox.remotecoursebox .remotecourseinfo {float:left;width: 40%;}
@@ -133,7 +133,7 @@ li.section.hidden span.commands a.editing_show {cursor:default;}
 .dir-rtl .coursebox .teachers {float:right;}
 .dir-rtl .coursebox .enrolmenticons,
 .dir-rtl .coursebox .moreinfo {float:left;}
-.dir-rtl .courses .coursebox .summary,
+.dir-rtl .coursebox .summary,
 .dir-rtl .coursebox .coursecat {float:left;}
 .dir-rtl .coursebox .coursecat {text-align:left;clear:left;}
 
index 3dedc11..6643e4f 100644 (file)
@@ -904,26 +904,26 @@ div.coursebox h3.name a {
     font-weight: 400;
 }
 
-.courses .coursebox div.summary {
+.coursebox div.summary {
     font-size: 14px;
     color: #666666;
     font-style: italic;
 }
 
-.courses .coursebox ul.teachers {
+.coursebox ul.teachers {
     display: block;
-    padding-left: 5px !important;
+    padding-left: 5px;
 }
 
-.dir-rtl .courses .coursebox ul.teachers {
+.dir-rtl .coursebox ul.teachers {
     padding-right: 5px;
 }
 
-.courses .coursebox ul.teachers li {
+.coursebox ul.teachers li {
     font-size: 10px;
 }
 
-.courses .coursebox ul.teachers li a {
+.coursebox ul.teachers li a {
     font-size: 11px;
 }