style: fix some styling

client/app/bundles/course/assessment/containers/AssessmentForm/index.jsx

@@ -24,7 +24,6 @@ const styles = {
   },
   flexChild: {
     flex: 1,
-    marginLeft: 10,
   },
   toggle: {
     marginTop: 16,

client/app/bundles/course/assessment/question/programming/components/OnlineEditor/TestCase.jsx

@@ -72,6 +72,9 @@ class TestCase extends Component {
         error={!!test.getIn(['error', field])}
         fullWidth
         helperText={test.getIn(['error', field]) && placeholder}
+        InputLabelProps={{
+          shrink: true,
+        }}
         multiline
         name={TestCase.getTestInputName(this.props.type, field)}
         onChange={(event) => {

client/app/bundles/course/assessment/question/programming/containers/ProgrammingQuestionForm/ProgrammingQuestionForm.jsx

@@ -383,6 +383,9 @@ class ProgrammingQuestionForm extends Component {
           label={(required ? '* ' : '') + label}
           helperText={error}
           id={ProgrammingQuestionForm.getInputId(field)}
+          InputLabelProps={{
+            shrink: true,
+          }}
           name={ProgrammingQuestionForm.getInputName(field)}
           onChange={(event) => {
             this.handleChange(field, event.target.value);

client/app/bundles/course/assessment/submission/containers/PastAnswers.jsx

@@ -23,8 +23,8 @@ import { questionTypes } from '../constants';
 
 const styles = {
   horizontalRule: {
-    marginTop: 40,
-    marginBottom: 40,
+    marginTop: 20,
+    marginBottom: 20,
   },
 };
 

client/app/bundles/course/assessment/submission/pages/SubmissionEditIndex/SubmissionEditStepForm.jsx

@@ -59,7 +59,7 @@ const styles = {
   },
   contineButton: {
     backgroundColor: green[500],
-    labelColor: 'white',
+    color: 'white',
     marginBottom: 10,
     marginRight: 10,
   },
@@ -177,16 +177,16 @@ class SubmissionEditStepForm extends Component {
 
   renderContinueButton() {
     const { intl } = this.props;
+    const disabled = this.shouldDisableContinueButton();
     if (this.shouldRenderContinueButton()) {
       return (
         <Button
           variant="contained"
-          disabled={this.shouldDisableContinueButton()}
+          disabled={disabled}
           onClick={() => this.handleNext()}
           style={{
             ...styles.formButton,
-            backgroundColor: green[500],
-            color: 'white',
+            ...(!disabled && styles.contineButton),
           }}
         >
           {intl.formatMessage(translations.continue)}

client/app/bundles/course/duplication/components/CourseDropdownMenu.jsx

@@ -67,7 +67,7 @@ class CourseDropdownMenu extends PureComponent {
           <Select
             disabled={disabled}
             onChange={onChange}
-            value={selectedCourseId}
+            value={selectedCourseId || ''}
             style={styles.dropDown}
             {...dropDownMenuProps}
           >

client/app/bundles/course/group/components/GroupCard.jsx

@@ -168,8 +168,12 @@ GroupCard.propTypes = {
   titleButtons: PropTypes.arrayOf(groupCardTitleButtonShape),
   bottomButtons: PropTypes.arrayOf(groupCardBottomButtonShape),
   className: PropTypes.string,
-  children: PropTypes.oneOfType([PropTypes.array, PropTypes.element])
-    .isRequired,
+  children: PropTypes.oneOfType([
+    PropTypes.array,
+    PropTypes.element,
+    PropTypes.string,
+    PropTypes.object,
+  ]).isRequired,
 };
 
 export default GroupCard;

client/app/bundles/course/survey/pages/SurveyShow/Section/QuestionCard.jsx

@@ -43,6 +43,7 @@ const styles = {
   },
   fields: {
     marginTop: 0,
+    paddingTop: 0,
   },
   panelSummaryText: {
     flexDirection: 'column',
@@ -189,7 +190,7 @@ class QuestionCard extends Component {
             ) : null}
           </div>
         </AccordionSummary>
-        <CardContent style={styles.field}>
+        <CardContent style={styles.fields}>
           {QuestionCard.renderSpecificFields(question)}
         </CardContent>
       </Accordion>

client/app/bundles/course/survey/pages/SurveyShow/SurveyDetails.jsx

@@ -31,6 +31,10 @@ const styles = {
   button: {
     marginRight: 15,
   },
+  toggleContainer: {
+    paddingTop: 0,
+    paddingBottom: 0,
+  },
 };
 
 class SurveyDetails extends Component {
@@ -79,7 +83,7 @@ class SurveyDetails extends Component {
     }
 
     return (
-      <CardContent>
+      <CardContent style={styles.toggleContainer}>
         <FormControlLabel
           control={
             <Switch

client/app/lib/components/course/ConditionList/index.jsx

@@ -80,7 +80,7 @@ class ConditionList extends Component {
         <TableCell colSpan="3">{condition.description}</TableCell>
         <TableCell colSpan="2" style={styles.alignRight}>
           <IconButton href={condition.edit_url}>
-            <Edit htmlColor="black" />
+            <Edit htmlColor="black" style={{ padding: 0 }} />
           </IconButton>
 
           <IconButton
@@ -93,7 +93,7 @@ class ConditionList extends Component {
             style={styles.alignMiddle}
             id={condition.delete_url}
           >
-            <Delete htmlColor="black" />
+            <Delete htmlColor="black" style={{ padding: 0 }} />
           </IconButton>
         </TableCell>
       </TableRow>