diff --git a/common/lib/xmodule/xmodule/css/capa/display.scss b/common/lib/xmodule/xmodule/css/capa/display.scss index f5136ef1f9..e5a0a373a5 100644 --- a/common/lib/xmodule/xmodule/css/capa/display.scss +++ b/common/lib/xmodule/xmodule/css/capa/display.scss @@ -953,7 +953,6 @@ div.problem { // +Problem - Actions // ==================== div.problem .action { - margin-top: $baseline; min-height: $baseline; width: 100%; display: flex; @@ -963,6 +962,13 @@ div.problem .action { align-items: center; flex-wrap: wrap; + .problem-action-buttons-wrapper { + display: inline-flex; + justify-content: flex-end; + width: 100%; + padding-bottom: $baseline; + } + .problem-action-button-wrapper { @include border-right(1px solid $light-gray1); @@ -977,6 +983,8 @@ div.problem .action { @include margin-right($baseline / 5); max-width: 110px; + text-decoration: underline; + text-transform: none; .icon { margin-bottom: $baseline / 10; @@ -1009,13 +1017,12 @@ div.problem .action { display: inline-block; } .submit-cta-description { - margin-left: 8px; + color: $blue; + font-size: small; } .submit-cta-link-button { - background: none; - border: none; - color: $blue; - cursor: pointer; + text-decoration: underline; + text-transform: none; } } diff --git a/lms/djangoapps/courseware/tests/test_masquerade.py b/lms/djangoapps/courseware/tests/test_masquerade.py index a8d857be29..bd4a8437ad 100644 --- a/lms/djangoapps/courseware/tests/test_masquerade.py +++ b/lms/djangoapps/courseware/tests/test_masquerade.py @@ -150,11 +150,11 @@ class MasqueradeTestCase(SharedModuleStoreTestCase, LoginEnrollmentTestCase, Mas def verify_show_answer_present(self, show_answer_expected): """ - Verifies that "Show Answer" is only present when expected (for staff only). + Verifies that "Show answer" is only present when expected (for staff only). """ problem_html = json.loads(self.get_problem().content.decode('utf-8'))['html'] self.assertIn(self.problem_display_name, problem_html) - self.assertEqual(show_answer_expected, "Show Answer" in problem_html) + self.assertEqual(show_answer_expected, "Show answer" in problem_html) def ensure_masquerade_as_group_member(self, partition_id, group_id): """ diff --git a/lms/static/sass/course/layout/_banner_cta.scss b/lms/static/sass/course/layout/_banner_cta.scss index b87454fa2e..bad337c9c2 100644 --- a/lms/static/sass/course/layout/_banner_cta.scss +++ b/lms/static/sass/course/layout/_banner_cta.scss @@ -49,7 +49,7 @@ &.has-button { .banner-cta-text { flex: 1 1 20em; - max-width: 70%; + max-width: 100%; } } @@ -61,6 +61,12 @@ .banner-cta-button { align-self: start; flex: none; + margin-top: $baseline; + + @media only screen and (min-width: 630px) { + margin-top: 0px; + margin-left: 10px; + } a { text-decoration: none; diff --git a/lms/templates/problem.html b/lms/templates/problem.html index 5a6e44987a..3c6b5e17b5 100644 --- a/lms/templates/problem.html +++ b/lms/templates/problem.html @@ -26,6 +26,31 @@ from openedx.core.djangolib.markup import HTML % endif +
+ % if demand_hint_possible: + + + + % endif + % if save_button: + + + + % endif + % if reset_button: + + + + % endif + % if answer_available: + + + + % endif +
+ + + + (${submit_disabled_cta['description']}) % else:
% for form_name, form_value in submit_disabled_cta['form_values'].items(): % endfor - + + + + (${submit_disabled_cta['description']})
% endif % endif @@ -65,32 +92,6 @@ from openedx.core.djangolib.markup import HTML ${_("Some problems have options such as save, reset, hints, or show answer. These options follow the Submit button.")}
-
- % if demand_hint_possible: - - - - % endif - % if save_button: - - - - % endif - % if reset_button: - - - - % endif - % if answer_available: - - - - % endif -
<%include file="problem_notifications.html" args=" notification_type='warning',