Merge pull request #8817 from edx/zub/story/ecom-1648-update-credit-eligibility-ui-text

update credit eligibility ui text

cms/djangoapps/contentstore/views/tests/test_credit_eligibility.py

@@ -29,8 +29,8 @@ class CreditEligibilityTest(CourseTestCase):
         """
         response = self.client.get_html(self.course_details_url)
         self.assertEqual(response.status_code, 200)
-        self.assertNotContains(response, "Credit Eligibility Requirements")
-        self.assertNotContains(response, "Steps needed for credit eligibility")
+        self.assertNotContains(response, "Course Credit Requirements")
+        self.assertNotContains(response, "Steps required to earn course credit")
 
     @mock.patch.dict("django.conf.settings.FEATURES", {'ENABLE_CREDIT_ELIGIBILITY': True})
     def test_course_details_with_enabled_setting(self):
@@ -41,8 +41,8 @@ class CreditEligibilityTest(CourseTestCase):
         # course is not set as credit course
         response = self.client.get_html(self.course_details_url)
         self.assertEqual(response.status_code, 200)
-        self.assertNotContains(response, "Credit Eligibility Requirements")
-        self.assertNotContains(response, "Steps needed for credit eligibility")
+        self.assertNotContains(response, "Course Credit Requirements")
+        self.assertNotContains(response, "Steps required to earn course credit")
 
         # verify that credit eligibility requirements block shows if the
         # course is set as credit course and it has eligibility requirements
@@ -55,5 +55,5 @@ class CreditEligibilityTest(CourseTestCase):
 
         response = self.client.get_html(self.course_details_url)
         self.assertEqual(response.status_code, 200)
-        self.assertContains(response, "Credit Eligibility Requirements")
-        self.assertContains(response, "Steps needed for credit eligibility")
+        self.assertContains(response, "Course Credit Requirements")
+        self.assertContains(response, "Steps required to earn course credit")

cms/static/js/views/settings/main.js

@@ -54,8 +54,8 @@ var DetailsView = ValidatingView.extend({
 
         if (options.showMinGradeWarning || false) {
             new NotificationView.Warning({
-                title: gettext("Credit Eligibility Requirements"),
-                message: gettext("Minimum passing grade for credit is not set."),
+                title: gettext("Course Credit Requirements"),
+                message: gettext("The minimum grade for course credit is not set."),
                 closeIcon: true
             }).show();
         }

cms/static/sass/views/_settings.scss

@@ -97,6 +97,10 @@
     }
 
     // in form  -UI hints/tips/messages
+    .header-help {
+      margin: 0 0 $baseline 0;
+    }
+
     .instructions {
       @extend %t-copy-sub1;
       margin: 0 0 $baseline 0;

cms/templates/settings.html

@@ -112,8 +112,8 @@ CMS.URL.UPLOAD_ASSET = '${upload_asset_url}';
               <p>${_(
                 'Your course summary page will not be viewable until your course '
                 'has been announced. To provide content for the page and preview '
-                'it, follow the instructions provided by your Program Manager.')
-              }</p>
+                'it, follow the instructions provided by your Program Manager.')}
+              </p>
             </div>
           </div>
           % endif
@@ -123,16 +123,18 @@ CMS.URL.UPLOAD_ASSET = '${upload_asset_url}';
         % if credit_eligibility_enabled and is_credit_course:
           <section class="group-settings basic">
             <header>
-              <h2 class="title-2">${_("Credit Eligibility Requirements")}</h2>
-              <span class="tip">${_("Steps needed for credit eligibility")}</span>
+              <h2 class="title-2">${_("Course Credit Requirements")}</h2>
+              <span class="tip">${_("Steps required to earn course credit")}</span>
             </header>
+            <span class="header-help tip">A requirement appears in this list when you publish the unit that contains the requirement.</span>
+
               % if credit_requirements:
                 <ol class="list-input">
                   % if 'grade' in credit_requirements:
                     <li class="field text is-not-editable" id="credit-minimum-passing-grade">
-                      <label>${_("Minimum Passing Grade")}</label>
+                      <label>${_("Minimum Grade")}</label>
                         % for requirement in credit_requirements['grade']:
-                          <label for="${requirement['name']}" class="sr">${_("Minimum Passing Grade")}</label>
+                          <label for="${requirement['name']}" class="sr">${_("Minimum Grade")}</label>
                           <input title="${_('This field is disabled: this information cannot be changed.')}" type="text"
                               class="long" id="${requirement['name']}" value="${'{0:.0f}%'.format(float(requirement['criteria']['min_grade'] or 0)*100)}" readonly />
                         % endfor
@@ -152,12 +154,11 @@ CMS.URL.UPLOAD_ASSET = '${upload_asset_url}';
 
                   % if 'reverification' in credit_requirements:
                     <li class="field text is-not-editable" id="credit-reverification-requirements">
-                      <label>${_("Successful In-Course Reverification")}</label>
+                      <label>${_("ID Verification")}</label>
                         % for requirement in credit_requirements['reverification']:
-                          ## Translators: 'Access to Assessment 1' means the access for a requirement with name 'Assessment 1'
                           <label for="${requirement['name']}" class="sr">${_('In-Course Reverification {number}').format(number=loop.index+1)}</label>
                           <input title="${_('This field is disabled: this information cannot be changed.')}" type="text"
-                              class="long" id="${requirement['name']}" value="${_('Access to {display_name}').format(display_name=requirement['display_name'])}" readonly />
+                              class="long" id="${requirement['name']}" value="${requirement['display_name']}" readonly />
                         % endfor
                     </li>
                   % endif

cms/templates/settings_graders.html

@@ -79,15 +79,15 @@
         % if settings.FEATURES.get("ENABLE_CREDIT_ELIGIBILITY", False) and is_credit_course:
           <section class="group-settings grade-rules">
             <header>
-              <h2 class="title-2">${_("Credit Grade &amp; Eligibility")}</h2>
-              <span class="tip">${_("Settings for credit eligibility")}</span>
+              <h2 class="title-2">${_("Credit Eligibility")}</h2>
+              <span class="tip">${_("Settings for course credit eligibility")}</span>
             </header>
 
             <ol class="list-input">
               <li class="field text" id="field-course-minimum_grade_credit">
-                <label for="course-minimum_grade_credit">${_("Minimum Passing Grade to Earn Credit:")}</label>
+                <label for="course-minimum_grade_credit">${_("Minimum Credit-Eligible Grade:")}</label>
                 <input type="text" class="short time" id="course-minimum_grade_credit" value="0" placeholder="80%" autocomplete="off" aria-describedby="minimum_grade_description"/>
-                <span class="tip tip-inline" id="minimum_grade_description">${_("Must be greater than or equal to passing grade")}</span>
+                <span class="tip tip-inline" id="minimum_grade_description">${_("Must be greater than or equal to the course passing grade")}</span>
               </li>
             </ol>
           </section>