\ No newline at end of file
diff --git a/lms/static/sass/features/_course-experience.scss b/lms/static/sass/features/_course-experience.scss
index 70b7789846..0d3e0544cf 100644
--- a/lms/static/sass/features/_course-experience.scss
+++ b/lms/static/sass/features/_course-experience.scss
@@ -457,9 +457,10 @@ button.accordion-trigger, button.prerequisite-button {
.date-summary-container {
.date-summary {
@include clearfix;
- @include border-left(3px solid $gray-300);
- padding: $baseline/2;
+ display: flex;
+ justify-content: space-between;
+ padding: $baseline/2 $baseline/2 $baseline/2 0;
.heading {
font-size: 0.9rem;
@@ -469,18 +470,38 @@ button.accordion-trigger, button.prerequisite-button {
color: theme-color("dark");
}
- .description {
- margin-top: $baseline/2;
- margin-bottom: $baseline/2;
- display: inline-block;
+ .left-column {
+ flex: 5%;
+
+ .calendar-icon {
+ margin-top: 3px;
+ height: 1em;
+ width: auto;
+ background: url('#{$static-path}/images/calendar-alt-regular.svg');
+ background-repeat: no-repeat;
+ }
}
- .date-summary-link {
- font-weight: $font-semibold;
+ .right-column {
+ flex: 85%;
- a {
- color: $link-color;
- font-weight: $font-regular;
+ .localized-datetime {
+ font-weight: $font-weight-bold;
+ margin-bottom: 8px;
+ }
+
+ .description {
+ margin-bottom: $baseline/2;
+ display: inline-block;
+ }
+
+ .date-summary-link {
+ font-weight: $font-semibold;
+
+ a {
+ color: $link-color;
+ font-weight: $font-regular;
+ }
}
}
diff --git a/openedx/features/course_experience/templates/course_experience/course-dates-fragment.html b/openedx/features/course_experience/templates/course_experience/course-dates-fragment.html
index 1339148c24..2464682db7 100644
--- a/openedx/features/course_experience/templates/course_experience/course-dates-fragment.html
+++ b/openedx/features/course_experience/templates/course_experience/course-dates-fragment.html
@@ -7,12 +7,13 @@
<%!
from django.utils.translation import ugettext as _
%>
-${_("Important Course Dates")}
-## Should be organized by date, last date appearing at the bottom
-
-% for course_date in course_date_blocks:
- <%include file="dates-summary.html" args="course_date=course_date" />
-% endfor
+% if len(course_date_blocks) > 0:
+ ${_("Upcoming Dates")}
+ ## Should be organized by date, last date appearing at the bottom
+ % for course_date_block in course_date_blocks:
+ <%include file="dates-summary.html" args="course_date=course_date_block" />
+ % endfor
+% endif
<%static:require_module_async module_name="js/dateutil_factory" class_name="DateUtilFactory">
DateUtilFactory.transform('.localized-datetime');
diff --git a/openedx/features/course_experience/templates/course_experience/dates-summary.html b/openedx/features/course_experience/templates/course_experience/dates-summary.html
index b13035bad0..36952923c8 100644
--- a/openedx/features/course_experience/templates/course_experience/dates-summary.html
+++ b/openedx/features/course_experience/templates/course_experience/dates-summary.html
@@ -4,23 +4,24 @@ from django.utils.translation import ugettext as _
<%page args="course_date" expression_filter="h"/>
- % if course_date.title:
- % if course_date.title == 'current_datetime':
-
- % else:
+
+
+ % if course_date.date:
+
+ % endif
+ % if course_date.title:
${course_date.title}
% endif
- % endif
- % if course_date.date and course_date.title != 'current_datetime':
-
- % endif
- % if course_date.description:
-
${course_date.description}
- % endif
- % if course_date.link and course_date.link_text:
-
- ${course_date.link_text}
-
- % endif
+ % if course_date.description:
+
${course_date.description}
+ % endif
+ % if course_date.link and course_date.link_text:
+
+ ${course_date.link_text}
+
+ % endif
+
diff --git a/openedx/features/course_experience/tests/views/test_course_dates.py b/openedx/features/course_experience/tests/views/test_course_dates.py
index 35a6905416..db81ed41b6 100644
--- a/openedx/features/course_experience/tests/views/test_course_dates.py
+++ b/openedx/features/course_experience/tests/views/test_course_dates.py
@@ -41,7 +41,6 @@ class TestCourseDatesFragmentView(ModuleStoreTestCase):
def test_course_dates_fragment(self):
response = self.client.get(self.dates_fragment_url)
- self.assertContains(response, 'Today is')
self.assertContains(response, 'Course End')
self.client.logout()
diff --git a/openedx/features/course_experience/tests/views/test_course_home.py b/openedx/features/course_experience/tests/views/test_course_home.py
index e9301a1c84..1b3309b231 100644
--- a/openedx/features/course_experience/tests/views/test_course_home.py
+++ b/openedx/features/course_experience/tests/views/test_course_home.py
@@ -70,7 +70,6 @@ from .test_course_updates import create_course_update, remove_course_updates
TEST_PASSWORD = 'test'
TEST_CHAPTER_NAME = 'Test Chapter'
TEST_COURSE_TOOLS = 'Course Tools'
-TEST_COURSE_TODAY = 'Today is'
TEST_BANNER_CLASS = ''
TEST_WELCOME_MESSAGE = '
Welcome!
'
TEST_UPDATE_MESSAGE = 'Test Update!
'
@@ -311,7 +310,6 @@ class TestCourseHomePageAccess(CourseHomePageTestCase):
# Verify that the course tools and dates are always shown
self.assertContains(response, TEST_COURSE_TOOLS)
- self.assertContains(response, TEST_COURSE_TODAY)
is_anonymous = user_type is CourseUserType.ANONYMOUS
is_enrolled = user_type is CourseUserType.ENROLLED
@@ -365,7 +363,6 @@ class TestCourseHomePageAccess(CourseHomePageTestCase):
# Verify that the course tools and dates are always shown
self.assertContains(response, TEST_COURSE_TOOLS)
- self.assertContains(response, TEST_COURSE_TODAY)
# Verify that welcome messages are never shown
self.assertNotContains(response, TEST_WELCOME_MESSAGE)
@@ -639,7 +636,6 @@ class TestCourseHomePageAccess(CourseHomePageTestCase):
response = self.client.get(course_home_url(audit_only_course))
self.assertEqual(response.status_code, 200)
self.assertContains(response, TEST_COURSE_TOOLS)
- self.assertContains(response, TEST_COURSE_TODAY)
self.assertNotContains(response, TEST_BANNER_CLASS)
@mock.patch.dict(settings.FEATURES, {'DISABLE_START_DATES': False})
diff --git a/openedx/features/course_experience/views/course_dates.py b/openedx/features/course_experience/views/course_dates.py
index 72dca8fa61..204a273494 100644
--- a/openedx/features/course_experience/views/course_dates.py
+++ b/openedx/features/course_experience/views/course_dates.py
@@ -28,7 +28,7 @@ class CourseDatesFragmentView(EdxFragmentView):
course_date_blocks = get_course_date_blocks(course, request.user)
context = {
- 'course_date_blocks': course_date_blocks
+ 'course_date_blocks': [block for block in course_date_blocks if block.title != 'current_datetime']
}
html = render_to_string(self.template_name, context)
dates_fragment = Fragment(html)