Commit be0a4ee7 by Will Daly

Merge pull request #5909 from edx/will/logistration-ab-test-css-classes

Logistration: Add CSS classes so we can compare clicks between the old and new login/registration pages
parents 5085b716 ed652bbc
...@@ -199,7 +199,7 @@ ...@@ -199,7 +199,7 @@
% endif % endif
<div class="form-actions"> <div class="form-actions">
<button name="submit" type="submit" id="submit" class="action action-primary action-update"></button> <button name="submit" type="submit" id="submit" class="action action-primary action-update login-button"></button>
</div> </div>
</form> </form>
...@@ -215,7 +215,7 @@ ...@@ -215,7 +215,7 @@
% for enabled in provider.Registry.enabled(): % for enabled in provider.Registry.enabled():
## Translators: provider_name is the name of an external, third-party user authentication provider (like Google or LinkedIn). ## Translators: provider_name is the name of an external, third-party user authentication provider (like Google or LinkedIn).
<button type="submit" class="button button-primary button-${enabled.NAME}" onclick="thirdPartySignin(event, '${pipeline_url[enabled.NAME]}');"><span class="icon ${enabled.ICON_CLASS}"></span>${_('Sign in with {provider_name}').format(provider_name=enabled.NAME)}</button> <button type="submit" class="button button-primary button-${enabled.NAME} login-${enabled.NAME}" onclick="thirdPartySignin(event, '${pipeline_url[enabled.NAME]}');"><span class="icon ${enabled.ICON_CLASS}"></span>${_('Sign in with {provider_name}').format(provider_name=enabled.NAME)}</button>
% endfor % endfor
</div> </div>
......
...@@ -125,7 +125,7 @@ ...@@ -125,7 +125,7 @@
% for enabled in provider.Registry.enabled(): % for enabled in provider.Registry.enabled():
## Translators: provider_name is the name of an external, third-party user authentication service (like Google or LinkedIn). ## Translators: provider_name is the name of an external, third-party user authentication service (like Google or LinkedIn).
<button type="submit" class="button button-primary button-${enabled.NAME}" onclick="thirdPartySignin(event, '${pipeline_urls[enabled.NAME]}');"><span class="icon ${enabled.ICON_CLASS}"></span>${_('Sign up with {provider_name}').format(provider_name=enabled.NAME)}</button> <button type="submit" class="button button-primary button-${enabled.NAME} register-${enabled.NAME}" onclick="thirdPartySignin(event, '${pipeline_urls[enabled.NAME]}');"><span class="icon ${enabled.ICON_CLASS}"></span>${_('Sign up with {provider_name}').format(provider_name=enabled.NAME)}</button>
% endfor % endfor
</div> </div>
...@@ -362,7 +362,7 @@ ...@@ -362,7 +362,7 @@
% endif % endif
<div class="form-actions"> <div class="form-actions">
<button name="submit" type="submit" id="submit" class="action action-primary action-update">${_('Register')} <span class="orn-plus">+</span> ${_('Create My Account')}</button> <button name="submit" type="submit" id="submit" class="action action-primary action-update register-button">${_('Register')} <span class="orn-plus">+</span> ${_('Create My Account')}</button>
</div> </div>
</form> </form>
</section> </section>
......
...@@ -14,12 +14,12 @@ ...@@ -14,12 +14,12 @@
<%= context.fields %> <%= context.fields %>
<button class="action action-primary action-update js-login"><%- gettext("Log in") %></button> <button class="action action-primary action-update js-login login-button"><%- gettext("Log in") %></button>
</form> </form>
<% _.each( context.providers, function( provider ) { <% _.each( context.providers, function( provider ) {
if ( provider.loginUrl ) { %> if ( provider.loginUrl ) { %>
<button type="submit" class="button button-primary button-<%- provider.name %> login-provider" data-provider-url="<%- provider.loginUrl %>"> <button type="submit" class="button button-primary button-<%- provider.name %> login-provider login-<%- provider.name %>" data-provider-url="<%- provider.loginUrl %>">
<span class="icon <%- provider.iconClass %>" aria-hidden="true"></span> <span class="icon <%- provider.iconClass %>" aria-hidden="true"></span>
<%- _.sprintf(gettext("Log in using %(name)s"), provider) %> <%- _.sprintf(gettext("Log in using %(name)s"), provider) %>
</button> </button>
......
...@@ -8,7 +8,7 @@ ...@@ -8,7 +8,7 @@
<% } else { <% } else {
_.each( context.providers, function( provider) { _.each( context.providers, function( provider) {
if ( provider.registerUrl ) { %> if ( provider.registerUrl ) { %>
<button type="submit" class="button button-primary button-<%- provider.name %> login-provider" data-provider-url="<%- provider.registerUrl %>"> <button type="submit" class="button button-primary button-<%- provider.name %> login-provider register-<%- provider.name %>" data-provider-url="<%- provider.registerUrl %>">
<span class="icon <%- provider.iconClass %>" aria-hidden="true"></span> <span class="icon <%- provider.iconClass %>" aria-hidden="true"></span>
<%- _.sprintf(gettext("Register using %(name)s"), provider) %> <%- _.sprintf(gettext("Register using %(name)s"), provider) %>
</button> </button>
...@@ -24,5 +24,5 @@ ...@@ -24,5 +24,5 @@
<%= context.fields %> <%= context.fields %>
<button class="action action-primary action-update js-register"><%- gettext("Register") %></button> <button class="action action-primary action-update js-register register-button"><%- gettext("Register") %></button>
</form> </form>
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