From c10f1521dc508ee4f34db6a20d7107650feae2be Mon Sep 17 00:00:00 2001
From: Awais Ansari <79941147+awais-ansari@users.noreply.github.com>
Date: Tue, 8 Jun 2021 23:23:47 +0500
Subject: [PATCH] refactor: legacyConfigForm UI (#130)

* refactor: update legacyConfigForm UI and divider colour
---
 .../app-config-form/apps/legacy/LegacyConfigForm.jsx        | 6 +++---
 .../app-config-form/apps/shared/AppConfigFormDivider.jsx    | 2 +-
 .../discussions/app-config-form/apps/shared/messages.js     | 4 ++--
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/pages-and-resources/discussions/app-config-form/apps/legacy/LegacyConfigForm.jsx b/src/pages-and-resources/discussions/app-config-form/apps/legacy/LegacyConfigForm.jsx
index 8e2edd2c9..1e019ff68 100644
--- a/src/pages-and-resources/discussions/app-config-form/apps/legacy/LegacyConfigForm.jsx
+++ b/src/pages-and-resources/discussions/app-config-form/apps/legacy/LegacyConfigForm.jsx
@@ -71,15 +71,15 @@ function LegacyConfigForm({
               onChange={handleChange}
               values={values}
             />
-            <AppConfigFormDivider />
+            <AppConfigFormDivider thick />
+            <DiscussionTopics />
+            <AppConfigFormDivider thick />
             <DivisionByGroupFields
               onBlur={handleBlur}
               onChange={handleChange}
               appConfig={values}
             />
             <AppConfigFormDivider thick />
-            <DiscussionTopics />
-            <AppConfigFormDivider thick />
             <BlackoutDatesField
               errors={errors}
               onBlur={handleBlur}
diff --git a/src/pages-and-resources/discussions/app-config-form/apps/shared/AppConfigFormDivider.jsx b/src/pages-and-resources/discussions/app-config-form/apps/shared/AppConfigFormDivider.jsx
index 8eab7a5bf..c49a9ca6d 100644
--- a/src/pages-and-resources/discussions/app-config-form/apps/shared/AppConfigFormDivider.jsx
+++ b/src/pages-and-resources/discussions/app-config-form/apps/shared/AppConfigFormDivider.jsx
@@ -4,7 +4,7 @@ import PropTypes from 'prop-types';
 export default function AppConfigFormDivider({ thick }) {
   return (
     <hr
-      className="my-2 mx-sm-n5 mx-n4"
+      className="my-2 mx-sm-n5 mx-n4 border-light-300"
       style={{
         borderTopWidth: thick ? '3px' : '1px',
       }}
diff --git a/src/pages-and-resources/discussions/app-config-form/apps/shared/messages.js b/src/pages-and-resources/discussions/app-config-form/apps/shared/messages.js
index ebcfb04a8..4b8d3cfbf 100644
--- a/src/pages-and-resources/discussions/app-config-form/apps/shared/messages.js
+++ b/src/pages-and-resources/discussions/app-config-form/apps/shared/messages.js
@@ -18,12 +18,12 @@ const messages = defineMessages({
   },
   divideCourseTopicsByCohortsLabel: {
     id: 'authoring.discussions.builtIn.divideCourseTopicsByCohorts.label',
-    defaultMessage: 'Divide course wide discussion topics',
+    defaultMessage: 'Divide course-wide discussion topics',
     description: 'Label for a switch that enables dividing course wide topics by cohorts.',
   },
   divideCourseTopicsByCohortsHelp: {
     id: 'authoring.discussions.builtIn.divideCourseTopicsByCohorts.help',
-    defaultMessage: 'Choose which of your general course wide discussion topics you would like to divide.',
+    defaultMessage: 'Choose which of your general course-wide discussion topics you would like to divide.',
     description: 'Help text asking the user to pick course-wide topics that should be divided by cohort.',
   },
   divideGeneralTopic: {