Studio: addressing course re-run PR feedback

cms/static/js/views/course_rerun.js

@@ -52,7 +52,7 @@ require(["domReady", "jquery", "underscore", "js/utils/cancel_on_escape"],
             );
             // Go into creating re-run state
             $('.rerun-course-save').addClass('is-disabled').addClass('is-processing').html(
-               '<i class="icon-refresh icon-spin"></i>' + gettext('Processing Re-run Request')
+               '<i class="icon icon-refresh icon-spin"></i>' + gettext('Processing Re-run Request')
             );
             $('.action-cancel').addClass('is-hidden');
         };
@@ -176,4 +176,4 @@ require(["domReady", "jquery", "underscore", "js/utils/cancel_on_escape"],
                 }
             });
         });
-    });
\ No newline at end of file
+    });

cms/static/sass/views/_course-create.scss

@@ -73,6 +73,12 @@
 
     .rerun-course-save {
       @include blue-button;
+
+      .icon {
+        display: inline-block;
+        vertical-align: middle;
+        margin-right: ($baseline/4);
+      }
     }
 
     .rerun-course-cancel {

cms/static/sass/views/_dashboard.scss

@@ -236,7 +236,7 @@
         @extend %t-strong;
         margin: ($baseline/2);
 
-        [class^="icon-"] {
+        .icon {
           margin-right: ($baseline/4);
         }
       }
@@ -302,6 +302,14 @@
       padding-bottom: ($baseline/2);
       color: $gray-l2;
     }
+
+    .title {
+      @extend %t-title6;
+      margin-bottom: $baseline;
+      border-bottom: 1px solid $gray-l3;
+      padding-bottom: ($baseline/2);
+      color: $gray-l2;
+    }
 	}
 
   .list-courses {
@@ -333,12 +341,12 @@
 
         .value {
 
-          .copy, *[class^="icon"] {
+          .copy, .icon {
             display: inline-block;
             vertical-align: middle;
           }
 
-          *[class^="icon"] {
+          .icon {
             @extend %t-icon4;
             margin-right: ($baseline/2);
           }

cms/templates/index.html

@@ -162,7 +162,7 @@ require(["domReady!", "jquery", "jquery.form", "js/index"], function(doc, $) {
                 <dl class="course-status">
                   <dt class="label sr">This re-run processing status:</dt>
                   <dd class="value">
-                    <i class="icon-refresh icon-spin"></i>
+                    <i class="icon icon-refresh icon-spin"></i>
                     <span class="copy">Configuring as re-run</span>
                   </dd>
                 </dl>
@@ -200,7 +200,7 @@ require(["domReady!", "jquery", "jquery.form", "js/index"], function(doc, $) {
                 <dl class="course-status">
                   <dt class="label sr">This re-run processing status:</dt>
                   <dd class="value">
-                    <i class="icon-warning-sign"></i>
+                    <i class="icon icon-warning-sign"></i>
                     <span class="copy">Configuration Error</span>
                   </dd>
                 </dl>

cms/templates/ux/reference/course-create-rerun.html

@@ -328,7 +328,7 @@ TODO:
 
               <div class="actions">
                 <button type="submit" class="action action-primary rerun-course-save is-disabled is-processing">
-                  <i class="icon-refresh icon-spin"></i>
+                  <i class="icon icon-refresh icon-spin"></i>
                   ${_('Processing Re-run Request')}
                 </button>
               </div>