From bb3ac95f24bd2d6634e0d7a7ca2748bd2721500b Mon Sep 17 00:00:00 2001
From: Chris Rodriguez <github@chrisrodriguez.me>
Date: Mon, 30 Nov 2015 12:34:10 -0500
Subject: [PATCH 1/2] Fix: correcting play/skip icon for bumper

---
 .../xmodule/xmodule/js/src/video/09_play_skip_control.js  | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/common/lib/xmodule/xmodule/js/src/video/09_play_skip_control.js b/common/lib/xmodule/xmodule/js/src/video/09_play_skip_control.js
index d4759580e6..d33cb2174b 100644
--- a/common/lib/xmodule/xmodule/js/src/video/09_play_skip_control.js
+++ b/common/lib/xmodule/xmodule/js/src/video/09_play_skip_control.js
@@ -25,9 +25,9 @@ define('video/09_play_skip_control.js', [], function() {
 
     PlaySkipControl.prototype = {
         template: [
-            '<button class="control video_control play play-skip-control" aria-disabled="false">',
+            '<button class="control video_control play play-skip-control">',
                 '<span class="icon-fallback-img">',
-                    '<span class="icon icon-play" aria-hidden="true"></span>',
+                    '<span class="icon fa fa-play" aria-hidden="true"></span>',
                     '<span class="text control-text">',
                         gettext('Play'),
                     '</span>',
@@ -79,8 +79,8 @@ define('video/09_play_skip_control.js', [], function() {
                 .removeClass('play')
                 .addClass('skip')
                 .find('.icon')
-                    .removeClass('icon-play')
-                    .addClass('icon-step-forward')
+                    .removeClass('fa-play')
+                    .addClass('fa-step-forward')
                 .find('.control-text')
                     .text(gettext('Skip'));
             // Disable possibility to pause the video.

From 4161ae99802131f1b23cce145a183e31a19179cb Mon Sep 17 00:00:00 2001
From: Chris Rodriguez <github@chrisrodriguez.me>
Date: Mon, 30 Nov 2015 13:24:58 -0500
Subject: [PATCH 2/2] Tests

---
 .../xmodule/js/spec/video/video_play_skip_control_spec.js  | 6 ------
 .../xmodule/js/spec/video/video_skip_control_spec.js       | 7 -------
 2 files changed, 13 deletions(-)

diff --git a/common/lib/xmodule/xmodule/js/spec/video/video_play_skip_control_spec.js b/common/lib/xmodule/xmodule/js/spec/video/video_play_skip_control_spec.js
index 725bdd5c8a..7e15b597af 100644
--- a/common/lib/xmodule/xmodule/js/spec/video/video_play_skip_control_spec.js
+++ b/common/lib/xmodule/xmodule/js/spec/video/video_play_skip_control_spec.js
@@ -25,12 +25,6 @@
             expect($('.video_control.play')).toExist();
         });
 
-        it('add ARIA attributes to play control', function () {
-            expect($('.video_control.play')).toHaveAttrs({
-                'aria-disabled': 'false'
-            });
-        });
-
         it('can update state on play', function () {
             state.el.trigger('play');
             expect($('.video_control.play')).not.toExist();
diff --git a/common/lib/xmodule/xmodule/js/spec/video/video_skip_control_spec.js b/common/lib/xmodule/xmodule/js/spec/video/video_skip_control_spec.js
index d3c0b47d52..1eca712477 100644
--- a/common/lib/xmodule/xmodule/js/spec/video/video_skip_control_spec.js
+++ b/common/lib/xmodule/xmodule/js/spec/video/video_skip_control_spec.js
@@ -30,13 +30,6 @@
             expect($('.skip-control')).toExist();
         });
 
-        it('add ARIA attributes to play control', function () {
-            state.el.trigger('play');
-            expect($('.skip-control')).toHaveAttrs({
-                'aria-disabled': 'false'
-            });
-        });
-
         it('can skip the video on click', function () {
             spyOn(state.bumperState.videoBumper, 'skipAndDoNotShowAgain');
             state.el.trigger('play');