diff --git a/lms/djangoapps/courseware/views/views.py b/lms/djangoapps/courseware/views/views.py index 79d2976d2c97..0ece88777360 100644 --- a/lms/djangoapps/courseware/views/views.py +++ b/lms/djangoapps/courseware/views/views.py @@ -276,14 +276,14 @@ def courses(request): """ courses_list = [] course_discovery_meanings = getattr(settings, 'COURSE_DISCOVERY_MEANINGS', {}) - if not settings.FEATURES.get('ENABLE_COURSE_DISCOVERY'): - courses_list = get_courses(request.user) + # if not settings.FEATURES.get('ENABLE_COURSE_DISCOVERY'): + courses_list = get_courses(request.user) - if configuration_helpers.get_value("ENABLE_COURSE_SORTING_BY_START_DATE", - settings.FEATURES["ENABLE_COURSE_SORTING_BY_START_DATE"]): - courses_list = sort_by_start_date(courses_list) - else: - courses_list = sort_by_announcement(courses_list) + if configuration_helpers.get_value("ENABLE_COURSE_SORTING_BY_START_DATE", + settings.FEATURES["ENABLE_COURSE_SORTING_BY_START_DATE"]): + courses_list = sort_by_start_date(courses_list) + else: + courses_list = sort_by_announcement(courses_list) # Add marketable programs to the context. programs_list = get_programs_with_type(request.site, include_hidden=False) diff --git a/lms/envs/common.py b/lms/envs/common.py index c0182d47355a..a4c7683c5145 100644 --- a/lms/envs/common.py +++ b/lms/envs/common.py @@ -657,7 +657,7 @@ # .. toggle_target_removal_date: None # .. toggle_warning: The COURSE_DISCOVERY_MEANINGS setting should be properly defined. # .. toggle_tickets: https://github.com/openedx/edx-platform/pull/7845 - 'ENABLE_COURSE_DISCOVERY': False, + 'ENABLE_COURSE_DISCOVERY': True, # .. toggle_name: FEATURES['ENABLE_COURSE_FILENAME_CCX_SUFFIX'] # .. toggle_implementation: DjangoSetting diff --git a/lms/envs/devstack.py b/lms/envs/devstack.py index c2454ee71fc2..2c8b3cf7b63c 100644 --- a/lms/envs/devstack.py +++ b/lms/envs/devstack.py @@ -198,7 +198,7 @@ def should_show_debug_toolbar(request): # lint-amnesty, pylint: disable=missing 'language': LANGUAGE_MAP, } -FEATURES['ENABLE_COURSE_DISCOVERY'] = False +FEATURES['ENABLE_COURSE_DISCOVERY'] = True # Setting for overriding default filtering facets for Course discovery # COURSE_DISCOVERY_FILTERS = ["org", "language", "modes"] FEATURES['COURSES_ARE_BROWSEABLE'] = True diff --git a/lms/static/images/filter_icon.png b/lms/static/images/filter_icon.png new file mode 100644 index 000000000000..c67379346c00 Binary files /dev/null and b/lms/static/images/filter_icon.png differ diff --git a/lms/static/images/magnifier.png b/lms/static/images/magnifier.png new file mode 100644 index 000000000000..b94d6dac45c7 Binary files /dev/null and b/lms/static/images/magnifier.png differ diff --git a/lms/static/sass/discussion/views/_search.scss b/lms/static/sass/discussion/views/_search.scss index aa836067237a..b19fa5429050 100644 --- a/lms/static/sass/discussion/views/_search.scss +++ b/lms/static/sass/discussion/views/_search.scss @@ -17,14 +17,14 @@ margin-top: 0; } - > label { - position: absolute; - display: block; - color: #495057; - transition: all 0.1s ease-in-out; - padding-left: 10px; - padding-top: 10px; - } + // > label { + // position: absolute; + // display: block; + // color: #495057; + // transition: all 0.1s ease-in-out; + // padding-left: 10px; + // padding-top: 10px; + // } input::placeholder { color: transparent; diff --git a/lms/static/sass/multicourse/_courses.scss b/lms/static/sass/multicourse/_courses.scss index 9da6d4b7e164..9a4acf4bf053 100644 --- a/lms/static/sass/multicourse/_courses.scss +++ b/lms/static/sass/multicourse/_courses.scss @@ -286,7 +286,7 @@ $facet-background-color: #007db8; @include border-radius(3px); @include padding-right(55px); - border: 2px solid $gray-l3; + border: none; width: 100%; height: $course-search-input-height; color: $black; diff --git a/lms/templates/courseware/courses.html b/lms/templates/courseware/courses.html index db303073676d..1bef30b2d4a4 100644 --- a/lms/templates/courseware/courses.html +++ b/lms/templates/courseware/courses.html @@ -1,7 +1,7 @@ <%page expression_filter="h"/> <%! import json - from django.utils.translation import ugettext as _ + from django.utils.translation import gettext as _ from openedx.core.djangolib.js_utils import js_escaped_string, dump_js_escaped_json %> <%inherit file="../main.html" /> @@ -35,7 +35,7 @@
% if course_discovery_enabled: -