Commit cb22b909 by Lucky Jindal Committed by Albert St. Aubin

AC-588: Fixed header levels,removed unneccessary element

parent ab44b12a
...@@ -48,7 +48,7 @@ div.info-wrapper { ...@@ -48,7 +48,7 @@ div.info-wrapper {
width: 100%; width: 100%;
display: block; display: block;
h1 { h1,h3 {
@include text-align(left); @include text-align(left);
@extend %t-strong; @extend %t-strong;
@extend %t-title6; @extend %t-title6;
...@@ -60,7 +60,7 @@ div.info-wrapper { ...@@ -60,7 +60,7 @@ div.info-wrapper {
margin-bottom: lh(); margin-bottom: lh();
} }
> ol,section { > ol,section,div {
list-style: none; list-style: none;
margin-bottom: lh(); margin-bottom: lh();
padding-left: 0; padding-left: 0;
...@@ -104,7 +104,7 @@ div.info-wrapper { ...@@ -104,7 +104,7 @@ div.info-wrapper {
} }
} }
h2.date { .date {
@extend %t-title9; @extend %t-title9;
margin-bottom: ($baseline/4); margin-bottom: ($baseline/4);
text-transform: none; text-transform: none;
...@@ -112,6 +112,7 @@ div.info-wrapper { ...@@ -112,6 +112,7 @@ div.info-wrapper {
@include padding-left($baseline); @include padding-left($baseline);
@include float(left); @include float(left);
} }
.toggle-visibility-button { .toggle-visibility-button {
@extend %t-title9; @extend %t-title9;
......
<%! from django.utils.translation import ugettext as _ %> <%! from django.utils.translation import ugettext as _ %>
<section aria-labelledby="course-updates-heading"> <div>
<h2 class="hd hd-2 sr" id="course-updates-heading">${_('All course updates')}</h2>
<div class="recent-updates"> <div class="recent-updates">
% for index, update in enumerate(visible_updates): % for index, update in enumerate(visible_updates):
<article class="updates-article"> <article class="updates-article">
% if not update.get("is_error"): % if not update.get("is_error"):
<h2 class="date" id="msg-date-${index}">${update.get("date")}</h2> <div class="date" id="msg-date-${index}">${update.get("date")}</div>
<button <button
class="toggle-visibility-button" class="toggle-visibility-button"
data-hide="${_('Hide')}" data-hide="${_('Hide')}"
...@@ -16,7 +15,7 @@ ...@@ -16,7 +15,7 @@
></button> ></button>
% endif % endif
<div class="toggle-visibility-element article-content ${'hidden' if index >= 1 else ''}" id="msg-content-${index}"> <div class="toggle-visibility-element article-content ${'hidden' if index >= 1 else ''}" id="msg-content-${index}">
${update.get("content")} ${update.get("content")}
</div> </div>
</article> </article>
% endfor % endfor
...@@ -35,7 +34,7 @@ ...@@ -35,7 +34,7 @@
<div class="old-updates hidden toggle-visibility-element" id="old-updates"> <div class="old-updates hidden toggle-visibility-element" id="old-updates">
% for index, update in enumerate(hidden_updates): % for index, update in enumerate(hidden_updates):
<article class="updates-article"> <article class="updates-article">
<h2 class="date" id="msg-date-${index + len(visible_updates)}">${update.get("date")}</h2> <div class="date" id="msg-date-${index + len(visible_updates)}">${update.get("date")}</div>
<button <button
class="toggle-visibility-button" class="toggle-visibility-button"
data-hide="${_('Hide')}" data-hide="${_('Hide')}"
...@@ -48,4 +47,4 @@ ...@@ -48,4 +47,4 @@
</article> </article>
% endfor % endfor
</div> </div>
</section> <div>
...@@ -60,8 +60,9 @@ from openedx.core.djangolib.markup import HTML, Text ...@@ -60,8 +60,9 @@ from openedx.core.djangolib.markup import HTML, Text
<div class="container"> <div class="container">
<div class="home"> <div class="home">
<div class="page-header-main"> <div class="page-header-main">
<h1 class="page-title">${_("Welcome to {org}'s {course_name}!").format(org=course.display_org_with_default, course_name=course.display_number_with_default)}</h1> <h3 class="page-title">${_("Welcome to {org}'s {course_name}!").format(org=course.display_org_with_default, course_name=course.display_number_with_default)}
<h2 class="page-subtitle">${course.display_name_with_default}</h2> <div class="page-subtitle">${course.display_name_with_default}</div>
</h3>
</div> </div>
% if last_accessed_courseware_url: % if last_accessed_courseware_url:
<div class="page-header-secondary"> <div class="page-header-secondary">
...@@ -80,7 +81,7 @@ from openedx.core.djangolib.markup import HTML, Text ...@@ -80,7 +81,7 @@ from openedx.core.djangolib.markup import HTML, Text
</div> </div>
% endif % endif
<h1>${_("Course Updates and News")}</h1> <h4>${_("Course Updates and News")}</h4>
${HTML(get_course_info_section(request, masquerade_user, course, 'updates'))} ${HTML(get_course_info_section(request, masquerade_user, course, 'updates'))}
## CourseTalk widget ## CourseTalk widget
...@@ -93,7 +94,7 @@ from openedx.core.djangolib.markup import HTML, Text ...@@ -93,7 +94,7 @@ from openedx.core.djangolib.markup import HTML, Text
<section aria-label="${_('Handout Navigation')}" class="handouts"> <section aria-label="${_('Handout Navigation')}" class="handouts">
% if SelfPacedConfiguration.current().enable_course_home_improvements: % if SelfPacedConfiguration.current().enable_course_home_improvements:
<h1 class="handouts-header">${_("Important Course Dates")}</h1> <h4 class="handouts-header">${_("Important Course Dates")}</h1>
## Should be organized by date, last date appearing at the bottom ## Should be organized by date, last date appearing at the bottom
% for course_date in get_course_date_blocks(course, user): % for course_date in get_course_date_blocks(course, user):
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment