MDL-40829: courses heading: remove 'headerblock' class from header an…

…d fix heading levels
moodle · Sep 10, 2013 · 278d645 · 278d645
1 parent 62b35d5
commit 278d645
Show file tree

Hide file tree

Showing 28 changed files with 22 additions and 132 deletions.
diff --git a/course/format/renderer.php b/course/format/renderer.php
@@ -651,7 +651,7 @@ public function print_single_section_page($course, $sections, $mods, $modnames,
         // Title with section navigation links.
         $sectionnavlinks = $this->get_nav_links($course, $modinfo->get_section_info_all(), $displaysection);
         $sectiontitle = '';
-        $sectiontitle .= html_writer::start_tag('div', array('class' => 'section-navigation header headingblock'));
+        $sectiontitle .= html_writer::start_tag('div', array('class' => 'section-navigation'));
         $sectiontitle .= html_writer::tag('span', $sectionnavlinks['previous'], array('class' => 'mdl-left'));
         $sectiontitle .= html_writer::tag('span', $sectionnavlinks['next'], array('class' => 'mdl-right'));
         // Title attributes

diff --git a/course/renderer.php b/course/renderer.php
@@ -1129,9 +1129,11 @@ protected function coursecat_coursebox(coursecat_helper $chelper, $course, $addi
         $classes = trim('coursebox clearfix '. $additionalclasses);
         if ($chelper->get_show_courses() >= self::COURSECAT_SHOW_COURSES_EXPANDED) {
             $nametag = 'h3';
+            $nameattributes = array();
         } else {
             $classes .= ' collapsed';
             $nametag = 'div';
+            $nameattributes = array('class' => 'name');
         }
         $content .= html_writer::start_tag('div', array('class' => $classes)); // .coursebox
 
@@ -1141,7 +1143,7 @@ protected function coursecat_coursebox(coursecat_helper $chelper, $course, $addi
         $coursename = $chelper->get_course_formatted_name($course);
         $coursenamelink = html_writer::link(new moodle_url('/course/view.php', array('id' => $course->id)),
                 $coursename, array('class' => $course->visible ? '' : 'dimmed'));
-        $content .= html_writer::tag($nametag, $coursenamelink, array('class' => 'name'));
+        $content .= html_writer::tag($nametag, $coursenamelink, $nameattributes);
 
         // 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'));
@@ -1513,7 +1515,7 @@ protected function coursecat_category(coursecat_helper $chelper, $coursecat, $de
                     array('title' => get_string('numberofcourses'), 'class' => 'numberofcourse'));
         }
         $content .= html_writer::start_tag('div', array('class' => 'info'));
-        $content .= html_writer::tag(($depth > 1) ? 'h4' : 'h3', $categoryname, array('class' => 'name'));
+        $content .= html_writer::tag(($depth > 1) ? 'h4' : 'h3', $categoryname);
         $content .= html_writer::end_tag('div'); // .info
 
         // add category content to the output

diff --git a/index.php b/index.php
@@ -191,11 +191,11 @@
                         } else {
                             $subtext = get_string('subscribe', 'forum');
                         }
-                        echo $OUTPUT->heading($forumname, 2, 'headingblock header');
+                        echo $OUTPUT->heading($forumname);
                         $suburl = new moodle_url('/mod/forum/subscribe.php', array('id' => $newsforum->id, 'sesskey' => sesskey()));
                         echo html_writer::tag('div', html_writer::link($suburl, $subtext), array('class' => 'subscribelink'));
                     } else {
-                        echo $OUTPUT->heading($forumname, 2, 'headingblock header');
+                        echo $OUTPUT->heading($forumname);
                     }
 
                     forum_print_latest_discussions($SITE, $newsforum, $SITE->newsitems, 'plain', 'p.modified DESC');
@@ -215,7 +215,7 @@
                     //wrap frontpage course list in div container
                     echo html_writer::start_tag('div', array('id'=>'frontpage-course-list'));
 
-                    echo $OUTPUT->heading(get_string('mycourses'), 2, 'headingblock header');
+                    echo $OUTPUT->heading(get_string('mycourses'));
                     echo $mycourseshtml;
 
                     //end frontpage course list div container
@@ -234,7 +234,7 @@
                     //wrap frontpage course list in div container
                     echo html_writer::start_tag('div', array('id'=>'frontpage-course-list'));
 
-                    echo $OUTPUT->heading(get_string('availablecourses'), 2, 'headingblock header');
+                    echo $OUTPUT->heading(get_string('availablecourses'));
                     echo $availablecourseshtml;
 
                     //end frontpage course list div container
@@ -250,7 +250,7 @@
                 //wrap frontpage category names in div container
                 echo html_writer::start_tag('div', array('id'=>'frontpage-category-names'));
 
-                echo $OUTPUT->heading(get_string('categories'), 2, 'headingblock header');
+                echo $OUTPUT->heading(get_string('categories'));
                 echo $courserenderer->frontpage_categories_list();
 
                 //end frontpage category names div container
@@ -265,7 +265,7 @@
                 //wrap frontpage category combo in div container
                 echo html_writer::start_tag('div', array('id'=>'frontpage-category-combo'));
 
-                echo $OUTPUT->heading(get_string('courses'), 2, 'headingblock header');
+                echo $OUTPUT->heading(get_string('courses'));
                 echo $courserenderer->frontpage_combo_list();
 
                 //end frontpage category combo div container

diff --git a/tag/index.php b/tag/index.php
@@ -90,7 +90,7 @@
     $tagname =  '<span class="flagged-tag">' . $tagname . '</span>';
 }
 
-echo $OUTPUT->heading($tagname, 2, 'headingblock header tag-heading');
+echo $OUTPUT->heading($tagname, 2);
 tag_print_management_box($tag);
 tag_print_description_box($tag);
 // Check what type of results are avaialable

diff --git a/theme/afterburner/style/afterburner_styles.css b/theme/afterburner/style/afterburner_styles.css
@@ -22,11 +22,6 @@ hr {
     border-bottom: 1px dotted #808080;
     border-top: 0px;
 }
-h2.headingblock {
-    border-bottom: 1px solid #ddd;
-    font-size: 1.4em;
-    font-weight: bold;
-}
 /*
 Page
 -----------------------*/
@@ -254,10 +249,7 @@ Course Section Topic & Weekly
 .course-content ul.topics li.section {
   padding-top: 1em;
 }
-.path-course-view .headingblock {
-    display: none; /* hides topic items title or weekly itens title */
-    margin-bottom: 9px;
-}
+
 /*Login (Login page)*/
 .loginbox .loginform .form-label {
     width:46%;

diff --git a/theme/anomaly/style/general.css b/theme/anomaly/style/general.css
@@ -106,12 +106,7 @@ h2.tagline {
 .sitetopic {
     border-width: 0;
 }
-.headingblock {
-    margin-bottom: 10px;
-    border-bottom: 1px solid #697F55;
-    background-color: #E3E3E3;
-    padding: 4px 5px;
-}
+
 .coursebox {
     width: 100%;
     margin: 10px 0;
@@ -144,9 +139,6 @@ h2.tagline {
     border:none;
 }
 
-.course-content .headingblock.outline {
-    margin-top: 0;
-}
 .course-content .section.main {
     border:1px solid #E3E3E3;
     margin-bottom: 10px;

diff --git a/theme/arialist/style/core.css b/theme/arialist/style/core.css
@@ -110,14 +110,6 @@ fieldset#general legend {
 /* Courses
 ----------------------------*/
 
-h2.headingblock {
-    font-weight: normal;
-    color:#999;
-    font-size: 1.5em;
-    border-bottom:1px solid #eee;
-    padding-bottom:2px;
-}
-
 .coursebox {
     border-color: #eee;
 }

diff --git a/theme/base/style/core.css b/theme/base/style/core.css
@@ -90,8 +90,7 @@ a.autolink.glossary:hover {cursor: help;}
 .warning,
 .errorbox .title,
 .pagingbar .title,
-.pagingbar .thispage,
-.headingblock {font-weight: bold;}
+.pagingbar .thispage {font-weight: bold;}
 img.resize {height: 1em;width: 1em;}
 .block img.resize,
 .breadcrumb img.resize {height: 0.9em;width: 0.8em;}

diff --git a/theme/base/style/course.css b/theme/base/style/course.css
@@ -63,11 +63,9 @@
 .course-content .single-section .section-navigation .mdl-bottom { margin-top: 0; }
 
 #page-site-index .subscribelink {text-align:right;}
-#page-site-index .headingblock {margin-bottom: 9px;}
 #page-site-index .clearfloat {float: none;clear: both; height:0px;}
 
 .path-course-view a.reduce-sections {padding-left: 0.2em;}
-.path-course-view .headingblock {margin-bottom: 9px;}
 .path-course-view .subscribelink {text-align:right;}
 .path-course-view .unread {margin-left: 3em;}
 .path-course-view .block.drag .header {cursor: move;}

diff --git a/theme/bootstrapbase/less/moodle/core.less b/theme/bootstrapbase/less/moodle/core.less
@@ -165,8 +165,7 @@ a.autolink.glossary:hover {
 .warning,
 .errorbox .title,
 .pagingbar .title,
-.pagingbar .thispage,
-.headingblock {
+.pagingbar .thispage {
     font-weight: bold;
 }
 img.resize {

diff --git a/theme/bootstrapbase/less/moodle/course.less b/theme/bootstrapbase/less/moodle/course.less
@@ -201,16 +201,10 @@
 #page-site-index .subscribelink {
     text-align: right;
 }
-#page-site-index .headingblock {
-    margin-bottom: 9px;
-}
 
 .path-course-view a.reduce-sections {
     padding-left: 0.2em;
 }
-.path-course-view .headingblock {
-    margin-bottom: 9px;
-}
 .path-course-view .subscribelink {
     text-align: right;
 }

diff --git a/theme/bootstrapbase/style/moodle.css b/theme/bootstrapbase/style/moodle.css
diff --git a/theme/boxxie/style/core.css b/theme/boxxie/style/core.css
@@ -264,10 +264,6 @@ td.eventskey {
 
 /* Course */
 
-h2.headingblock {
-    font-weight: normal;
-}
-
 .coursebox {
     border-bottom: 1px solid #eee;
 }
@@ -293,11 +289,6 @@ h2.headingblock {
     font-size: 0.95em;
 }
 
-h2.headingblock {
-    border-bottom: 1px solid #eee;
-    font-size: 1.5em;
-}
-
 #page-course-index-category .categorypicker {
     text-align: center;
     margin: 10px 0 20px;

diff --git a/theme/brick/style/colors.css b/theme/brick/style/colors.css
@@ -27,7 +27,7 @@ div.block {
 	border-bottom: 1px solid [[setting:maincolor]];
 }
 
-.generalbox h2,h3.sectionname, h2.headingblock,h2,h3, h2 a, h3 a, div.loginpanel h2, div.signuppanel h2, .forumpost .subject {
+.generalbox h2,h3.sectionname,h2,h3, h2 a, h3 a, div.loginpanel h2, div.signuppanel h2, .forumpost .subject {
 	color: [[setting:headingcolor]];
 }
 

diff --git a/theme/brick/style/core.css b/theme/brick/style/core.css
@@ -272,7 +272,7 @@ div.block.hidden div.header {
 
 /** main headings **/
 
-.generalbox h2,h3.sectionname, h2.headingblock,h2,h3, h2 a, h3 a, div.loginpanel h2, div.signuppanel h2 {
+.generalbox h2,h3.sectionname,h2,h3, h2 a, h3 a, div.loginpanel h2, div.signuppanel h2 {
     font-weight: 600;
     letter-spacing: -1px;
     line-height: 1.3em;

diff --git a/theme/canvas/style/core.css b/theme/canvas/style/core.css
@@ -17,12 +17,6 @@ body {
     border: none;
 }
 
-.headingblock {
-    border-bottom: 1px solid #ddd;
-    font-size: 1.3em;
-    color: #333;
-}
-
 #notice {
     text-align: center;
 }
@@ -439,11 +433,6 @@ body {
     text-align: right;
 }
 
-#page-tag-index .headingblock {
-    text-align: center;
-    margin-bottom: 0;
-}
-
 #page-tag-index #tag-management-box {
     border-bottom: 1px solid #ddd;
     padding: 4px 0;

diff --git a/theme/formal_white/style/core.css b/theme/formal_white/style/core.css
@@ -1,7 +1,3 @@
-.headingblock {
-    font-size:130%;
-}
-
 #notice {
     padding:1em;
 }

diff --git a/theme/fusion/style/core.css b/theme/fusion/style/core.css
@@ -258,13 +258,6 @@ margin: 0;
 /* Courses
 ----------------------------*/
 
-h2.headingblock {
-    font-weight: normal;
-    color:#999;
-    font-size: 1.5em;
-    border-bottom:1px dotted #eee;
-}
-
 .coursebox {
     border-color: #eee;
 }

diff --git a/theme/leatherbound/style/core.css b/theme/leatherbound/style/core.css
@@ -151,15 +151,6 @@ a:active {
 /* Course
 ---------------------------*/
 
-h2.headingblock {
-    border-width: 0 0 1px 0;
-    padding: 5px 5px 2px;
-    font-weight: normal;
-    font-size: 1.2em;
-    background: #f7f5f1;
-    color: #251c17;
-}
-
 .course-content .main {
     background: #f7f5f1;
     border: 1px solid #ddd

diff --git a/theme/magazine/style/colors.css b/theme/magazine/style/colors.css
@@ -46,7 +46,7 @@ div.dockedtitle {
 
 /** main headings and forum post subject default color #4e0000 **/
 
-#nologo, #nologo a, .generalbox h2,h3.sectionname, div.tabtree li.selected a span, .forumpost .subject, .blog_entry div.subject a, h2.headingblock,h2,h3, h2 a, h3 a, div.loginpanel h2, div.signuppanel h2 {
+#nologo, #nologo a, .generalbox h2,h3.sectionname, div.tabtree li.selected a span, .forumpost .subject, .blog_entry div.subject a, h2,h3, h2 a, h3 a, div.loginpanel h2, div.signuppanel h2 {
     color: [[setting:headingcolor]];
 }
 

diff --git a/theme/magazine/style/core.css b/theme/magazine/style/core.css
@@ -697,7 +697,7 @@ html>
 
 /** main headings **/
 
-.generalbox h2,h3.sectionname, h2.headingblock,h2,h3, h2 a, h3 a, div.loginpanel h2, div.signuppanel h2 {
+.generalbox h2,h3.sectionname,h2,h3, h2 a, h3 a, div.loginpanel h2, div.signuppanel h2 {
     font-size: 30px;
     padding-bottom: 2px;
     margin: 3px 0 2px;

diff --git a/theme/mymobile/style/core.css b/theme/mymobile/style/core.css
@@ -28,11 +28,6 @@ ul {
     margin: 0.5em 0;
     padding: 0;
 }
-.headingblock {
-    border-bottom:0px solid #DDDDDD;
-    color:inherit;
-    font-size:inherit;
-}
 h1.helpheading,
 h2.helpheading {
     font-size: 1.6em;

diff --git a/theme/nimble/style/core.css b/theme/nimble/style/core.css
@@ -79,11 +79,6 @@ h1.headermain {
 
 /* Content */
 
-h2.headingblock {
-	text-shadow: 1px 1px 3px #aaa;
-	color: #445566;
-}
-
 /* Region Post */
 
 #region-post-wrap-1 {

diff --git a/theme/serenity/style/core.css b/theme/serenity/style/core.css
@@ -116,15 +116,6 @@ a:hover {
 
 /* Course
 ----------------------------*/
-
-.headingblock {
-    background: url([[pix:theme|headingblock]]) repeat-x 0 0;
-    color: #fff;
-    font-weight: normal;
-    border: none;
-    padding: 2px 5px;
-}
-
 .course-content .main {
     border: 1px solid #eee;
     background: #eee;

diff --git a/theme/sky_high/style/core.css b/theme/sky_high/style/core.css
@@ -184,15 +184,6 @@ p.helplink a {
 
 /* @group Course */
 
-.headingblock {
-    border: 0px none;
-    border-bottom: 1px solid #98bcd6;
-    padding-bottom: 0px;
-    padding-top: 5px;
-    color: #336699;
-    font-size: 1.8em;
-    font-weight: normal;
-}
 .completionprogress {
     color: #336699;
     text-transform: uppercase;