Commit 1d5a310f by Ahsan Ulhaq

Continue to Payment button is not accessible

When a user starts the payment/verification flow.
Tab navigation skips past the "Continue to Payment" button

ECOM-1302
parent f0c52e32
...@@ -103,9 +103,10 @@ define([ ...@@ -103,9 +103,10 @@ define([
expect($el.length).toEqual(_.size(buttons)); expect($el.length).toEqual(_.size(buttons));
_.each(buttons, function( expectedText, expectedId ) { _.each(buttons, function( expectedText, expectedId ) {
var buttonEl = $( '#' + expectedId ); var buttonEl = $( '#' + expectedId );
buttonEl.removeAttr('disabled');
expect( buttonEl.length ).toEqual( 1 ); expect( buttonEl.length ).toEqual( 1 );
expect( buttonEl[0] ).toHaveClass( 'payment-button' ); expect( buttonEl[0] ).toHaveClass( 'payment-button' );
expect( buttonEl[0].text ).toEqual( expectedText ); expect( buttonEl[0] ).toHaveText( expectedText );
buttonEl[0].click(); buttonEl[0].click();
expect(requests[requests.length - 1].requestBody.split('&')).toContain('processor=' + expectedId); expect(requests[requests.length - 1].requestBody.split('&')).toContain('processor=' + expectedId);
}); });
......
...@@ -58,7 +58,7 @@ var edx = edx || {}; ...@@ -58,7 +58,7 @@ var edx = edx || {};
_getPaymentButtonHtml: function(processorName) { _getPaymentButtonHtml: function(processorName) {
var self = this; var self = this;
return _.template( return _.template(
'<a class="next action-primary payment-button" id="<%- name %>" tab-index="0"><%- text %></a> ' '<button class="next action-primary payment-button" id="<%- name %>" ><%- text %></button> '
)({name: processorName, text: self._getPaymentButtonText(processorName)}); )({name: processorName, text: self._getPaymentButtonText(processorName)});
}, },
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment