Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Standard Lock auth button for google is not compliant with Google Sign-In Branding Guidelines #1510

Closed
alexander-ignatow opened this issue Oct 17, 2018 · 3 comments · Fixed by #1512

Comments

@alexander-ignatow
Copy link

Default Google sign in button (as of v 11.10.0):

image

That look is not compliant with Google's Sign In Branding Guidelines

That issue caused our app to become "Not verified by Google"

As a temporary workaround I have added following CSS to fix the issue:

<style>
    .auth0-lock-social-button[data-provider^=google] .auth0-lock-social-button-icon {
      background-image: url("https://developers.google.com/identity/images/g-logo.png");
    }
    .auth0-lock.auth0-lock .auth0-lock-social-button.auth0-lock-social-big-button[data-provider^=google] .auth0-lock-social-button-icon,
    .auth0-lock.auth0-lock .auth0-lock-social-button.auth0-lock-social-big-button[data-provider^=google]:hover .auth0-lock-social-button-icon{
      background-color: white;
      border: 1px #4285f4 solid;
      border-radius: 3px 0px 0px 3px;
      height: 38px;
    }
  </style>
@luisrudge
Copy link
Contributor

@alexander-ignatow can you send a print screen with the end result?

@alexander-ignatow
Copy link
Author

The result of implementing my fix:
image

What Google expect:
image

However Google has not responded anything on my fixed app yet, so I cannot be 100% sure that the fix works for them.
For instance the button text uses default font (Google want Roboto) and comes in all caps

@luisrudge
Copy link
Contributor

Ok, let me talk with the design team. I'll get back to you on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants