Commit fb013272 by Renzo Lucioni

Wrap full name placeholder in quotes to deal with spaces

Also fix failing JS test which was looking for a removed button.
parent e9e1d350
...@@ -86,16 +86,10 @@ define([ ...@@ -86,16 +86,10 @@ define([
}; };
var expectPaymentDisabledBecauseInactive = function() { var expectPaymentDisabledBecauseInactive = function() {
var payButton = $( '#pay_button'), var payButton = $( '#pay_button' );
activateButton = $( '#activate_button' );
// Payment button should be hidden // Payment button should be hidden
expect( payButton.length ).toEqual(0); expect( payButton.length ).toEqual(0);
// Activate button should be displayed and disabled
expect( activateButton.length ).toEqual(1);
expect( activateButton.hasClass( 'is-disabled' ) ).toBe( true );
expect( activateButton.prop( 'disabled' ) ).toBe( true );
}; };
var goToPayment = function( requests, kwargs ) { var goToPayment = function( requests, kwargs ) {
......
...@@ -38,11 +38,6 @@ var edx = edx || {}; ...@@ -38,11 +38,6 @@ var edx = edx || {};
// Set the payment button to disabled by default // Set the payment button to disabled by default
this.setPaymentEnabled( false ); this.setPaymentEnabled( false );
// The activate button is always disabled
$( '#activate_button' )
.addClass( 'is-disabled' )
.prop( 'disabled', true );
// Update the contribution amount with the amount the user // Update the contribution amount with the amount the user
// selected in a previous screen. // selected in a previous screen.
if ( templateContext.contributionAmount ) { if ( templateContext.contributionAmount ) {
......
...@@ -35,7 +35,7 @@ ...@@ -35,7 +35,7 @@
<div class="copy expandable-area"> <div class="copy expandable-area">
<p><%- gettext( "You should change the name on your account to match your ID." ) %></p> <p><%- gettext( "You should change the name on your account to match your ID." ) %></p>
<input type="text" name="new-name" id="new-name" placeholder=<%= fullName %>> <input type="text" name="new-name" id="new-name" placeholder="<%= fullName %>">
</div> </div>
</div> </div>
</li> </li>
......
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